Wrap GCed raw pointer parameters of WTF::bind with Persistent

third_party/WebKit/Source/bindings/core/v8/RejectedPromises.cpp

Index: third_party/WebKit/Source/bindings/core/v8/RejectedPromises.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/RejectedPromises.cpp b/third_party/WebKit/Source/bindings/core/v8/RejectedPromises.cpp
index 2661227470ac0759f7b83803a1774adbb0116511..7a55bfa490e6b32f603fa8ce224524a97a4ac91f 100644
--- a/third_party/WebKit/Source/bindings/core/v8/RejectedPromises.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/RejectedPromises.cpp
@@ -210,7 +210,7 @@ void RejectedPromises::handlerAdded(v8::PromiseRejectMessage data)
         OwnPtr<Message>& message = m_reportedAsErrors.at(i);
         if (!message->isCollected() && message->hasPromise(data.GetPromise())) {
             message->makePromiseStrong();
-            Platform::current()->currentThread()->scheduler()->timerTaskRunner()->postTask(BLINK_FROM_HERE, bind(&RejectedPromises::revokeNow, this, passed(std::move(message))));
+            Platform::current()->currentThread()->scheduler()->timerTaskRunner()->postTask(BLINK_FROM_HERE, bind(&RejectedPromises::revokeNow, RefPtr<RejectedPromises>(this), passed(std::move(message))));

[hiroshige, 2016/06/15 12:12:39]

I think it is better to split this change into a separate CL, because this
changes the behavior while other changes don't change the behavior and quite
mechanical.
(this itself looks good according to yhirano@'s comment)

[tzik, 2016/06/21 09:09:04]

Done. I split this part into https://codereview.chromium.org/2083953002/ 

             m_reportedAsErrors.remove(i);
             return;
         }