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

Issue 10383297: Revert 138415 - Makes the browser send pixels to the GPU process where it should. (Closed)

Created:
8 years, 7 months ago by pkotwicz
Modified:
8 years, 7 months ago
Reviewers:
pkotwicz
CC:
chromium-reviews, yusukes+watch_chromium.org, jochen+watch-content_chromium.org, jonathan.backer, Ian Vollick, dhollowa+watch_chromium.org, jam, penghuang+watch_chromium.org, joi+watch-content_chromium.org, darin-cc_chromium.org, piman+watch_chromium.org, James Su
Visibility:
Public.

Description

Revert 138415 - Makes the browser send pixels to the GPU process where it should. This patch fixes the places where we will be sending DIP to the GPU process but should be sending pixels instead to the GPU process once https://bugs.webkit.org/show_bug.cgi?id=86051 lands. This patch reverts: http://codereview.chromium.org/127553/ http://codereview.chromium.org/10332077/ Bug=127455, 114677 Test=Manual SchedulePaintInRect is called with parameters in DIP TBR=piman@chromium.org,sky@chromium.org,jamesr@chromium.org Review URL: https://chromiumcodereview.appspot.com/10411086 TBR=pkotwicz@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=138419

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -185 lines) Patch
D content/browser/renderer_host/dip_util.h View 1 chunk +0 lines, -40 lines 0 comments Download
D content/browser/renderer_host/dip_util.cc View 1 chunk +0 lines, -59 lines 0 comments Download
M content/browser/renderer_host/render_widget_host_view_aura.h View 1 chunk +4 lines, -4 lines 0 comments Download
M content/browser/renderer_host/render_widget_host_view_aura.cc View 12 chunks +27 lines, -44 lines 0 comments Download
M content/browser/web_contents/web_contents_impl.cc View 1 chunk +0 lines, -6 lines 0 comments Download
M content/common/view_messages.h View 1 chunk +0 lines, -1 line 0 comments Download
M content/content_browser.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M content/port/browser/render_widget_host_view_port.h View 2 chunks +4 lines, -4 lines 0 comments Download
M ui/compositor/layer.h View 2 chunks +7 lines, -9 lines 0 comments Download
M ui/compositor/layer.cc View 5 chunks +14 lines, -12 lines 0 comments Download
M webkit/glue/webpreferences.h View 1 chunk +0 lines, -1 line 0 comments Download
M webkit/glue/webpreferences.cc View 2 chunks +0 lines, -3 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
pkotwicz
8 years, 7 months ago (2012-05-23 02:16:57 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698