cc: Set the mapped memory reclaim limit for the renderer compositor on Android

cc/trees/layer_tree_host_unittest.cc

 // Copyright 2011 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 "cc/trees/layer_tree_host.h"
 
 #include <algorithm>
 
 #include "base/auto_reset.h"
 #include "base/synchronization/lock.h"
@@ skipping 646 matching lines @@
     // If we make it without kicking a frame, we pass!
     EndTestAfterDelay(1);
   }
 
   virtual void Layout() OVERRIDE {
     ASSERT_FALSE(true);
     EndTest();
   }
 
   virtual void AfterTest() OVERRIDE {}
-
- private:
 };
 
 MULTI_THREAD_TEST_F(LayerTreeHostTestAbortFrameWhenInvisible);
 
 // This test verifies that properties on the layer tree host are commited
 // to the impl side.
 class LayerTreeHostTestCommit : public LayerTreeHostTest {
  public:
   LayerTreeHostTestCommit() {}
 
@@ skipping 3778 matching lines @@
   }
 
  private:
   int num_will_begin_frames_;
   int num_impl_commits_;
 };
 
 // Commits can only be aborted when using the thread proxy.
 MULTI_THREAD_TEST_F(LayerTreeHostTestAbortEvictedTextures);
 
+class LayerTreeHostTestMaxTransferBufferUsageBytes : public LayerTreeHostTest {
+ protected:
+  virtual void InitializeSettings(LayerTreeSettings* settings) OVERRIDE {
+    settings->impl_side_painting = true;
+  }
+
+  virtual void SetupTree() OVERRIDE {
+    scoped_refptr<FakePictureLayer> root_layer =
+        FakePictureLayer::Create(&client_);
+    // 6000 * 6000 * 4bpp = 144MB
+    root_layer->SetBounds(gfx::Size(6000, 6000));
+    root_layer->SetIsDrawable(true);
+
+    layer_tree_host()->SetRootLayer(root_layer);
+    LayerTreeHostTest::SetupTree();
+  }
+
+  virtual void BeginTest() OVERRIDE {
+    PostSetNeedsCommitToMainThread();
+  }
+
+  virtual void DidActivateTreeOnThread(LayerTreeHostImpl* impl) OVERRIDE {
+    TestWebGraphicsContext3D* context = static_cast<TestWebGraphicsContext3D*>(
+        impl->output_surface()->context_provider()->Context3d());
+    // The default MaxTransferBufferUsageBytes value is 64MB.

[danakj, 2013/09/06 21:48:17]

Can you assert this value somewhere int he test so we know the test is testing
what it claims?

[kaanb, 2013/09/06 21:58:45]

If I add a getter for to TileManager yes, but reveman wasn't in favor of it.
That's why I'm using the transfer buffer memory allocated in the context as a
proxy for it.

[danakj, 2013/09/06 22:01:13]

What I mean is the limit that the we're passing into the tile manager. What if
we changed the limit to 32MB one day, and broke the PBRWP so we were using 2*the
limit, then this test would still pass.

How about overriding the caps limit on the Context3d (by overriding
CreateOutputSurface())? And verify we're using some value "close" to the limit,
for two different limits on the context3d?

[reveman, 2013/09/06 22:28:27]

+1

[kaanb, 2013/09/07 01:30:31]

Done.

+    // Expect that we're using less than that.
+    EXPECT_GT(64 * 1024 * 1024u, context->transfer_buffer_memory_used_bytes());
+    EndTest();
+  }
+
+  virtual void AfterTest() OVERRIDE {}
+
+ private:
+  FakeContentLayerClient client_;
+};
+
+// Impl-side painting is a multi-threaded compositor feature.
+MULTI_THREAD_TEST_F(LayerTreeHostTestMaxTransferBufferUsageBytes);
+
 }  // namespace
 
 }  // namespace cc