Mac: Recreate browser-side GL contexts on GPU switch

content/browser/renderer_host/compositing_iosurface_context_mac.mm

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/compositing_iosurface_context_mac.h"
 
 #include <OpenGL/gl.h>
 #include <OpenGL/OpenGL.h>
 #include <vector>
 
 #include "base/command_line.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "content/browser/renderer_host/compositing_iosurface_shader_programs_mac.h"
+#include "content/public/browser/gpu_data_manager.h"

[Zhenyao Mo, 2014/04/09 22:24:24]

Usually inside content side, we directly use gpu_data_manager_impl.h instead.

[ccameron, 2014/04/09 22:37:28]

Done.

 #include "ui/base/ui_base_switches.h"
 #include "ui/gl/gl_switches.h"
 #include "ui/gl/gpu_switching_manager.h"
 
 namespace content {
 
 CoreAnimationStatus GetCoreAnimationStatus() {
   static CoreAnimationStatus status =
       CommandLine::ForCurrentProcess()->HasSwitch(
           switches::kDisableCoreAnimation) ?
@@ skipping 150 matching lines @@
     scoped_ptr<CompositingIOSurfaceShaderPrograms> shader_program_cache)
     : window_number_(window_number),
       nsgl_context_(nsgl_context),
       cgl_context_strong_(cgl_context_strong),
       cgl_context_(cgl_context),
       is_vsync_disabled_(is_vsync_disabled),
       shader_program_cache_(shader_program_cache.Pass()),
       poisoned_(false) {
   DCHECK(window_map()->find(window_number_) == window_map()->end());
   window_map()->insert(std::make_pair(window_number_, this));
+
+  content::GpuDataManager::GetInstance()->AddObserver(this);

[Zhenyao Mo, 2014/04/09 22:24:24]

nit:no need for content namespace, and use GpuDataManagerImpl instead.

same below.

[ccameron, 2014/04/09 22:37:28]

Done.

 }
 
 CompositingIOSurfaceContext::~CompositingIOSurfaceContext() {
+  content::GpuDataManager::GetInstance()->RemoveObserver(this);
+
   {
     gfx::ScopedCGLSetCurrentContext scoped_set_current_context(cgl_context_);
     shader_program_cache_->Reset();
   }
   if (!poisoned_) {
     DCHECK(window_map()->find(window_number_) != window_map()->end());
     DCHECK(window_map()->find(window_number_)->second == this);
     window_map()->erase(window_number_);
   } else {
     WindowMap::const_iterator found = window_map()->find(window_number_);
     if (found != window_map()->end())
       DCHECK(found->second != this);
   }
 }
 
 NSOpenGLContext* CompositingIOSurfaceContext::nsgl_context() const {
   // This should not be called from any CoreAnimation paths.
   CHECK(GetCoreAnimationStatus() == CORE_ANIMATION_DISABLED);
   return nsgl_context_;
 }
 
+void CompositingIOSurfaceContext::OnGpuSwitching() {
+  // Recreate all browser-side GL contexts whenever the GPU switches. If this
+  // is not done, performance will suffer.
+  // http://crbug.com/361493
+  PoisonContextAndSharegroup();
+}
+
 // static
 CompositingIOSurfaceContext::WindowMap*
     CompositingIOSurfaceContext::window_map() {
   return window_map_.Pointer();
 }
 
 // static
 base::LazyInstance<CompositingIOSurfaceContext::WindowMap>
     CompositingIOSurfaceContext::window_map_;
 
 }  // namespace content