Bug 1059813: Use getter/setter methods of |UnixSocketRawData| in RIL helpers, r=vyang
☠☠ backed out by 927da4815ef9 ☠ ☠
authorThomas Zimmermann <tdz@users.sourceforge.net>
Wed, 03 Sep 2014 16:43:13 +0200
changeset 203371 ab4056a1aa06d05695a4ecfff2ee7d0356fa3cff
parent 203370 d08a91fee65a4e4c95fe92e2dbc28b5a13633d78
child 203372 f991f546dbcdc00086d3cebbbd7a06232204e088
push id48665
push userryanvm@gmail.com
push dateWed, 03 Sep 2014 20:40:15 +0000
treeherdermozilla-inbound@0da762e6868a [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvyang
bugs1059813
milestone35.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 1059813: Use getter/setter methods of |UnixSocketRawData| in RIL helpers, r=vyang
ipc/ril/Ril.cpp
--- a/ipc/ril/Ril.cpp
+++ b/ipc/ril/Ril.cpp
@@ -179,21 +179,23 @@ private:
         nsAutoPtr<UnixSocketRawData> mMessage;
 };
 
 bool
 DispatchRILEvent::RunTask(JSContext *aCx)
 {
     JS::Rooted<JSObject*> obj(aCx, JS::CurrentGlobalOrNull(aCx));
 
-    JS::Rooted<JSObject*> array(aCx, JS_NewUint8Array(aCx, mMessage->mSize));
+    JS::Rooted<JSObject*> array(aCx,
+                                JS_NewUint8Array(aCx, mMessage->GetSize()));
     if (!array) {
         return false;
     }
-    memcpy(JS_GetArrayBufferViewData(array), mMessage->mData, mMessage->mSize);
+    memcpy(JS_GetArrayBufferViewData(array),
+           mMessage->GetData(), mMessage->GetSize());
 
     JS::AutoValueArray<2> args(aCx);
     args[0].setNumber((uint32_t)mClientId);
     args[1].setObject(*array);
 
     JS::Rooted<JS::Value> rval(aCx);
     return JS_CallFunctionName(aCx, obj, "onRILMessage", args, &rval);
 }