Global GPU memory manager for android webview

android_webview/browser/global_tile_manager_client.h

Index: android_webview/browser/global_tile_manager_client.h
diff --git a/android_webview/browser/global_tile_manager_client.h b/android_webview/browser/global_tile_manager_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..5dbeb4cf4c9664aab7d47d27fee4aaf1efbebadb
--- /dev/null
+++ b/android_webview/browser/global_tile_manager_client.h
@@ -0,0 +1,28 @@
+// 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.
+
+#ifndef ANDROID_WEBVIEW_BROWSER_GLOBAL_TILE_MANAGER_CLIENT_H_
+#define ANDROID_WEBVIEW_BROWSER_GLOBAL_TILE_MANAGER_CLIENT_H_
+
+namespace android_webview {
+// GlobalTileManagerClient requests tile resources from GlobalTileManager.
+class GlobalTileManagerClient {
+ public:
+  // Calculate how many tiles are needed.
+  virtual size_t CalculateTileRequest() const = 0;

[boliu, 2014/05/01 00:37:40]

new line after each method declaration

[hush (inactive), 2014/05/01 19:03:04]

Done.

+  // Force drop the hardware tiles in gpu.
+  virtual size_t ForceDropTiles() = 0;
+  // True if the client wants the manager to evict other clients, when the
+  // manager cannot satisfy this client's tile request.
+  virtual bool NeedsEviction() const = 0;
+  // Get the number of tiles allocated to the client.
+  virtual size_t GetTilesNum() const = 0;

[boliu, 2014/05/01 00:37:40]

GetNumTiles?

Also add comment saying it's only called for debugging purpose

[hush (inactive), 2014/05/01 19:03:04]

Changed the function name.
Actually I use it more places than debugging.
In GlobalTileManager::RequestTiles, we actually don't need to pass in
"old_num_of_tiles", because we can get it from "key".
In GlobalTileManager::Remove, I will get the number of tiles of the removed
client and update the total_allocated_tiles_ (This change will appear in the
next patch set)

It also hides/encapsulates the fact that we not directly storing the number of
tiles in the client, but in a "shared renderer state"
On 2014/05/01 00:37:40, boliu wrote:

+
+ protected:
+  virtual ~GlobalTileManagerClient() {}
+};
+
+}  // namespace android_webview
+
+#endif  // ANDROID_WEBVIEW_BROWSER_GLOBAL_TILE_MANAGER_CLIENT_H_