DescriptionRevert of SurfaceManager::CreateSurface must set the surface's factory (patchset #6 id:120001 of https://codereview.chromium.org/2760433002/ )
Reason for revert:
Should not be necessary anymore now that crbug.com/701988 is fixed.
Original issue's description:
> SurfaceManager::CreateSurface must set the surface's factory
>
> This change is necessary because DelegatedFrameHost can be destroyed
> and recreated when RendererCompositorFrameSink is alive the whole time
> and can potentially send the same LocalSurfaceId to both instances of
> DelegatedFrameHost.
>
> BUG=697864
>
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
>
> Review-Url: https://codereview.chromium.org/2760433002
> Cr-Commit-Position: refs/heads/master@{#458138}
> Committed: https://chromium.googlesource.com/chromium/src/+/9e0fd9a87d69362103af512b8e7b8946604abe71
TBR=fsamuel@chromium.org,jbauman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=697864
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
Review-Url: https://codereview.chromium.org/2794993002
Cr-Commit-Position: refs/heads/master@{#461535}
Committed: https://chromium.googlesource.com/chromium/src/+/5ecb82217368349f41e3227f28cd238f26e04d50
Patch Set 1 #Patch Set 2 : Rebase #
Messages
Total messages: 23 (19 generated)
|