Bug 975246 - Part3 - Using nsRefPtr->* operator. r=nfroyd
authorJames Cheng <jacheng@mozilla.com>
Tue, 28 Jul 2015 23:08:00 +0200
changeset 287095 4075070296955acf65f29edf2097c56e782db305
parent 287094 c016da5a123d5f7246901196a085f079d96ac06e
child 287096 4b328a6f7448002a7f0e4980a0bbae56a0f0b019
push id5067
push userraliiev@mozilla.com
push dateMon, 21 Sep 2015 14:04:52 +0000
treeherdermozilla-beta@14221ffe5b2f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersnfroyd
bugs975246
milestone42.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 975246 - Part3 - Using nsRefPtr->* operator. r=nfroyd
dom/promise/Promise.cpp
--- a/dom/promise/Promise.cpp
+++ b/dom/promise/Promise.cpp
@@ -1507,19 +1507,17 @@ public:
 
     // Here we convert the buffer to a JS::Value.
     JS::Rooted<JS::Value> value(aCx);
     if (!mBuffer.read(aCx, &value, mCallbacks, mPromiseWorkerProxy)) {
       JS_ClearPendingException(aCx);
       return false;
     }
 
-    // TODO Bug 975246 - nsRefPtr should support operator |nsRefPtr->*funcType|.
-    (workerPromise.get()->*mFunc)(aCx,
-                                  value);
+    (workerPromise->*mFunc)(aCx, value);
 
     // Release the Promise because it has been resolved/rejected for sure.
     mPromiseWorkerProxy->CleanUp(aCx);
     return true;
   }
 
 protected:
   ~PromiseWorkerProxyRunnable() {}