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

Unified Diff: content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClient.java

Issue 11788005: Switch Android code to use ExecuteJavascriptInWebFrameCallbackResult. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: new approach with Messages Created 7 years, 12 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/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClient.java
diff --git a/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClient.java b/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClient.java
index 64322b062f2d11ec83b9011a46eb9a824a60c5f1..b1ad650ed34c8a1cfc630826341adf3456709a6d 100644
--- a/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClient.java
+++ b/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClient.java
@@ -41,9 +41,9 @@ public class TestContentViewClient extends ContentViewClient {
* stop working!
*/
@Override
joth 2013/01/05 00:54:42 remove override. we should remove this method alto
Avi (use Gerrit) 2013/01/05 01:03:34 This was intended as a snapshot of progress. This
- public void onEvaluateJavaScriptResult(int id, String jsonResult) {
- super.onEvaluateJavaScriptResult(id, jsonResult);
- mOnEvaluateJavaScriptResultHelper.notifyCalled(id, jsonResult);
+ public void onEvaluateJavaScriptResult(String jsonResult) {
+ super.onEvaluateJavaScriptResult(jsonResult);
+ mOnEvaluateJavaScriptResultHelper.notifyCalled(jsonResult);
}
@Override

Powered by Google App Engine
This is Rietveld 408576698