Chromium Code Reviews
[email protected] (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 14951002: Compositor method renames (Closed)

Created:
7 years, 7 months ago by boliu
Modified:
7 years, 7 months ago
Reviewers:
danakj, jamesr, no sievers, piman
CC:
chromium-reviews, jonathan.backer, Ian Vollick, jam, apatrick_chromium, joi+watch-content_chromium.org, darin-cc_chromium.org, piman+watch_chromium.org, cc-bugs_chromium.org
Visibility:
Public.

Description

Compositor method renames LayerTreeHostClient::DidRecreateOutputSurface > DidInitializeOutputSurface LayerTreeHost::SetSurfaceReady > SetLayerTreeHostClientReady TBR=sievers,jamesr,piman NOTRY=true BUG= Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=199023

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -42 lines) Patch
M cc/test/fake_layer_tree_host_client.h View 1 chunk +1 line, -1 line 0 comments Download
M cc/test/fake_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M cc/test/layer_tree_test.h View 1 1 chunk +1 line, -1 line 0 comments Download
M cc/test/layer_tree_test.cc View 1 2 chunks +3 lines, -3 lines 0 comments Download
M cc/trees/layer_tree_host.h View 1 chunk +1 line, -2 lines 0 comments Download
M cc/trees/layer_tree_host.cc View 3 chunks +4 lines, -4 lines 0 comments Download
M cc/trees/layer_tree_host_client.h View 1 chunk +1 line, -2 lines 0 comments Download
M cc/trees/layer_tree_host_unittest_context.cc View 1 9 chunks +9 lines, -9 lines 0 comments Download
M cc/trees/proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M cc/trees/single_thread_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M cc/trees/single_thread_proxy.cc View 1 chunk +1 line, -1 line 0 comments Download
M cc/trees/thread_proxy.h View 1 2 chunks +2 lines, -2 lines 0 comments Download
M cc/trees/thread_proxy.cc View 1 1 chunk +6 lines, -5 lines 0 comments Download
M content/browser/renderer_host/compositor_impl_android.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/compositor_impl_android.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/gpu/render_widget_compositor.h View 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/gpu/render_widget_compositor.cc View 1 2 chunks +2 lines, -2 lines 0 comments Download
M ui/compositor/compositor.h View 1 chunk +1 line, -1 line 0 comments Download
M ui/compositor/compositor.cc View 1 chunk +1 line, -1 line 0 comments Download
M webkit/compositor_bindings/web_layer_tree_view_impl_for_testing.h View 1 chunk +1 line, -1 line 0 comments Download
M webkit/compositor_bindings/web_layer_tree_view_impl_for_testing.cc View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 7 (0 generated)
boliu
Dana: Could I get a stamp from you. TBRing other owners after.
7 years, 7 months ago (2013-05-08 20:30:18 UTC) #1
danakj
LGTM
7 years, 7 months ago (2013-05-08 20:33:45 UTC) #2
boliu
TBRing owners for simple method rename sievers - content/browser/renderer_host/compositor_impl_android jamesr - content/renderer/gpu/render_widget_compositor, webkit/compositor_bindings piman - ...
7 years, 7 months ago (2013-05-08 20:43:38 UTC) #3
piman
lgtm
7 years, 7 months ago (2013-05-08 20:44:01 UTC) #4
jamesr
lgtm
7 years, 7 months ago (2013-05-08 20:44:59 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/[email protected]/14951002/5001
7 years, 7 months ago (2013-05-08 20:52:01 UTC) #6
commit-bot: I haz the power
7 years, 7 months ago (2013-05-08 21:51:31 UTC) #7
Message was sent while issue was closed. 
Change committed as 199023

Powered by Google App Engine
This is Rietveld 408576698