Global GPU memory manager for android webview

android_webview/native/aw_contents.cc

 // Copyright 2012 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/native/aw_contents.h"
 
 #include <limits>
 
 #include "android_webview/browser/aw_browser_context.h"
 #include "android_webview/browser/aw_browser_main_parts.h"
@@ skipping 1033 matching lines @@
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   render_view_host_ext_->SetJsOnlineProperty(network_up);
 }
 
 void AwContents::TrimMemoryOnRenderThread(JNIEnv* env,
                                           jobject obj,
                                           jint level,
                                           jboolean visible) {
   if (hardware_renderer_) {
     if (hardware_renderer_->TrimMemory(level, visible)) {
-      content::BrowserThread::PostTask(
-          content::BrowserThread::UI,
+      BrowserThread::PostTask(
+          BrowserThread::UI,
           FROM_HERE,
-          base::Bind(&AwContents::ForceFakeComposite, ui_thread_weak_ptr_));
+          base::Bind(&AwContents::TrimMemory, ui_thread_weak_ptr_));
     }
   }
 }
 
-void AwContents::ForceFakeComposite() {
-  browser_view_renderer_.ForceFakeCompositeSW();
+void AwContents::TrimMemory() {
+  browser_view_renderer_.TrimMemory();

[boliu, 2014/04/28 22:22:15]

This might be out of scope for this patch, but can you try this?

Move everything in HardwareRenderer::TrimMemory directly into BVR, call it
directly on UI thread, and test if it works (ie doesn't deadlock)? I think it
shouldn't deadlock but you never know...

Also another idea. What if TrimMemory goes to GlobalTileManager instead of BVR?
Is GlobalTileManager able to make better decisions about what to do?

[hush (inactive), 2014/04/30 20:50:17]

I put trim memory into BVR.
Haven't thought about putting it into GlobalTileManager yet. Right now, every
webview will lose all tiles during trim memory. If we need to trim less memory
and keep some tiles for some webviews, then we will need to do it
GlobalTileManager. But I don't think this is necessary in near future.

On 2014/04/28 22:22:15, boliu wrote:

 }
 
 void SetShouldDownloadFavicons(JNIEnv* env, jclass jclazz) {
   g_should_download_favicons = true;
 }
 
 }  // namespace android_webview