Add media device attach notification mechanism.

Add media device attach notification mechanism.

Split out from https://chromiumcodereview.appspot.com/9363008/

BUG=110400
TEST=NONE


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

[vandebo (ex-Chrome), 2012-03-02 19:15:07]

Will, can you give this a quick once over and stamp it?  I had to pull out the
framework changes from the Mac impl because there's a hard to find issue in the
Mac code and this is holding up the Win and Linux implementation.

[willchan no longer on Chromium, 2012-03-02 19:18:09]

On 2012/03/02 19:15:07, vandebo wrote:
> Will, can you give this a quick once over and stamp it?  I had to pull out the
> framework changes from the Mac impl because there's a hard to find issue in
the
> Mac code and this is holding up the Win and Linux implementation.

In the future, it'd be nice to upload the revert of the revert, and then upload
your change so I can see how it differs. Is there anything different other than
the removal of the mac implementation?

[vandebo (ex-Chrome), 2012-03-02 19:29:47]

On 2012/03/02 19:18:09, willchan wrote:
> On 2012/03/02 19:15:07, vandebo wrote:
> > Will, can you give this a quick once over and stamp it?  I had to pull out
the
> > framework changes from the Mac impl because there's a hard to find issue in
> the
> > Mac code and this is holding up the Win and Linux implementation.
> 
> In the future, it'd be nice to upload the revert of the revert, and then
upload
> your change so I can see how it differs. Is there anything different other
than
> the removal of the mac implementation?

Of course, sorry.  Patch Set 2 shows that the diff is just the mac stuff.  In
the previous CL there were a few changes in base after your LG - I changed a
SystemMonitor test to use a mock and moved it to the test_support_base target so
it could be used for the Windows implementation.

[willchan no longer on Chromium, 2012-03-02 19:33:16]

lgtm

[commit-bot: I haz the power, 2012-03-02 19:33:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/vandebo@chromium.org/9580018/4002

[commit-bot: I haz the power, 2012-03-02 23:13:43]

Change committed as 124763