Add the InvalidationState struct to bundle local state about invalidations.

Add the InvalidationState struct to bundle local state about invalidations.

Currently this only has one member, but I anticipate adding several more
as we add support for local acknowledgements.

BUG=124149
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=163077

[akalin, 2012-10-19 21:21:26]

lgtm

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
File chrome/browser/sync/invalidations/invalidator_storage.cc (right):

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
chrome/browser/sync/invalidations/invalidator_storage.cc:27:
invalidation::ObjectId* id,
fix the indent here

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
File chrome/browser/sync/invalidations/invalidator_storage_unittest.cc (right):

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
chrome/browser/sync/invalidations/invalidator_storage_unittest.cc:20: bool
operator==(const InvalidationState& lhs, const InvalidationState& rhs) {
yeah, i don't quite like this.  but it's yet another yak to shave -- maybe add a
TODO to replace with Equals?

[dcheng, 2012-10-19 22:24:40]

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
File chrome/browser/sync/invalidations/invalidator_storage.cc (right):

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
chrome/browser/sync/invalidations/invalidator_storage.cc:27:
invalidation::ObjectId* id,
On 2012/10/19 21:21:26, akalin wrote:
> fix the indent here

Done.

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
File chrome/browser/sync/invalidations/invalidator_storage_unittest.cc (right):

http://codereview.chromium.org/11229015/diff/1/chrome/browser/sync/invalidati...
chrome/browser/sync/invalidations/invalidator_storage_unittest.cc:20: bool
operator==(const InvalidationState& lhs, const InvalidationState& rhs) {
On 2012/10/19 21:21:26, akalin wrote:
> yeah, i don't quite like this.  but it's yet another yak to shave -- maybe add
a
> TODO to replace with Equals?

Eventually I'd like to write a template helper for this. We already have a lot
of things that compare maps where we need to use Equals() for the value, and it
seems silly to keep copying and pasting that code.