cc: Adding BeginFrameTracker object and removing Now() from LTHI.

cc/trees/layer_tree_host_unittest_context.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 "cc/trees/layer_tree_host.h"
 
 #include "base/basictypes.h"
 #include "cc/layers/content_layer.h"
 #include "cc/layers/delegated_frame_provider.h"
 #include "cc/layers/delegated_frame_resource_collection.h"
@@ skipping 375 matching lines @@
   void InitializeSettings(LayerTreeSettings* settings) override {
     settings->single_thread_proxy_scheduler = false;
   }
 
   void RequestNewOutputSurface() override {
     EXPECT_GE(1, ++request_count_);
     EndTest();
   }
 
   void BeginTest() override {
-    layer_tree_host()->Composite(base::TimeTicks());
-    layer_tree_host()->Composite(base::TimeTicks());
+    layer_tree_host()->Composite(base::TimeTicks::FromInternalValue(1));
+    layer_tree_host()->Composite(base::TimeTicks::FromInternalValue(2));

[brianderson, 2014/12/18 02:01:38]

Are real timestamps needed to avoid hitting new DCHECKs?

[mithro-old, 2014/12/18 17:22:41]

Yeah, base::TimeTicks() is the "null equivalent" value. It should *never* be
seen by this method and should really already be DCHECKing on it.

   }
 
   scoped_ptr<OutputSurface> CreateOutputSurface() override {
     EXPECT_TRUE(false);
     return nullptr;
   }
 
   void DidInitializeOutputSurface() override { EXPECT_TRUE(false); }
 
   void AfterTest() override {}
@@ skipping 15 matching lines @@
 
   void RequestNewOutputSurface() override {
     if (request_count_ == 0) {
       ExpectCreateToFail();
       layer_tree_host()->SetOutputSurface(
           make_scoped_ptr(new FailureOutputSurface(false)));
     }
   }
 
   void BeginTest() override {
-    layer_tree_host()->Composite(base::TimeTicks());
-    layer_tree_host()->Composite(base::TimeTicks());
+    layer_tree_host()->Composite(base::TimeTicks::FromInternalValue(1));
+    layer_tree_host()->Composite(base::TimeTicks::FromInternalValue(2));
   }
 
   void DidInitializeOutputSurface() override { EXPECT_TRUE(false); }
 
   void DidFailToInitializeOutputSurface() override {
     LayerTreeHostContextTest::DidFailToInitializeOutputSurface();
     EXPECT_GE(2, ++request_count_);
     EndTest();
   }
 
@@ skipping 21 matching lines @@
                        CreateAndSetOutputSurface,
                    base::Unretained(this)));
   }
 
   void CreateAndSetOutputSurface() {
     creating_output_ = true;
     layer_tree_host()->SetOutputSurface(
         LayerTreeHostContextTest::CreateOutputSurface());
   }
 
-  void BeginTest() override { layer_tree_host()->Composite(base::TimeTicks()); }
+  void BeginTest() override {
+    layer_tree_host()->Composite(base::TimeTicks::FromInternalValue(1));
+  }
 
   void ScheduleComposite() override {
     if (creating_output_)
       EndTest();
   }
 
   void AfterTest() override {}
 
   bool creating_output_;
 };
@@ skipping 12 matching lines @@
   }
 
   void RequestNewOutputSurface() override {
     layer_tree_host()->SetOutputSurface(
         LayerTreeHostContextTest::CreateOutputSurface());
     EndTest();
   }
 
   void BeginTest() override {
     in_composite_ = true;
-    layer_tree_host()->Composite(base::TimeTicks());
+    layer_tree_host()->Composite(base::TimeTicks::FromInternalValue(1));
     in_composite_ = false;
   }
 
   void ScheduleComposite() override { EXPECT_FALSE(in_composite_); }
 
   void AfterTest() override {}
 
   bool in_composite_;
 };
 
@@ skipping 1280 matching lines @@
   void AfterTest() override {}
 
   bool deferred_;
 };
 
 SINGLE_AND_MULTI_THREAD_TEST_F(
     LayerTreeHostContextTestLoseAfterSendingBeginMainFrame);
 
 }  // namespace
 }  // namespace cc