Changeset 178042 in webkit for trunk/Source/JavaScriptCore/replay/EncodedValue.cpp

Timestamp:

Jan 7, 2015, 11:32:05 AM (10 years ago)

Author:

[email protected]

Message:

Unreviewed, rolling out r178039.
​https://bugs.webkit.org/show_bug.cgi?id=140187

Breaks ObjC Inspector Protocol (Requested by JoePeck on
#webkit).

Reverted changeset:

"Web Inspector: purge PassRefPtr from Inspector code and use
Ref for typed and untyped protocol objects"
​https://bugs.webkit.org/show_bug.cgi?id=140053
​http://trac.webkit.org/changeset/178039

File:

• trunk/Source/JavaScriptCore/replay/EncodedValue.cpp (modified) (6 diffs)

trunk/Source/JavaScriptCore/replay/EncodedValue.cpp

@@ -38 +38 @@
 namespace JSC {
 
-RefPtr<InspectorObject> EncodedValue::asObject()
+PassRefPtr<InspectorObject> EncodedValue::asObject()
 {
     RefPtr<InspectorObject> result;
@@ -44 +44 @@
     ASSERT_UNUSED(castSucceeded, castSucceeded);
 
-    return result;
-}
-
-RefPtr<InspectorArray> EncodedValue::asArray()
+    return result.release();
+}
+
+PassRefPtr<InspectorArray> EncodedValue::asArray()
 {
     RefPtr<InspectorArray> result;
@@ -53 +53 @@
     ASSERT_UNUSED(castSucceeded, castSucceeded);
 
-    return result;
+    return result.release();
 }
 
@@ -176 +176 @@
 void EncodedValue::put<EncodedValue>(const String& key, const typename EncodingTraits<EncodedValue>::DecodedType& value)
 {
-    asObject()->setValue(key, value.m_value.copyRef());
+    asObject()->setValue(key, value.m_value);
 }
 
@@ -182 +182 @@
 void EncodedValue::append<EncodedValue>(const typename EncodingTraits<EncodedValue>::DecodedType& value)
 {
-    asArray()->pushValue(value.m_value.copyRef());
+    asArray()->pushValue(value.m_value);
 }
 
@@ -188 +188 @@
 bool EncodedValue::get<EncodedValue>(const String& key, typename EncodingTraits<EncodedValue>::DecodedType& decodedValue)
 {
-    RefPtr<Inspector::InspectorValue> value;
-    if (!asObject()->getValue(key, value))
+    RefPtr<Inspector::InspectorValue> inspectorValue(asObject()->get(key));
+    if (!inspectorValue)
         return false;
 
-    decodedValue = EncodedValue(WTF::move(value));
+    decodedValue = EncodedValue(inspectorValue);
     return true;
 }