Added support for multiple sink observers per frame.

Added support for multiple sink observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org,mfoltz@chromium.org

Committed: https://crrev.com/fc8d378ad2b8f61081cc74ddef519023aab79a2c
Cr-Commit-Position: refs/heads/master@{#378095}

[btolsch, 2016-02-26 22:11:14]

Description was changed from

==========
Added support for multiple observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org
==========

to

==========
Added support for multiple sink observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org,mfoltz@chromium.org
==========

[btolsch, 2016-02-26 22:28:24]

btolsch@chromium.org changed reviewers:
+ mfoltz@chromium.org

[btolsch, 2016-02-26 22:29:43]

Here is my patch for supporting multiple sink observers per frame in
PresentationServiceDelegateImpl. PTAL.

[mark a. foltz, 2016-02-27 00:28:13]

LGTM, nice patch

https://codereview.chromium.org/1744673002/diff/1/chrome/browser/media/router...
File chrome/browser/media/router/presentation_service_delegate_impl_unittest.cc
(right):

https://codereview.chromium.org/1744673002/diff/1/chrome/browser/media/router...
chrome/browser/media/router/presentation_service_delegate_impl_unittest.cc:198:
std::string presentation_url1("http://url1.fakeUrl");
Nit: Please use a valid TLD like .com, .org etc.

[btolsch, 2016-02-27 01:56:39]

The CQ bit was checked by btolsch@chromium.org

[btolsch, 2016-02-27 01:56:40]

The patchset sent to the CQ was uploaded after l-g-t-m from mfoltz@chromium.org
Link to the patchset: https://codereview.chromium.org/1744673002/#ps20001
(title: "Fixed url nit and SmallMap TODO")

[commit-bot: I haz the power, 2016-02-27 01:57:57]

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

[commit-bot: I haz the power, 2016-02-27 02:56:35]

Description was changed from

==========
Added support for multiple sink observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org,mfoltz@chromium.org
==========

to

==========
Added support for multiple sink observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org,mfoltz@chromium.org
==========

[commit-bot: I haz the power, 2016-02-27 02:56:36]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-02-27 02:59:35]

Description was changed from

==========
Added support for multiple sink observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org,mfoltz@chromium.org
==========

to

==========
Added support for multiple sink observers per frame.

PresentationServiceDelegate was incorrectly storing only the most recent
listener that was added per frame. This also adds a tests for this case.

BUG=589706
R=imcheng@chromium.org,mfoltz@chromium.org

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

[commit-bot: I haz the power, 2016-02-27 02:59:36]

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