Update android WebView to use WebElementCollection instead of WebNodeCollection

Update android WebView to use WebElementCollection instead of WebNodeCollection

Update android WebView to use WebElementCollection instead of WebNodeCollection.
After Blink r166411 and r166500, WebNodeCollection is renamed to
WebElementCollection and its API returns WebElements instead of WebNodes.

The new code is clearer and avoids an unnecessary isElementNode() check.

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

[benm (inactive), 2014-02-07 14:46:15]

thanks!

https://codereview.chromium.org/149313005/diff/40001/android_webview/renderer...
File android_webview/renderer/aw_render_view_ext.cc (right):

https://codereview.chromium.org/149313005/diff/40001/android_webview/renderer...
android_webview/renderer/aw_render_view_ext.cc:57: blink::WebElement
GetImgChild(const blink::WebNode& node) {
Should the parameter be WebElement too?

[Inactive, 2014-02-07 15:09:41]

https://codereview.chromium.org/149313005/diff/40001/android_webview/renderer...
File android_webview/renderer/aw_render_view_ext.cc (right):

https://codereview.chromium.org/149313005/diff/40001/android_webview/renderer...
android_webview/renderer/aw_render_view_ext.cc:57: blink::WebElement
GetImgChild(const blink::WebNode& node) {
On 2014/02/07 14:46:15, benm wrote:
> Should the parameter be WebElement too?

This is not strictly necessary but it doesn't hurt. I updated the patch
accordingly.

[Inactive, 2014-02-10 20:32:48]

ping review?

[benm (inactive), 2014-02-10 21:14:05]

lgtm

sorry for the delay

[Inactive, 2014-02-10 21:24:09]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-02-10 21:25:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/149313005/100001

[commit-bot: I haz the power, 2014-02-10 22:02:15]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-02-10 22:02:16]

Retried try job too often on win for step(s) compile
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...

[Inactive, 2014-02-10 22:27:49]

The CQ bit was checked by ch.dumez@samsung.com

[commit-bot: I haz the power, 2014-02-10 22:32:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/149313005/410001

[commit-bot: I haz the power, 2014-02-10 22:58:18]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/149313005/410001

[commit-bot: I haz the power, 2014-02-10 23:54:08]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ch.dumez@samsung.com/149313005/410001

[commit-bot: I haz the power, 2014-02-11 02:14:00]

Change committed as 250284