|
Revert of Re-land: cc: Use worker context for one-copy tile initialization. (patchset #29 id:560001 of https://codereview.chromium.org/1230203007/ )
Reason for revert:
Sorry Dave. I'm reverting as this broke mandoline. To see the breakage build mandoline:all, and run with:
mandoline --oopif-always-create-new-frame-tree http://phuang.github.io/iframe.html .
We do have mandoline tests that run, but they run with the code path gfx::GLSurfaceTestSupport::InitializeOneOff(), which is why they likely still pass. I'll see about making the tests not run with that config.
Original issue's description:
> Re-land: cc: Use worker context for one-copy tile initialization.
>
> This moves management of staging resources to
> OneCopyTileTaskWorkerPool class. This makes it possible
> to use a worker context to issue and detect when copy
> operations complete.
>
> BUG= 490295
> TBR=jamesr@chromium.org, sky@chromium.org
> CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
>
> Committed: https://crrev.com/af0ab609b53c64ce43172be42e20e7ca8beab497
> Cr-Commit-Position: refs/heads/master@{#343947}
TBR=jamesr@chromium.org,kbr@chromium.org,piman@chromium.org,primiano@chromium.org,vmiura@chromium.org,vmpstr@chromium.org,reveman@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 490295
Committed: https://crrev.com/c4967eface72bc99452d8a83800c7cdd19a872ee
Cr-Commit-Position: refs/heads/master@{#344042}
|
Unified diffs |
Side-by-side diffs |
Delta from patch set |
Stats (+854 lines, -798 lines) |
Patch |
|
M |
cc/output/context_provider.h
|
View
|
|
2 chunks |
+0 lines, -22 lines |
0 comments
|
Download
|
|
M |
cc/output/output_surface.cc
|
View
|
|
4 chunks |
+19 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/raster/gpu_tile_task_worker_pool.cc
|
View
|
|
2 chunks |
+11 lines, -4 lines |
0 comments
|
Download
|
|
M |
cc/raster/one_copy_tile_task_worker_pool.h
|
View
|
|
5 chunks |
+60 lines, -63 lines |
0 comments
|
Download
|
|
M |
cc/raster/one_copy_tile_task_worker_pool.cc
|
View
|
|
11 chunks |
+339 lines, -495 lines |
0 comments
|
Download
|
|
M |
cc/raster/tile_task_worker_pool_perftest.cc
|
View
|
|
2 chunks |
+5 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/raster/tile_task_worker_pool_unittest.cc
|
View
|
|
3 chunks |
+4 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/resources/resource_format.h
|
View
|
|
2 chunks |
+0 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/resources/resource_format.cc
|
View
|
|
1 chunk |
+0 lines, -19 lines |
0 comments
|
Download
|
|
M |
cc/resources/resource_pool.h
|
View
|
|
1 chunk |
+4 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/resources/resource_pool.cc
|
View
|
|
1 chunk |
+4 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/resources/resource_pool_unittest.cc
|
View
|
|
4 chunks |
+8 lines, -5 lines |
0 comments
|
Download
|
|
M |
cc/resources/resource_provider.h
|
View
|
|
7 chunks |
+15 lines, -3 lines |
0 comments
|
Download
|
|
M |
cc/resources/resource_provider.cc
|
View
|
|
10 chunks |
+153 lines, -3 lines |
0 comments
|
Download
|
|
M |
cc/resources/resource_provider_unittest.cc
|
View
|
|
25 chunks |
+109 lines, -25 lines |
0 comments
|
Download
|
|
M |
cc/surfaces/display.cc
|
View
|
|
1 chunk |
+3 lines, -0 lines |
0 comments
|
Download
|
|
M |
cc/surfaces/surface_display_output_surface.h
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
cc/surfaces/surface_display_output_surface.cc
|
View
|
|
1 chunk |
+2 lines, -3 lines |
0 comments
|
Download
|
|
M |
cc/surfaces/surface_display_output_surface_unittest.cc
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
cc/test/fake_output_surface.h
|
View
|
|
1 chunk |
+3 lines, -5 lines |
0 comments
|
Download
|
|
M |
cc/test/fake_resource_provider.h
|
View
|
|
4 chunks |
+4 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/test/layer_tree_pixel_resource_test.h
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/test/layer_tree_pixel_resource_test.cc
|
View
|
|
6 chunks |
+21 lines, -6 lines |
0 comments
|
Download
|
|
M |
cc/test/layer_tree_pixel_test.h
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/test/layer_tree_pixel_test.cc
|
View
|
|
1 chunk |
+0 lines, -6 lines |
0 comments
|
Download
|
|
M |
cc/test/layer_tree_test.h
|
View
|
|
1 chunk |
+2 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/test/layer_tree_test.cc
|
View
|
|
2 chunks |
+6 lines, -4 lines |
0 comments
|
Download
|
|
M |
cc/test/pixel_test.cc
|
View
|
|
2 chunks |
+2 lines, -2 lines |
0 comments
|
Download
|
|
M |
cc/test/test_in_process_context_provider.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/tiles/tile_manager.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
cc/trees/layer_tree_host_impl.h
|
View
|
|
2 chunks |
+3 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/trees/layer_tree_host_impl.cc
|
View
|
|
9 chunks |
+32 lines, -7 lines |
0 comments
|
Download
|
|
M |
cc/trees/layer_tree_host_pixeltest_tiles.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/trees/layer_tree_settings.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
cc/trees/layer_tree_settings.cc
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
components/html_viewer/web_layer_tree_view_impl.cc
|
View
|
|
1 chunk |
+3 lines, -7 lines |
0 comments
|
Download
|
|
M |
content/browser/compositor/browser_compositor_output_surface.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/browser/compositor/browser_compositor_output_surface.cc
|
View
|
|
1 chunk |
+3 lines, -5 lines |
0 comments
|
Download
|
|
M |
content/browser/compositor/gpu_browser_compositor_output_surface.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/browser/compositor/gpu_browser_compositor_output_surface.cc
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
|
M |
content/browser/compositor/gpu_process_transport_factory.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/browser/compositor/gpu_process_transport_factory.cc
|
View
|
|
5 chunks |
+7 lines, -30 lines |
0 comments
|
Download
|
|
M |
content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/browser/compositor/gpu_surfaceless_browser_compositor_output_surface.cc
|
View
|
|
2 chunks |
+0 lines, -2 lines |
0 comments
|
Download
|
|
M |
content/browser/compositor/offscreen_browser_compositor_output_surface.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/browser/compositor/offscreen_browser_compositor_output_surface.cc
|
View
|
|
1 chunk |
+0 lines, -2 lines |
0 comments
|
Download
|
|
M |
content/browser/compositor/reflector_impl_unittest.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/browser/gpu/compositor_util.cc
|
View
|
|
1 chunk |
+8 lines, -0 lines |
0 comments
|
Download
|
|
M |
content/browser/renderer_host/compositor_impl_android.cc
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
content/common/gpu/client/command_buffer_metrics.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
content/common/gpu/client/command_buffer_metrics.cc
|
View
|
|
2 chunks |
+0 lines, -6 lines |
0 comments
|
Download
|
|
M |
content/renderer/gpu/render_widget_compositor.cc
|
View
|
|
1 chunk |
+0 lines, -5 lines |
0 comments
|
Download
|
|
M |
gpu/command_buffer/build_gles2_cmd_buffer.py
|
View
|
|
1 chunk |
+1 line, -1 line |
0 comments
|
Download
|
|
M |
gpu/command_buffer/client/cmd_buffer_helper.cc
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
gpu/command_buffer/service/query_manager.cc
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
mojo/cc/output_surface_mojo.h
|
View
|
|
1 chunk |
+3 lines, -5 lines |
0 comments
|
Download
|
|
M |
mojo/cc/output_surface_mojo.cc
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
mojo/gpu/mojo_gles2_impl_autogen.cc
|
View
|
|
1 chunk |
+1 line, -2 lines |
0 comments
|
Download
|
|
M |
third_party/mojo/src/mojo/public/c/gles2/gles2_call_visitor_chromium_miscellaneous_autogen.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
ui/compositor/test/in_process_context_factory.cc
|
View
|
|
3 chunks |
+9 lines, -14 lines |
0 comments
|
Download
|
|
M |
ui/compositor/test/in_process_context_provider.h
|
View
|
|
1 chunk |
+0 lines, -1 line |
0 comments
|
Download
|
|
M |
ui/compositor/test/in_process_context_provider.cc
|
View
|
|
1 chunk |
+0 lines, -4 lines |
0 comments
|
Download
|
Total messages: 4 (0 generated)
|