Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Issue 8822017: [Sync] Set HasCustomGroupsToChange() to true for ProcessUpdatesCommand (Closed)

Created:
9 years ago by akalin
Modified:
9 years ago
CC:
chromium-reviews, Raghu Simha, ncarter (slow), tim (not reviewing)
Visibility:
Public.

Description

[Sync] Set HasCustomGroupsToChange() to true for ProcessUpdatesCommand Follow-up to 113090 to see which ModelChangingSyncerCommand triggers a perf regression. BUG=97832 TEST= Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114090 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=114485

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -2 lines) Patch
M chrome/browser/sync/engine/process_updates_command.cc View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
akalin
+tim for review (although I may TBR this)
9 years ago (2011-12-06 17:25:38 UTC) #1
akalin
On 2011/12/06 17:25:38, akalin wrote: > +tim for review (although I may TBR this) Note ...
9 years ago (2011-12-06 17:26:04 UTC) #2
tim (not reviewing)
lgtm
9 years ago (2011-12-07 04:37:55 UTC) #3
akalin
9 years ago (2011-12-13 00:48:09 UTC) #4
On 2011/12/07 04:37:55, timsteele wrote:
> lgtm

This seems to have regressed both delete_typed_urls and some autofill profiles
stuff:
http://build.chromium.org/f/chromium/perf/mac-release-10.6/sync/report.html?h...

Powered by Google App Engine
This is Rietveld 408576698