Revert 175489

Revert 175489

[javac] /b/build/slave/Android_Builder__dbg_/build/src/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClientWrapper.java:62: cannot find symbol
[javac] symbol  : method onEvaluateJavaScriptResult(int,java.lang.String)
[javac] location: class org.chromium.content.browser.test.util.TestContentViewClient
[javac]         super.onEvaluateJavaScriptResult(id, jsonResult);
[javac]              ^
[javac] /b/build/slave/Android_Builder__dbg_/build/src/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClientWrapper.java:63: cannot find symbol
[javac] symbol  : method onEvaluateJavaScriptResult(int,java.lang.String)
[javac] location: class org.chromium.content.browser.ContentViewClient
[javac]         mWrappedClient.onEvaluateJavaScriptResult(id, jsonResult);
[javac]                       ^
[javac] /b/build/slave/Android_Builder__dbg_/build/src/content/public/test/android/javatests/src/org/chromium/content/browser/test/util/TestContentViewClientWrapper.java:60: method does not override or implement a method from a supertype
[javac]     @Override
[javac]     ^
[javac] 3 errors

> Switch Android code to use ExecuteJavascriptInWebFrameCallbackResult.
> 
> BUG=168169
> TEST=should all pass
> 
> 
> Review URL: https://chromiumcodereview.appspot.com/11788005

TBR=avi@chromium.org
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=175491

[Avi (use Gerrit), 2013-01-08 15:41:00]

Sigh. Yay trybots.


On Tue, Jan 8, 2013 at 1:19 AM, <dbeam@chromium.org> wrote:

> Reviewers: Avi,
>
> Description:
> Revert 175489
>
> [javac]
> /b/build/slave/Android_**Builder__dbg_/build/src/**
> content/public/test/android/**javatests/src/org/chromium/**
> content/browser/test/util/**TestContentViewClientWrapper.**java:62:
> cannot find symbol
> [javac] symbol  : method onEvaluateJavaScriptResult(**
> int,java.lang.String)
> [javac] location: class
> org.chromium.content.browser.**test.util.**TestContentViewClient
> [javac]         super.**onEvaluateJavaScriptResult(id, jsonResult);
> [javac]              ^
> [javac]
> /b/build/slave/Android_**Builder__dbg_/build/src/**
> content/public/test/android/**javatests/src/org/chromium/**
> content/browser/test/util/**TestContentViewClientWrapper.**java:63:
> cannot find symbol
> [javac] symbol  : method onEvaluateJavaScriptResult(**
> int,java.lang.String)
> [javac] location: class org.chromium.content.browser.**ContentViewClient
> [javac]         mWrappedClient.**onEvaluateJavaScriptResult(id,
> jsonResult);
> [javac]                       ^
> [javac]
> /b/build/slave/Android_**Builder__dbg_/build/src/**
> content/public/test/android/**javatests/src/org/chromium/**
> content/browser/test/util/**TestContentViewClientWrapper.**java:60:
> method does not override or implement a method from a supertype
> [javac]     @Override
> [javac]     ^
> [javac] 3 errors
>
>  Switch Android code to use ExecuteJavascriptInWebFrameCal**lbackResult.
>>
>
>  BUG=168169
>> TEST=should all pass
>>
>
>
>  Review URL:
https://chromiumcodereview.**appspot.com/11788005<https://chromiumcodereview....
>>
>
> TBR=avi@chromium.org
>
> Please review this at
https://codereview.chromium.**org/11778034/<https://codereview.chromium.org/1...
>
> SVN Base:
svn://svn.chromium.org/chrome/**trunk/src/<http://svn.chromium.org/chrome/trunk/src/>
>
> Affected files:
>   M     android_webview/javatests/src/**org/chromium/android_webview/**
> test/TestAwContentsClient.java
>   M     android_webview/javatests/src/**org/chromium/android_webview/**
> test/util/JSUtils.java
>   MM    build/android/findbugs_filter/**findbugs_exclude.xml
>   M     content/browser/android/**content_view_core_impl.h
>   M     content/browser/android/**content_view_core_impl.cc
>   M     content/public/android/java/**src/org/chromium/content/**
> browser/ContentView.java
>   M     content/public/android/java/**src/org/chromium/content/**
> browser/ContentViewClient.java
>   M     content/public/android/java/**src/org/chromium/content/**
> browser/ContentViewCore.java
>   MM    content/public/android/java/**src/org/chromium/content/**
> browser/accessibility/**AccessibilityInjector.java
>   M     content/public/android/java/**src/org/chromium/content/**
> browser/accessibility/**JellyBeanAccessibilityInjector**.java
>   MM    content/public/test/android/**javatests/src/org/chromium/**
> content/browser/test/util/**TestCallbackHelperContainer.**java
>   MM    content/public/test/android/**javatests/src/org/chromium/**
> content/browser/test/util/**TestContentViewClient.java
>
>
>