commit | da15621ef5d721af63de8c4f0b013617abfeac0d | [log] [tgz] |
---|---|---|
author | danakj <[email protected]> | Fri Apr 05 20:34:43 2019 |
committer | Commit Bot <[email protected]> | Fri Apr 05 20:34:43 2019 |
tree | f4a0130d15d8872fb1c80b0e73b3f5462da97019 | |
parent | 79b7efd18bc7f0d329f8588a0d75f7d2f21d1450 [diff] |
Reland "Move RequestDecode and RequestPresentationCallback from WebLayerTreeView (8/n)" This is a reland of 07221eb8f25da04080febdc4f18e306eb6d612a5 Original change's description: > Move RequestDecode and RequestPresentationCallback from WebLayerTreeView (8/n) > > The RequestDecode() method moves to WebWidgetClient, and is implemented > in RenderWidget. > > The RequestPresentationCallback() is removed, and a method is added as > WebWidgetTestProxy::RequestCompositeAndPresentation() for test_runner to use > instead. It acts like RequestPresentation() but forces the scheduled sync > composite to run the full compositor stack instead of just the main frame > animation step. > > This removes the LayerTreeView::SynchronouslyComposite() method and > changes callers to it to call WebWidgetTestProxy::ScheduleAnimation() > instead of duplicating that logic out in production code. > > This moves ~60 lines of production code into content/shell/test_runner/ > to remove them from chrome.exe, as well as straight up removing another > ~80 lines of production code. Now RenderWidget and LayerTreeView > have no concept of synchronous composites, as they are entirely > managed by WebWidgetTestProxy. > > We now have only one caller of LayerTreeHost::Composite() in web tests > instead of previously having 4. > > [email protected] > > Bug: 912193 > Change-Id: I26cd9cbbe503e9b876a146f2eebccbf8e9121190 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544820 > Commit-Queue: Philip Rogers <[email protected]> > Reviewed-by: Philip Rogers <[email protected]> > Reviewed-by: Avi Drissman <[email protected]> > Auto-Submit: danakj <[email protected]> > Cr-Commit-Position: refs/heads/master@{#645998} Bug: 912193 Change-Id: I33a171a7efa054d716fa9f31432734c500557bb3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549414 Reviewed-by: Avi Drissman <[email protected]> Reviewed-by: Daniel Cheng <[email protected]> Commit-Queue: danakj <[email protected]> Cr-Commit-Position: refs/heads/master@{#648317}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .