Reland of Add support for device-change notifications to MediaStreamDispatcher

Reland of Add support for device-change notifications to MediaStreamDispatcher (patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
This reland should fix the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher and MediaStreamDispatcherHost (patchset #6 id:100001 of https://codereview.chromium.org/1918173002/ )
>
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
>
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
>
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
>
> TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.org,guidou@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
>
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.org,dpranke@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648
Committed: https://crrev.com/cb53c53e9816623ac51e9b151815897ba2a30b22
Cr-Commit-Position: refs/heads/master@{#391192}

[Guido Urdaneta, 2016-05-03 08:17:58]

Created Reland of Add support for device-change notifications to
MediaStreamDispatcher

[Guido Urdaneta, 2016-05-03 08:25:05]

Description was changed from

==========
Reland of Add support for device-change notifications to MediaStreamDispatcher
(patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
Reverting the revert and sending separate CL that fixes the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher
and MediaStreamDispatcherHost (patchset #6 id:100001 of
https://codereview.chromium.org/1918173002/ )
> 
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
> 
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
> 
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and
MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
> 
>
TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
> 
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648
==========

to

==========
Reland of Add support for device-change notifications to MediaStreamDispatcher
(patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
This reland should fix the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher
and MediaStreamDispatcherHost (patchset #6 id:100001 of
https://codereview.chromium.org/1918173002/ )
> 
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
> 
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
> 
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and
MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
> 
>
TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
> 
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648
==========

[Guido Urdaneta, 2016-05-03 08:28:18]

guidou@chromium.org changed reviewers:
+ perkj@chromium.org

[Guido Urdaneta, 2016-05-03 08:48:44]

perkj@, can you look at the fix in the unit test?

[perkj_chrome, 2016-05-03 08:58:30]

media_stream_dispatcher_unittest.cc changes lgtm
There are no other changes between ps #1 and #2.

(#1 is supposed to be a clean reland.)

[Guido Urdaneta, 2016-05-03 09:00:56]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2016-05-03 09:01:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1943213002/270001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1943213002/270001

[commit-bot: I haz the power, 2016-05-03 11:00:59]

Description was changed from

==========
Reland of Add support for device-change notifications to MediaStreamDispatcher
(patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
This reland should fix the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher
and MediaStreamDispatcherHost (patchset #6 id:100001 of
https://codereview.chromium.org/1918173002/ )
> 
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
> 
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
> 
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and
MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
> 
>
TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
> 
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648
==========

to

==========
Reland of Add support for device-change notifications to MediaStreamDispatcher
(patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
This reland should fix the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher
and MediaStreamDispatcherHost (patchset #6 id:100001 of
https://codereview.chromium.org/1918173002/ )
> 
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
> 
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
> 
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and
MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
> 
>
TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
> 
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648
==========

[commit-bot: I haz the power, 2016-05-03 11:01:00]

Committed patchset #2 (id:270001)

[commit-bot: I haz the power, 2016-05-03 11:02:22]

Description was changed from

==========
Reland of Add support for device-change notifications to MediaStreamDispatcher
(patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
This reland should fix the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher
and MediaStreamDispatcherHost (patchset #6 id:100001 of
https://codereview.chromium.org/1918173002/ )
> 
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
> 
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
> 
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and
MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
> 
>
TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
> 
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648
==========

to

==========
Reland of Add support for device-change notifications to MediaStreamDispatcher
(patchset #1 id:1 of https://codereview.chromium.org/1939313003/ )

Reason for revert:
This reland should fix the offending test.

Original issue's description:
> Revert of Add support for device-change notifications to MediaStreamDispatcher
and MediaStreamDispatcherHost (patchset #6 id:100001 of
https://codereview.chromium.org/1918173002/ )
>
> Reason for revert:
> This has caused content_browsertests to fail on multiple builders, e.g.:
>
> https://build.chromium.org/p/chromium.linux/builders/Cast%20Linux/builds/23972
> https://build.chromium.org/p/chromium.linux/builders/Android%20GN/builds/34086
>
> Original issue's description:
> > Add support for device-change notifications to MediaStreamDispatcher and
MediaStreamDispatcherHost
> >
> > This is a step towards supporting the "devicechange" event in Blink.
> >
> > BUG=388648
> >
> > Committed: https://crrev.com/ec334e1799ae25aaa1b0c66e25af641018a3a84a
> > Cr-Commit-Position: refs/heads/master@{#391114}
>
>
TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=388648
>
> Committed: https://crrev.com/2396227939dc83d23f2523a992b0e35da50d631e
> Cr-Commit-Position: refs/heads/master@{#391141}

TBR=tommi@chromium.org,palmer@chromium.org,bbudge@chromium.org,hta@chromium.o...
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=388648

Committed: https://crrev.com/cb53c53e9816623ac51e9b151815897ba2a30b22
Cr-Commit-Position: refs/heads/master@{#391192}
==========

[commit-bot: I haz the power, 2016-05-03 11:02:23]

Patchset 2 (id:??) landed as
https://crrev.com/cb53c53e9816623ac51e9b151815897ba2a30b22
Cr-Commit-Position: refs/heads/master@{#391192}

[henrika (OOO until Aug 14), 2016-05-03 11:50:07]

A revert of this CL (patchset #2 id:270001) has been created in
https://codereview.chromium.org/1945603002/ by henrika@chromium.org.

The reason for reverting is: Still seems to break tests. See e.g.

https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.9%20Tests/....