[Sync] Move BridgedSyncNotifier and ChromeSyncNotificationBridge to glue/

[Sync] Move BridgedSyncNotifier and ChromeSyncNotificationBridge to glue/

sync_notifier isn't supposed to have dependencies on chrome.

BUG=113723
TEST=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=124381

[akalin, 2012-02-29 02:19:52]

+zea for review

https://chromiumcodereview.appspot.com/9512005/diff/1/chrome/browser/sync/not...
File chrome/browser/sync/notifier/sync_notifier_factory_unittest.cc (left):

https://chromiumcodereview.appspot.com/9512005/diff/1/chrome/browser/sync/not...
chrome/browser/sync/notifier/sync_notifier_factory_unittest.cc:102: //
BridgedSyncNotifier) and then trigger a sync refresh notification. The
Not sure if there's a good place to put these tests.

[Nicolas Zea, 2012-02-29 21:51:37]

Is this is needed only because of the chrome_notification_types dependency? I
still think I'd prefer not having a public refresh notification, but not sure
how else to do it. :-/

With this though, the +chrome/common/chrome_notification_types.h line in
notifier/DEPS can be removed though, right?

[akalin, 2012-03-01 00:56:08]

On 2012/02/29 21:51:37, nzea wrote:
> Is this is needed only because of the chrome_notification_types dependency? I
> still think I'd prefer not having a public refresh notification, but not sure
> how else to do it. :-/

Yeah.  What do you mean by 'public', though?  I mean it was pretty much public
before -- anything could just send the right notification.

> With this though, the +chrome/common/chrome_notification_types.h line in
> notifier/DEPS can be removed though, right?

Right, done.

PTAL.

[Nicolas Zea, 2012-03-01 01:04:23]

By not public I mean something internal to sync. Basically, my instinct is to
prefer we had some sync internal notification we listened to, instead of the
chrome one, that way we can ensure we control who triggers this. But yeah.

LGTM

[akalin, 2012-03-01 03:22:01]

Had to add a few more DEPS to glue/*

tim, if you object to any of these, let me know, otherwise, I'll commit later
tonight.

[akalin, 2012-03-01 07:44:35]

On 2012/03/01 03:22:01, akalin wrote:
> Had to add a few more DEPS to glue/*
> 
> tim, if you object to any of these, let me know, otherwise, I'll commit later
> tonight.

I don't know what's up with check_deps on the trybot -- it's referring to a file
that's deleted (moved) in this CL.

I'll just check in and keep an eye on the tree.