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

Issue 6712050: Revert 78801 - Addition of GL-redirecting mechanism for the Skia/GPU back-end. During execution o... (Closed)

Created:
9 years, 9 months ago by bradn
Modified:
9 years, 7 months ago
Reviewers:
twiz, Jeff Timanus
CC:
chromium-reviews, darin-cc_chromium.org, apatrick_chromium
Visibility:
Public.

Description

Revert 78801 - Addition of GL-redirecting mechanism for the Skia/GPU back-end. During execution of the layout tests (DumpRenderTree), all GL-calls are invoked via Mesa. A GL re-direction mechanism was added to skia in revision (http://code.google.com/p/skia/source/detail?r=937) These changes populate the skia re-direction interface with the gl implementation currently in use in Chrome: The app\gfx\gl bindings. R=kbr, bsalomon Review URL: http://codereview.chromium.org/6677105 TBR=twiz@chromium.org R=* Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=78808

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -555 lines) Patch
M DEPS View 1 chunk +1 line, -1 line 0 comments Download
M app/app_base.gypi View 1 chunk +0 lines, -2 lines 0 comments Download
M app/gfx/gl/generate_bindings.py View 3 chunks +1 line, -3 lines 0 comments Download
D app/gfx/gl/gl_bindings_skia.h View 1 chunk +0 lines, -21 lines 0 comments Download
D app/gfx/gl/gl_bindings_skia.cc View 1 chunk +0 lines, -517 lines 0 comments Download
M app/gfx/gl/gl_interface.h View 2 chunks +0 lines, -4 lines 0 comments Download
M gpu/command_buffer/common/gl_mock.h View 2 chunks +0 lines, -4 lines 0 comments Download
M skia/skia.gyp View 1 chunk +0 lines, -1 line 0 comments Download
M webkit/support/test_webkit_client.cc View 2 chunks +0 lines, -2 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
bradn
9 years, 9 months ago (2011-03-19 03:53:04 UTC) #1
twiz
9 years, 9 months ago (2011-03-21 20:57:03 UTC) #2
LGTM.

Thanks for taking care of this, Bradley.

Powered by Google App Engine
This is Rietveld 408576698