Splitting DidSwap in cc::SwapPromise into WillSwap and DidSwap

content/renderer/gpu/queue_message_swap_promise_unittest.cc

Index: content/renderer/gpu/queue_message_swap_promise_unittest.cc
diff --git a/content/renderer/gpu/queue_message_swap_promise_unittest.cc b/content/renderer/gpu/queue_message_swap_promise_unittest.cc
index cc8777b6adbc3427f11989a43ae2a4701182f07b..961c14b92ffb86a703d5768a88f60907066488d1 100644
--- a/content/renderer/gpu/queue_message_swap_promise_unittest.cc
+++ b/content/renderer/gpu/queue_message_swap_promise_unittest.cc
@@ -120,7 +120,7 @@ class QueueMessageSwapPromiseTest : public testing::Test {
          ++i) {
       if (*i) {
         (*i)->DidActivate();
-        (*i)->DidSwap(NULL);
+        (*i)->WillSwap(NULL);

[vmpstr, 2016/12/15 21:54:32]

Does this need a DidSwap as well?

[Saman Sami, 2016/12/15 22:11:58]

Done.

       }
     }
   }
@@ -150,11 +150,11 @@ TEST_F(QueueMessageSwapPromiseTest, NextSwapPolicySchedulesMessageForNextSwap) {
 
   ASSERT_TRUE(promises_[0]);
   promises_[0]->DidActivate();
-  promises_[0]->DidSwap(NULL);
+  promises_[0]->WillSwap(NULL);

[vmpstr, 2016/12/15 21:54:31]

DidSwap as well?

[Saman Sami, 2016/12/15 22:11:58]

Done.

 
   EXPECT_TRUE(DirectSendMessages().empty());
   EXPECT_FALSE(frame_swap_message_queue_->Empty());
-  // frame_swap_message_queue_->DidSwap(1);
+  // frame_swap_message_queue_->WillSwap(1);
   EXPECT_TRUE(NextSwapHasMessage(messages_[0]));
 }
 
@@ -254,7 +254,7 @@ TEST_F(QueueMessageSwapPromiseTest, VisualStateSwapPromiseDidActivate) {
   QueueMessages(data, arraysize(data));
 
   promises_[0]->DidActivate();
-  promises_[0]->DidSwap(NULL);
+  promises_[0]->WillSwap(NULL);

[vmpstr, 2016/12/15 21:54:31]

DidSwap as well?

[Saman Sami, 2016/12/15 22:11:58]

Done.

   ASSERT_FALSE(promises_[1]);
   std::vector<std::unique_ptr<IPC::Message>> messages;
   messages.swap(NextSwapMessages());