[Sync] Remove logging/checks for case where server and local entries mismatch.

[Sync] Remove logging/checks for case where server and local entries mismatch.

We will now reliably hit this code due to the server changing the MTIME of any
committed item. As a result of this, and because the check and log is expensive,
it's better to just remove them altogether.

BUG=103838
TEST=none


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=109761

[Nicolas Zea, 2011-11-11 02:22:12]

PTAL. I had to remove both cases we discussed. I verified that, with the
testserver change they were both hit, and once removed everything applied
successfully, with no conflicts or error logs.

[ncarter (slow), 2011-11-11 03:07:53]

lgtm

LGTM; but are there really no test changes required?

http://codereview.chromium.org/8526018/diff/2001/net/tools/testserver/chromiu...
File net/tools/testserver/chromiumsync.py (right):

http://codereview.chromium.org/8526018/diff/2001/net/tools/testserver/chromiu...
net/tools/testserver/chromiumsync.py:470: self._entries[entry.id_string].mtime =
\
Don't escape newlines, use parens instead.

[Nicolas Zea, 2011-11-11 19:23:33]

Done. Committing once trybots go green (and no, there didn't appear to be any
test failures when I ran locally, so I guess we had no coverage of this code
:-/)

http://codereview.chromium.org/8526018/diff/2001/net/tools/testserver/chromiu...
File net/tools/testserver/chromiumsync.py (right):

http://codereview.chromium.org/8526018/diff/2001/net/tools/testserver/chromiu...
net/tools/testserver/chromiumsync.py:470: self._entries[entry.id_string].mtime =
\
On 2011/11/11 03:07:53, ncarter wrote:
> Don't escape newlines, use parens instead.

Done.

[commit-bot: I haz the power, 2011-11-12 01:33:25]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zea@chromium.org/8526018/7001

[commit-bot: I haz the power, 2011-11-12 02:46:34]

Change committed as 109761