|
[Sync] Add tests for invalid specifics field number handling
This is a follow up to r172232.
Change NOTREACHED() to DLOG(WARNING), since the server sending down
unknown/invalid field numbers is a valid event.
Add tests for the code that uses GetModelTypeFromSpecificsFieldNumber().
Clean up some code in sync/engine/ a bit.
BUG= 165171
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=172816
Total comments: 2
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+270 lines, -131 lines) |
Patch |
|
M |
sync/engine/DEPS
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
|
M |
sync/engine/store_timestamps_command.h
|
View
|
|
1 chunk |
+16 lines, -0 lines |
2 comments
|
Download
|
|
M |
sync/engine/store_timestamps_command.cc
|
View
|
|
1 chunk |
+22 lines, -17 lines |
0 comments
|
Download
|
|
A |
sync/engine/store_timestamps_command_unittest.cc
|
View
|
|
1 chunk |
+83 lines, -0 lines |
0 comments
|
Download
|
|
M |
sync/engine/syncer_proto_util.h
|
View
|
|
4 chunks |
+13 lines, -8 lines |
0 comments
|
Download
|
|
M |
sync/engine/syncer_proto_util.cc
|
View
|
|
6 chunks |
+95 lines, -100 lines |
0 comments
|
Download
|
|
M |
sync/engine/syncer_proto_util_unittest.cc
|
View
|
|
3 chunks |
+38 lines, -5 lines |
0 comments
|
Download
|
|
M |
sync/internal_api/public/base/model_type.h
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
sync/sync.gyp
|
View
|
|
1 chunk |
+1 line, -0 lines |
0 comments
|
Download
|
Total messages: 5 (0 generated)
|