Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(251)

Unified Diff: content/browser/android/java/gin_java_bridge_dispatcher_host.cc

Issue 2811253004: Remove ListValue::Append(raw ptr) on Android (Closed)
Patch Set: Comments Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/browser/android/java/gin_java_bridge_dispatcher_host.cc
diff --git a/content/browser/android/java/gin_java_bridge_dispatcher_host.cc b/content/browser/android/java/gin_java_bridge_dispatcher_host.cc
index 4b1fbeba4738f92d13dad852ab15429188415579..ed067a504f86b70b65610b86ee6431641cfaaca8 100644
--- a/content/browser/android/java/gin_java_bridge_dispatcher_host.cc
+++ b/content/browser/android/java/gin_java_bridge_dispatcher_host.cc
@@ -357,8 +357,7 @@ void GinJavaBridgeDispatcherHost::OnInvokeMethod(
routing_id);
}
wrapped_result->Append(
- GinJavaBridgeValue::CreateObjectIDValue(
- returned_object_id).release());
+ GinJavaBridgeValue::CreateObjectIDValue(returned_object_id));
} else {
wrapped_result->Append(base::MakeUnique<base::Value>());
}

Powered by Google App Engine
This is Rietveld 408576698