cc: Use BeginFrameArgs

cc/layers/texture_layer_unittest.cc

Index: cc/layers/texture_layer_unittest.cc
diff --git a/cc/layers/texture_layer_unittest.cc b/cc/layers/texture_layer_unittest.cc
index aa08e9f81ee9a854af56afa1ae2527637dd25656..d66a3d49de576dd00eaec3d77a17578dd28b53f0 100644
--- a/cc/layers/texture_layer_unittest.cc
+++ b/cc/layers/texture_layer_unittest.cc
@@ -840,7 +840,7 @@ class TextureLayerClientTest
     PostSetNeedsCommitToMainThread();
   }
 
-  virtual void DidCommitAndDrawFrame() OVERRIDE {
+  virtual void DidCommit() OVERRIDE {

[enne (OOO), 2013/08/02 19:41:25]

This test seems like it depends on PrepareToDrawOnThread resetting used
textures, and changing this function to DidCommit might make it flaky.

[brianderson, 2013/08/02 23:04:31]

I changed a number of DidCommitAndDrawFrame's to DidCommit's because delaying
the first draw after OutputSurfaceInitialization till after activation
introduced race conditions in existing tests that depended on the
source_frame_number on the Main thread. I got a little too trigger happy here.

     ++commit_count_;
     switch (commit_count_) {
       case 1:
@@ -963,7 +963,7 @@ class TextureLayerLostContextTest
     return true;
   }
 
-  virtual void DidCommitAndDrawFrame() OVERRIDE {
+  virtual void DidCommit() OVERRIDE {

[enne (OOO), 2013/08/02 19:41:25]

There's test logic in PrepareToDrawOnThread that seems like it could be skipped
if you make this change?

[brianderson, 2013/08/02 23:04:31]

Trigger happy again. Fixed.

     EndTest();
   }
 
@@ -1022,7 +1022,7 @@ class TextureLayerWithMailboxMainThreadDeleted : public LayerTreeTest {
     PostSetNeedsCommitToMainThread();
   }
 
-  virtual void DidCommitAndDrawFrame() OVERRIDE {
+  virtual void DidCommit() OVERRIDE {
     switch (layer_tree_host()->source_frame_number()) {
       case 1:
         // Delete the TextureLayer on the main thread while the mailbox is in
@@ -1090,7 +1090,7 @@ class TextureLayerWithMailboxImplThreadDeleted : public LayerTreeTest {
     PostSetNeedsCommitToMainThread();
   }
 
-  virtual void DidCommitAndDrawFrame() OVERRIDE {
+  virtual void DidCommit() OVERRIDE {
     switch (layer_tree_host()->source_frame_number()) {
       case 1:
         // Remove the TextureLayer on the main thread while the mailbox is in