Description[Sync] Post back to UI thread for USS GetAllNodes.
When running GetAllNodes, USS types post to the model thread to access
their processor/bridge. This means after they've gathered the nodes,
they need to return to the UI thread to give the results back to the
GetAllNodesRequestHelper.
Unfortunately I removed a wrapping BindToCurrentThread call in
https://codereview.chromium.org/2672493002/diff/80001/components/sync/driver/model_type_controller.cc
that did the jump back to the UI thread. This meant that non-UI USS
types (namely AUTOFILL) were updating GetAllNodesRequestHelper from a
different thread, through very non-thread safe code. To fix simply
added the BindToCurrentThread back.
Also added a added a thread checker to GetAllNodesRequestHelper in
hopes of stopping similar bugs from occurring again.
BUG=686172
Review-Url: https://codereview.chromium.org/2808113003
Cr-Commit-Position: refs/heads/master@{#463320}
Committed: https://chromium.googlesource.com/chromium/src/+/903ebf3235f4057351f7734c0e148e7938009dbd
Patch Set 1 #
Messages
Total messages: 17 (12 generated)
|