Fix up notifications so that they aren't only on Browser IOThread.

Fix up notifications so that they aren't only on Browser IOThread.

I wasn't thinking and realized I need to get rid of the dependency on Browser IOThread, because the service process doesn't have one.
This means I can move this back to chrome/common, but I won't do that until you have finished your implementations.

BUG=NONE
TEST=BUILD

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

[dmac, 2011-01-19 22:40:34]

Would you gentlemen please take a look?

[dmac, 2011-01-19 23:14:41]

http://codereview.chromium.org/6312007/diff/1/chrome/browser/multi_process_no...
File chrome/browser/multi_process_notification_mac.mm (right):

http://codereview.chromium.org/6312007/diff/1/chrome/browser/multi_process_no...
chrome/browser/multi_process_notification_mac.mm:436: return false;
On 2011/01/19 22:49:40, garykac wrote:
> Log an error here?

Done.

http://codereview.chromium.org/6312007/diff/1/chrome/browser/multi_process_no...
File chrome/browser/multi_process_notification_unittest.cc (right):

http://codereview.chromium.org/6312007/diff/1/chrome/browser/multi_process_no...
chrome/browser/multi_process_notification_unittest.cc:131: base::Thread
io_thread;
On 2011/01/19 22:49:40, garykac wrote:
> io_thread_

Done.

[awong, 2011-01-19 23:26:59]

LGTM

[dmac, 2011-01-20 00:27:07]

Sorry guys... now that I'm using my class seeing some minor changes. Just added
name and domain accessors to listeners. Mind taking one more look?

[garykac, 2011-01-20 01:10:50]

On 2011/01/20 00:27:07, dmac wrote:
> Sorry guys... now that I'm using my class seeing some minor changes. Just
added
> name and domain accessors to listeners. Mind taking one more look?

LGTM