[Android WebView] Pass absolute URL to requestFocusNodeHref.

[Android WebView] Pass absolute URL to requestFocusNodeHref.

This CL changes the semantic of the |url| field of the
requestFocusNodeHref method for compatility reasons, keeping consistent
with the behavior of the old WebView.

BUG=290905
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=223298

[boliu, 2013-09-13 16:02:24]

lgtm if WebKitHitTestTest passes

https://codereview.chromium.org/23454036/diff/1/android_webview/java/src/org/...
File android_webview/java/src/org/chromium/android_webview/AwContents.java
(right):

https://codereview.chromium.org/23454036/diff/1/android_webview/java/src/org/...
android_webview/java/src/org/chromium/android_webview/AwContents.java:1296:
data.putString("url", mPossiblyStaleHitTestData.hitTestResultExtraData);
Ah hahaha, I read the public documentation and assumed it to be correct:

http://developer.android.com/reference/android/webkit/WebView.html#requestFoc...

"url" returns the anchor's href attribute.

Wonder if one of the instrumentation tests needs to be updated after this.

[joth, 2013-09-13 20:24:14]

lgtm

https://codereview.chromium.org/23454036/diff/7001/android_webview/java/src/o...
File android_webview/java/src/org/chromium/android_webview/AwContents.java
(right):

https://codereview.chromium.org/23454036/diff/7001/android_webview/java/src/o...
android_webview/java/src/org/chromium/android_webview/AwContents.java:1292: //
the absolute (full) url is passed in the |url| field, not only the href
attribute.
(feel free to refer to the cleanup crbug here too if you like)

[Primiano Tucci (use gerrit), 2013-09-15 22:06:07]

org.chromium.android_webview.test.WebKitHitTestTest: all test passed.

[commit-bot: I haz the power, 2013-09-15 22:40:52]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/primiano@chromium.org/23454036/15001

[commit-bot: I haz the power, 2013-09-16 02:44:51]

Change committed as 223298