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

Issue 6544009: Merge 72458 - [SYNC] Speculative fix for autofill change processor destroying... (Closed)

Created:
9 years, 10 months ago by Nicolas Zea
Modified:
9 years, 7 months ago
Reviewers:
Nicolas Zea
CC:
chromium-reviews, Raghu Simha, ncarter (slow), tim (not reviewing), idana
Visibility:
Public.

Description

Merge 72458 - [SYNC] Speculative fix for autofill change processor destroying on the wrong thread. The data type manager should now tell any datatype controller that is not already stopping or not running to stop. Namely, this covers the MODEL_STARTING and ASSOCIATING cases, which weren't being covered before. Tests that can reproduce these race conditions still needed. BUG=61804 TEST=unit,sync_integration,self Review URL: http://codereview.chromium.org/6273011 TBR=zea@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=75346

Patch Set 1 #

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

Messages

Total messages: 1 (0 generated)
Nicolas Zea
9 years, 10 months ago (2011-02-18 01:24:39 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698