[Sync] Only get bridge on model thread for ModelTypeController.

[Sync] Only get bridge on model thread for ModelTypeController.

AutocompleteSyncBridge cannot be fetched on the UI thread, because of
the way AutofillWebDataService provides it. This can only be done on
the model thread. Previously the ModelTypeController was updated to
mostly respect this in https://codereview.chromium.org/2508263003/ .
However, we seem to have missed the DisableSync() case. This CL copies
the existing pattern for this scenario. 

BUG=686172
Review-Url: https://codereview.chromium.org/2668323002
Cr-Commit-Position: refs/heads/master@{#447779}
Committed: https://chromium.googlesource.com/chromium/src/+/c3389b63ea8217144db344e29451b9ee3bbd05e3

[skym, 2017-02-01 18:31:17]

The CQ bit was checked by skym@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-01 18:32:06]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-01 19:24:33]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-01 19:24:33]

Dry run: This issue passed the CQ dry run.

[skym, 2017-02-01 20:15:06]

Description was changed from

==========
[Sync] Only get bridge on model thread for ModelTypeController.

BUG=686172
==========

to

==========
[Sync] Only get bridge on model thread for ModelTypeController.

AutocompleteSyncBridge cannot be fetched on the UI thread, because of
the way AutofillWebDataService provides it. This can only be done on
the model thread. Previously the ModelTypeController was updated to
mostly respect this in https://codereview.chromium.org/2508263003/ .
However, we seem to have missed the DisableSync() case. This CL copies
the existing pattern for this scenario. 

BUG=686172
==========

[skym, 2017-02-01 20:15:07]

skym@chromium.org changed reviewers:
+ pnoland@chromium.org

[skym, 2017-02-01 20:16:45]

PTAL

[Patrick Noland, 2017-02-01 22:46:12]

lgtm

[skym, 2017-02-02 16:42:08]

The CQ bit was checked by skym@chromium.org

[commit-bot: I haz the power, 2017-02-02 16:42:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-02 16:42:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-02 16:42:31]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[skym, 2017-02-02 16:43:47]

skym@chromium.org changed reviewers:
+ maxbogue@chromium.org

[skym, 2017-02-02 16:44:04]

PTAL Max

[maxbogue, 2017-02-02 16:48:31]

lgtm

[skym, 2017-02-02 16:49:24]

The CQ bit was checked by skym@chromium.org

[commit-bot: I haz the power, 2017-02-02 16:49:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-02 16:57:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1486054164697230, "parent_rev":
"554935dc2ad0cf716ae2eb822800524ca18a6bee", "commit_rev":
"c3389b63ea8217144db344e29451b9ee3bbd05e3"}

[commit-bot: I haz the power, 2017-02-02 16:58:18]

Description was changed from

==========
[Sync] Only get bridge on model thread for ModelTypeController.

AutocompleteSyncBridge cannot be fetched on the UI thread, because of
the way AutofillWebDataService provides it. This can only be done on
the model thread. Previously the ModelTypeController was updated to
mostly respect this in https://codereview.chromium.org/2508263003/ .
However, we seem to have missed the DisableSync() case. This CL copies
the existing pattern for this scenario. 

BUG=686172
==========

to

==========
[Sync] Only get bridge on model thread for ModelTypeController.

AutocompleteSyncBridge cannot be fetched on the UI thread, because of
the way AutofillWebDataService provides it. This can only be done on
the model thread. Previously the ModelTypeController was updated to
mostly respect this in https://codereview.chromium.org/2508263003/ .
However, we seem to have missed the DisableSync() case. This CL copies
the existing pattern for this scenario. 

BUG=686172

Review-Url: https://codereview.chromium.org/2668323002
Cr-Commit-Position: refs/heads/master@{#447779}
Committed:
https://chromium.googlesource.com/chromium/src/+/c3389b63ea8217144db344e29451...
==========

[commit-bot: I haz the power, 2017-02-02 16:58:19]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/c3389b63ea8217144db344e29451...