cc: Trigger deadline immediately when commit is aborted.

cc/scheduler/scheduler_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/scheduler/scheduler.h"
 
 #include <string>
 #include <vector>
 
 #include "base/debug/trace_event.h"
@@ skipping 2182 matching lines @@
   // Disable auto-advancing of now_src
   client.task_runner().SetAutoAdvanceNowToPendingTasks(false);
 
   // Deadline task is pending
   EXPECT_TRUE(scheduler->BeginImplFrameDeadlinePending());
   client.task_runner().RunPendingTasks();
   // Deadline task runs immediately
   EXPECT_FALSE(scheduler->BeginImplFrameDeadlinePending());
 }
 
+TEST(SchedulerTest, BeginMainFrameAbortedTriggersImmediateDeadline) {

[brianderson, 2015/01/09 23:54:22]

Does this test fail before this patch?

[sunnyps, 2015/01/13 19:36:01]

Yes, it does.

+  FakeSchedulerClient client;
+  SchedulerSettings settings;
+  settings.use_external_begin_frame_source = true;
+
+  CREATE_SCHEDULER_AND_INIT_SURFACE(settings);
+
+  scheduler->SetNeedsCommit();
+  EXPECT_SINGLE_ACTION("SetNeedsBeginFrames(true)", client);
+  client.Reset();
+
+  client.AdvanceFrame();
+  EXPECT_ACTION("WillBeginImplFrame", client, 0, 2);
+  EXPECT_ACTION("ScheduledActionSendBeginMainFrame", client, 1, 2);
+  EXPECT_TRUE(scheduler->BeginImplFrameDeadlinePending());
+  client.Reset();
+
+  client.task_runner().RunUntilTime(client.now_src()->Now());
+  EXPECT_TRUE(scheduler->BeginImplFrameDeadlinePending());
+
+  scheduler->BeginMainFrameAborted(CommitEarlyOutReason::ABORTED_NOT_VISIBLE);
+
+  EXPECT_TRUE(scheduler->BeginImplFrameDeadlinePending());
+  client.task_runner().RunUntilTime(client.now_src()->Now());
+  EXPECT_FALSE(scheduler->BeginImplFrameDeadlinePending());
+
+  EXPECT_NO_ACTION(client);
+}
+
 }  // namespace
 }  // namespace cc