GitLab

This method, used for debugging (chrome://sync-internals), has no clear
semantics if a datatype hasn't been started. This is specially true with
future work in mind where we plan to introduce different operating modes
(regular vs ephemeral) that are bound to a start-stop configuration
cycle.

We also migrate away from callers having to use BindToCurrentSequence(),
since there seems to be no reason for it (and is really weird after
having recently introduced ModelTypeControllerDelegate).

Bug: 866814
Change-Id: I653e1ee117883feb42e032faac85d5ba14d28748
Reviewed-on: https://chromium-review.googlesource.com/1160229Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580141}