[chromedriver] Accept <webview> tags in chromedriver

[chromedriver] Accept <webview> tags in chromedriver

BUG=chromedriver:916
Committed: https://crrev.com/5dc7274c4f33b0609cfe0162492038a649fcb899
Cr-Commit-Position: refs/heads/master@{#299816}

[samuong, 2014-10-15 23:50:35]

samuong@chromium.org changed reviewers:
+ stgao@chromium.org

[stgao, 2014-10-16 00:00:11]

lgtm with nit.

https://codereview.chromium.org/655373002/diff/1/chrome/test/chromedriver/chr...
File chrome/test/chromedriver/chrome/chrome_impl.cc (right):

https://codereview.chromium.org/655373002/diff/1/chrome/test/chromedriver/chr...
chrome/test/chromedriver/chrome/chrome_impl.cc:55: if (view.type !=
WebViewInfo::kPage && view.type != WebViewInfo::kApp)
If we want to support switching to a webview tab, you may have to change this.

[samuong, 2014-10-16 00:03:16]

On 2014/10/16 00:00:11, Shuotao wrote:
> lgtm with nit.
> 
>
https://codereview.chromium.org/655373002/diff/1/chrome/test/chromedriver/chr...
> File chrome/test/chromedriver/chrome/chrome_impl.cc (right):
> 
>
https://codereview.chromium.org/655373002/diff/1/chrome/test/chromedriver/chr...
> chrome/test/chromedriver/chrome/chrome_impl.cc:55: if (view.type !=
> WebViewInfo::kPage && view.type != WebViewInfo::kApp)
> If we want to support switching to a webview tab, you may have to change this.

Yep. I'll put that in a separate commit once we've tested webviews a bit more
thoroughly.

[samuong, 2014-10-16 00:03:23]

The CQ bit was checked by samuong@chromium.org

[commit-bot: I haz the power, 2014-10-16 00:04:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/655373002/1

[commit-bot: I haz the power, 2014-10-16 01:59:47]

Committed patchset #1 (id:1)

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

Patchset 1 (id:??) landed as
https://crrev.com/5dc7274c4f33b0609cfe0162492038a649fcb899
Cr-Commit-Position: refs/heads/master@{#299816}