Splitting GpuMemoryAllocation into two parts: for renderer and browser

content/common/gpu/gpu_memory_allocation.h

 // Copyright (c) 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.
 

[mmocny, 2012/03/15 19:11:43]

I've separated out the two different "Allocations", one intended for the
renderer/client (manages allocation bytes, backbuffer), and one for the
browser/host (manages frontbuffer).

The GpuMemoryManager manages and assigns a single unified allocation, and the
GpuCommandBufferStub splits it accordingly.

 #ifndef CONTENT_COMMON_GPU_GPU_MEMORY_ALLOCATION_H_
 #define CONTENT_COMMON_GPU_GPU_MEMORY_ALLOCATION_H_
 #pragma once
 
 #include "base/basictypes.h"
+#include "ui/gfx/gl/gl_surface.h"

[mmocny, 2012/03/15 19:11:43]

I dont need this, will remove..

 
-// These are memory allocation limits assigned to a context.
-// They will change over time, given memory availability, and browser state
-// Exceeding these limits for an unreasonable amount of time will cause context
-// to be lost.
-class GpuMemoryAllocation {
+// These are per context memory allocation limits set by the GpuMemoryManager
+// and assigned to the browser and renderer context.
+// They will change over time, given memory availability, and browser state.
+
+
+// Memory Allocation which will be assigned to the renderer context.
+class GpuMemoryAllocationForRenderer {

[mmocny, 2012/03/15 19:11:43]

This name will need to change to something more general, but I couldn't come up
with something sufficiently general yet appropriate.  Producer/consumer
client/host etc seem not to apply well.

Suggestions welcome.

  public:
   enum {
     INVALID_RESOURCE_SIZE = -1
   };
 
+  // Exceeding this limit for an unreasonable amount of time may cause context
+  // to be lost.
   size_t gpu_resource_size_in_bytes;
-  bool has_frontbuffer;
-  bool has_backbuffer;
+  bool suggest_have_backbuffer;
 
+  GpuMemoryAllocationForRenderer()
+      : gpu_resource_size_in_bytes(0),
+        suggest_have_backbuffer(false) {
+  }
+
+  GpuMemoryAllocationForRenderer(size_t gpu_resource_size_in_bytes,
+                                 bool suggest_have_backbuffer)
+      : gpu_resource_size_in_bytes(gpu_resource_size_in_bytes),
+        suggest_have_backbuffer(suggest_have_backbuffer) {
+  }
+
+  friend inline bool operator==(const GpuMemoryAllocationForRenderer& lhs,
+                                const GpuMemoryAllocationForRenderer& rhs) {
+    return lhs.gpu_resource_size_in_bytes == rhs.gpu_resource_size_in_bytes &&
+        lhs.suggest_have_backbuffer == rhs.suggest_have_backbuffer;
+  }
+
+  friend inline bool operator!=(const GpuMemoryAllocationForRenderer& lhs,
+                                const GpuMemoryAllocationForRenderer& rhs) {
+    return !(lhs == rhs);
+  }
+};
+
+// Memory Allocation which will be assigned to the browser.
+class GpuMemoryAllocationForBrowser {

[mmocny, 2012/03/15 19:11:43]

ditto on naming here.

+ public:
+  bool suggest_have_frontbuffer;
+
+  GpuMemoryAllocationForBrowser()
+      : suggest_have_frontbuffer(false) {
+  }
+
+  GpuMemoryAllocationForBrowser(bool suggest_have_frontbuffer)
+      : suggest_have_frontbuffer(suggest_have_frontbuffer) {
+  }
+
+  friend inline bool operator==(const GpuMemoryAllocationForBrowser& lhs,
+                                const GpuMemoryAllocationForBrowser& rhs) {
+      return lhs.suggest_have_frontbuffer == rhs.suggest_have_frontbuffer;
+  }
+
+  friend inline bool operator!=(const GpuMemoryAllocationForBrowser& lhs,
+                                const GpuMemoryAllocationForBrowser& rhs) {
+    return !(lhs == rhs);
+  }
+};
+
+// Combination of the above two Memory Allocations which will be created by the
+// GpuMemoryManager.
+class GpuMemoryAllocation : public GpuMemoryAllocationForRenderer,

[mmocny, 2012/03/15 19:11:43]

This could have been std::pair<..ForRenderer, ..ForBrowser>, but I figured this
is nicer for the GpuMemoryManager.

+                            public GpuMemoryAllocationForBrowser {
+ public:
   GpuMemoryAllocation()
-      : gpu_resource_size_in_bytes(0),
-        has_frontbuffer(false),
-        has_backbuffer(false) {
+      : GpuMemoryAllocationForRenderer(),
+        GpuMemoryAllocationForBrowser() {
   }
 
   GpuMemoryAllocation(size_t gpu_resource_size_in_bytes,
-                      bool has_frontbuffer,
-                      bool has_backbuffer)
-      : gpu_resource_size_in_bytes(gpu_resource_size_in_bytes),
-        has_frontbuffer(has_frontbuffer),
-        has_backbuffer(has_backbuffer) {
+                      bool suggest_have_backbuffer,
+                      bool suggest_have_frontbuffer)
+      : GpuMemoryAllocationForRenderer(gpu_resource_size_in_bytes,
+                                       suggest_have_backbuffer),
+        GpuMemoryAllocationForBrowser(suggest_have_frontbuffer) {
+  }
+
+  friend inline bool operator==(const GpuMemoryAllocation& lhs,
+                                const GpuMemoryAllocation& rhs) {
+      return static_cast<const GpuMemoryAllocationForRenderer&>(lhs) ==
+             static_cast<const GpuMemoryAllocationForRenderer&>(rhs) &&
+             static_cast<const GpuMemoryAllocationForBrowser&>(lhs) ==
+             static_cast<const GpuMemoryAllocationForBrowser&>(rhs);
+  }
+
+  friend inline bool operator!=(const GpuMemoryAllocation& lhs,
+                                const GpuMemoryAllocation& rhs) {
+    return !(lhs == rhs);
   }
 };
 
-inline bool operator==(const GpuMemoryAllocation& lhs,
-                       const GpuMemoryAllocation& rhs) {
-  return lhs.gpu_resource_size_in_bytes == rhs.gpu_resource_size_in_bytes &&
-      lhs.has_frontbuffer == rhs.has_frontbuffer &&
-      lhs.has_backbuffer == rhs.has_backbuffer;
-}
-
-inline bool operator!=(const GpuMemoryAllocation& lhs,
-                       const GpuMemoryAllocation& rhs) {
-  return !(lhs == rhs);
-}
-
 #endif // CONTENT_COMMON_GPU_GPU_MEMORY_ALLOCATION_H_