Chromium Code Reviews
DescriptionRevert of Support listening for available screens for multiple presentation urls in the same frame (patchset #2 id:20001 of https://codereview.chromium.org/906113002/)
Reason for revert:
Bot failure: http://build.chromium.org/p/chromium.webkit/builders/GPU%20Linux%20Builder%20%28dbg%29/builds/28448
Original issue's description:
> Support listening for available screens for multiple
> Presentation URLs on a per frame basis.
>
> This is done by associating PresentationServiceImpl
> with the RenderFrameHost. When Presentation API
> requests are passed to PresentationServiceImpl, it
> will delegate to the embedder's Media Router along
> with the frame id via the PresentationServiceDelegate
> interface.
>
> Each PresentationServiceImpl also maintains a hash
> map of Presentation URLs currently registered to
> "context" objects. These objects wait for both result
> and renderer (i.e. callback) to both be ready before
> sending the result back to renderer /
> PresentationDispatcher.
>
> Furthermore, PresentationServiceImpl listens for
> changes to the RenderFrameHost. If an out of page
> navigation occurred or if RenderFrameHost is
> being destroyed, all current Presentation URLs
> are unregistered.
>
> Also added unit tests for PresentationSerivceImpl.
>
> BUG=412331
>
> Committed: https://crrev.com/ec17a20355dbc3b05e2133c0112c332167b63ce7
> Cr-Commit-Position: refs/heads/master@{#315442}
TBR=avi@chromium.org,avayvod@chromium.org,imcheng@chromium.org,imcheng@google.com
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=412331
Committed: https://crrev.com/9e4a7ad0e922d3f6a9d648e2ac1696a5136ea87f
Cr-Commit-Position: refs/heads/master@{#315449}
Patch Set 1 #
Created: 5 years, 10 months ago
(Patch set is too large to download)
Messages
Total messages: 4 (0 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||