Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6)

Issue 103163003: Revert 238458 "cc: Defer first OutputSurface creation until clie..." (Closed)

Created:
7 years ago by robliao
Modified:
7 years ago
Reviewers:
no sievers
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 238458 "cc: Defer first OutputSurface creation until clie..." Reason: Candidate for Mac 10.6 and 10.7 Break - IOSurfaceReuse > cc: Defer first OutputSurface creation until client is ready > > Remove the |first_output_surface| which was not used before > the client signals ready. > This allows the client to wait before creating a graphics context > until the gpu thread and client channel are set up. > > BUG=270179 > R=danakj@chromium.org, jamesr@chromium.org, jochen@chromium.org > > Review URL: https://codereview.chromium.org/85693007 TBR=sievers@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=238470

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+105 lines, -172 lines) Patch
M trunk/src/cc/test/fake_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/cc/test/layer_tree_test.h View 2 chunks +4 lines, -5 lines 0 comments Download
M trunk/src/cc/test/layer_tree_test.cc View 3 chunks +5 lines, -7 lines 0 comments Download
M trunk/src/cc/trees/layer_tree_host.h View 2 chunks +4 lines, -4 lines 0 comments Download
M trunk/src/cc/trees/layer_tree_host.cc View 3 chunks +18 lines, -10 lines 0 comments Download
M trunk/src/cc/trees/layer_tree_host_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/cc/trees/layer_tree_host_unittest_context.cc View 4 chunks +38 lines, -127 lines 0 comments Download
M trunk/src/cc/trees/proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/cc/trees/single_thread_proxy.h View 2 chunks +5 lines, -1 line 0 comments Download
M trunk/src/cc/trees/single_thread_proxy.cc View 2 chunks +6 lines, -3 lines 0 comments Download
M trunk/src/cc/trees/thread_proxy.h View 2 chunks +4 lines, -1 line 0 comments Download
M trunk/src/cc/trees/thread_proxy.cc View 3 chunks +6 lines, -3 lines 0 comments Download
M trunk/src/content/renderer/gpu/render_widget_compositor.h View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/content/renderer/gpu/render_widget_compositor.cc View 3 chunks +4 lines, -3 lines 0 comments Download
M trunk/src/content/test/test_webkit_platform_support.cc View 1 chunk +2 lines, -1 line 0 comments Download
M trunk/src/content/test/web_layer_tree_view_impl_for_testing.h View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/content/test/web_layer_tree_view_impl_for_testing.cc View 2 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
robliao
7 years ago (2013-12-03 22:57:25 UTC) #1
robliao
7 years ago (2013-12-03 22:57:46 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 manually as r238470.

Powered by Google App Engine
This is Rietveld 408576698