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

Issue 17110007: Delete usage and support for EGL_ANGLE_software_display extension. (Closed)

Created:
7 years, 6 months ago by apatrick_chromium
Modified:
7 years, 6 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, jam, darin-cc_chromium.org, Myles C. Maxfield (Amazon.com), marcheu, Stephen White
Visibility:
Public.

Description

Delete usage and support for EGL_ANGLE_software_display extension. We aren't using it anymore. It used to be used to switch to SwitchShader but the new version of SwiftShader exposes a EGL / GLES2 interface without ANGLE in between. TBR=kbr@chromium.org, senorblanco@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=206759

Patch Set 1 : #

Patch Set 2 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+37 lines, -123 lines) Patch
M content/common/gpu/gpu_channel_manager.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/common/gpu/image_transport_surface_android.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/common/gpu/image_transport_surface_linux.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/common/gpu/image_transport_surface_win.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/gpu/gpu_main.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/service/async_pixel_transfer_manager_egl.cc View 1 1 chunk +1 line, -2 lines 0 comments Download
M gpu/command_buffer/service/async_pixel_transfer_manager_share_group.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/tests/gl_manager.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M gpu/command_buffer/tests/gl_unittests_android.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/config/gpu_info_collector.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/demos/framework/window.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/gles2_conform_support/egl/display.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/tools/shader_bench/shader_bench.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/gl/gl_surface.h View 1 chunk +0 lines, -2 lines 0 comments Download
M ui/gl/gl_surface_egl.h View 4 chunks +2 lines, -5 lines 0 comments Download
M ui/gl/gl_surface_egl.cc View 1 10 chunks +10 lines, -64 lines 0 comments Download
M ui/gl/gl_surface_mac.cc View 1 2 chunks +1 line, -9 lines 0 comments Download
M ui/gl/gl_surface_win.cc View 4 chunks +2 lines, -9 lines 0 comments Download
M ui/gl/gl_surface_x11.cc View 4 chunks +3 lines, -13 lines 0 comments Download
M ui/surface/accelerated_surface_mac.cc View 1 chunk +1 line, -2 lines 0 comments Download
M webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
apatrick_chromium
FYI
7 years, 6 months ago (2013-06-15 00:32:34 UTC) #1
Ken Russell (switch to Gerrit)
Awesome, thanks. LGTM mymax@, marcheu@, senorblanco@: this is needed for the Mesa 9.0.3 roll.
7 years, 6 months ago (2013-06-15 00:49:13 UTC) #2
Stephen White
LGTM, FWIW
7 years, 6 months ago (2013-06-15 01:35:24 UTC) #3
apatrick_chromium
7 years, 6 months ago (2013-06-17 18:28:12 UTC) #4
Message was sent while issue was closed.
Committed patchset #2 manually as r206759 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698