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

Issue 8352033: Merge 106394 - Fixed bugs with Pepper 3D under dynamic GPU switching. (Closed)

Created:
9 years, 2 months ago by Ken Russell (switch to Gerrit)
Modified:
9 years, 2 months ago
CC:
chromium-reviews, joi+watch-content_chromium.org, darin-cc_chromium.org, jam, apatrick_chromium, dpranke-watch+content_chromium.org
Visibility:
Public.

Description

Merge 106394 - Fixed bugs with Pepper 3D under dynamic GPU switching. Made Pepper 3D context creation code fully aware of GPU switching and changed the order of context creation and and fetching of the parent context. Creating the context potentially causes all existing contexts for that renderer, including the parent, to be lost. Detect shutting down of the GPU channel more quickly on the renderer side. Only reject context creation in GLContextCGL upon GpuPreference mismatch if the system supports dual GPUs. Will update tools/histograms/histograms.xml as soon as this is committed; must occur in a separate CL. BUG=100507 TEST=tested navigating to and from, and reloading, NaCl Pepper 3D samples on Mac 10.6.8 and 10.7.1; WebGL tests Review URL: http://codereview.chromium.org/8342024 TBR=kbr@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=106440

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+110 lines, -45 lines) Patch
M content/common/gpu/gpu_process_launch_causes.h View 1 chunk +1 line, -0 lines 0 comments Download
M content/content_renderer.gypi View 1 chunk +2 lines, -0 lines 0 comments Download
M content/renderer/gpu/renderer_gl_context.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M content/renderer/gpu/webgraphicscontext3d_command_buffer_impl.cc View 1 chunk +1 line, -0 lines 0 comments Download
A + content/renderer/pepper_parent_context_provider.h View 0 chunks +-1 lines, --1 lines 0 comments Download
A + content/renderer/pepper_parent_context_provider.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
M content/renderer/pepper_platform_context_3d_impl.h View 2 chunks +5 lines, -1 line 0 comments Download
M content/renderer/pepper_platform_context_3d_impl.cc View 5 chunks +53 lines, -21 lines 0 comments Download
M content/renderer/pepper_plugin_delegate_impl.h View 3 chunks +6 lines, -1 line 0 comments Download
M content/renderer/pepper_plugin_delegate_impl.cc View 2 chunks +17 lines, -13 lines 0 comments Download
M content/renderer/render_widget_fullscreen_pepper.h View 3 chunks +6 lines, -1 line 0 comments Download
M content/renderer/render_widget_fullscreen_pepper.cc View 2 chunks +11 lines, -6 lines 0 comments Download
M ui/gfx/gl/gl_context_cgl.cc View 1 chunk +6 lines, -4 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
Ken Russell (switch to Gerrit)
9 years, 2 months ago (2011-10-20 02:11:36 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698