We should not use const_cast for PacketHeader to set PeerGroupMsgIdNotSynchronized state

[yufengwangca]

Problem

We are storing the IsPeerGroupMsgIdNotSynchronized state in the PacketHeader, which seems like as lightly odd place to put it and requires const_cast. We should not be adding const_cast to the code.

    // For all group messages, Set flag if peer group key message counter is not synchronized.
    if (ChipKeyId::IsAppGroupKey(packetHeader.GetEncryptionKeyID()))
    {
        const_cast<PacketHeader &>(packetHeader).SetPeerGroupMsgIdNotSynchronized(true);
    }

Proposed Solution

Use SecureSessionHandle to indicate if the peer's group key message counter is not synchronized.