[Sync] Ensure new non-frontend datatypes release shared change processor

[Sync] Ensure new non-frontend datatypes release shared change processor

If we failed to startup, Stop() doesn't get called, so ensure we cleanup up 
properly in StartDoneImpl.

BUG=102009
TEST=unit_tests


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=107820

[Nicolas Zea, 2011-10-28 00:34:27]

George as FYI, this should fix the unit tests issue you were running into.
Fred for review.

PTAL. This is what happens when the only tests use mocks :-/

[GeorgeY, 2011-10-28 18:03:36]

On 2011/10/28 00:34:27, nzea wrote:
> George as FYI, this should fix the unit tests issue you were running into.
> Fred for review.
> 
> PTAL. This is what happens when the only tests use mocks :-/

LGTM

[akalin, 2011-10-28 21:20:32]

On 2011/10/28 18:03:36, GeorgeY wrote:
> On 2011/10/28 00:34:27, nzea wrote:
> > George as FYI, this should fix the unit tests issue you were running into.
> > Fred for review.
> > 
> > PTAL. This is what happens when the only tests use mocks :-/
> 
> LGTM

LGTM

[commit-bot: I haz the power, 2011-10-28 21:33:16]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/zea@chromium.org/8341091/2001

[commit-bot: I haz the power, 2011-10-28 23:21:44]

Change committed as 107820