aw: Add new DrawGL modes

android_webview/browser/hardware_renderer.cc

 // Copyright 2014 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 "android_webview/browser/hardware_renderer.h"
 
 #include "android_webview/browser/aw_gl_surface.h"
 #include "android_webview/browser/browser_view_renderer_client.h"
 #include "android_webview/browser/gl_view_renderer_manager.h"
 #include "android_webview/browser/scoped_app_gl_state_restore.h"
@@ skipping 72 matching lines @@
     return false;
   }
 
   // TODO(boliu): Handle context loss.
   if (last_egl_context_ != current_context)
     DLOG(WARNING) << "EGLContextChanged";
 
   ScopedAppGLStateRestore state_restore(ScopedAppGLStateRestore::MODE_DRAW);
   internal::ScopedAllowGL allow_gl;
 
-  if (draw_info->mode == AwDrawGLInfo::kModeProcess)
+  if (draw_info->mode != AwDrawGLInfo::kModeDraw)

[hush (inactive), 2014/04/29 17:21:44]

is it true that we only access the shared renderer state in kModeDraw and no
other modes? (kModeProcess, kModeProcessNoContext, kModeSync)
put a comment here?

[boliu, 2014/04/29 17:33:53]

Done.

     return false;
 
   // Update memory budget. This will no-op in compositor if the policy has not
   // changed since last draw.
   content::SynchronousCompositorMemoryPolicy policy;
   policy.bytes_limit = g_memory_multiplier * kBytesPerPixel *
                        input.global_visible_rect.width() *
                        input.global_visible_rect.height();
   // Round up to a multiple of kMemoryAllocationStep.
   policy.bytes_limit =
@@ skipping 190 matching lines @@
   base::RefCountedThreadSafe<DeferredGpuCommandService>::AddRef();
 }
 
 void DeferredGpuCommandService::Release() const {
   base::RefCountedThreadSafe<DeferredGpuCommandService>::Release();
 }
 
 }  // namespace internal
 
 }  // namespace android_webview