[Chromedriver] Enabling java tests ClickScrollingTest & WindowSwitchingTest on android waterfall bo…

[Chromedriver] Enabling java tests ClickScrollingTest & WindowSwitchingTest on android waterfall bots.

BUG=chromedriver:1006, chromedriver:1044
Committed: https://crrev.com/835b8cdd89f21d32915d5f0ec46478808a98385c
Cr-Commit-Position: refs/heads/master@{#374716}

[gmanikpure, 2016-01-06 22:51:46]

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

[gmanikpure, 2016-01-06 22:51:47]

Hi Sam,

Could you please have a look?

For chromedriver issue 1006, ClickScollingTest works fine on android
chrome_stable, chrome_beta & chromium except for chromedriver_webview_shell. 

For chromedriver issue 1044, WindowSwitchingTest works fine on chromium but
fails on android chrome_stable, chrome_beta and chromedriver_webview_shell.

Enabling the tests accordingly.

[samuong, 2016-01-12 22:41:55]

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
File chrome/test/chromedriver/test/test_expectations (right):

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
chrome/test/chromedriver/test/test_expectations:237: #
https://bugs.chromium.org/p/chromedriver/issues/detail?id=1006   
'ClickScrollingTest.testShouldBeAbleToClickElementThatIsOutOfViewInANestedFrameThatIsOutOfView',
is this supposed to be two lines?

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
chrome/test/chromedriver/test/test_expectations:238: # Not applicable on
ChromeShell (doesn't support tabs).
"ChromeShell" -> "ChromeDriverWebViewShell"

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
chrome/test/chromedriver/test/test_expectations:239: #
https://bugs.chromium.org/p/chromedriver/issues/detail?id=1044
this bug doesn't apply here because the test doesn't apply to webview shell (you
can just delete this line)

[gmanikpure, 2016-02-08 22:37:21]

Hi Sam,

Re-tested on the latest versions of chrome_stable,chrome_beta,chromium &
chromedriver_webview_shell and enabled/disabled the tests accordingly.

Could you please have a look?

Thanks

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
File chrome/test/chromedriver/test/test_expectations (right):

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
chrome/test/chromedriver/test/test_expectations:237: #
https://bugs.chromium.org/p/chromedriver/issues/detail?id=1006   
'ClickScrollingTest.testShouldBeAbleToClickElementThatIsOutOfViewInANestedFrameThatIsOutOfView',
On 2016/01/12 22:41:54, samuong wrote:
> is this supposed to be two lines?

Done.

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
chrome/test/chromedriver/test/test_expectations:238: # Not applicable on
ChromeShell (doesn't support tabs).
On 2016/01/12 22:41:54, samuong wrote:
> "ChromeShell" -> "ChromeDriverWebViewShell"

Done.

https://codereview.chromium.org/1562803004/diff/1/chrome/test/chromedriver/te...
chrome/test/chromedriver/test/test_expectations:239: #
https://bugs.chromium.org/p/chromedriver/issues/detail?id=1044
On 2016/01/12 22:41:54, samuong wrote:
> this bug doesn't apply here because the test doesn't apply to webview shell
(you
> can just delete this line)

Done.

https://codereview.chromium.org/1562803004/diff/20001/chrome/test/chromedrive...
File chrome/test/chromedriver/test/test_expectations (right):

https://codereview.chromium.org/1562803004/diff/20001/chrome/test/chromedrive...
chrome/test/chromedriver/test/test_expectations:210: ]
WindowSwitchingTest.* now runs fine on latest version of chrome stable
v48.0.2564.95. Hence, removing from the negative filter in patch3

https://codereview.chromium.org/1562803004/diff/20001/chrome/test/chromedrive...
chrome/test/chromedriver/test/test_expectations:226: 'WindowSwitchingTest.*',
WindowSwitchingTest.* now runs fine on latest version of chrome beta
v49.0.2623.34. Hence, removing it from chrome_beta negative filter in patch3

[samuong, 2016-02-10 19:59:24]

lgtm

[gmanikpure, 2016-02-10 20:04:16]

The CQ bit was checked by gmanikpure@chromium.org

[commit-bot: I haz the power, 2016-02-10 20:07:24]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1562803004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1562803004/40001

[commit-bot: I haz the power, 2016-02-10 20:52:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-02-16 22:30:51]

Description was changed from

==========
[Chromedriver] Enabling java tests ClickScrollingTest & WindowSwitchingTest on
android waterfall bots.

BUG=chromedriver:1006 , chromedriver:1044
==========

to

==========
[Chromedriver] Enabling java tests ClickScrollingTest & WindowSwitchingTest on
android waterfall bots.

BUG=chromedriver:1006 , chromedriver:1044

Committed: https://crrev.com/835b8cdd89f21d32915d5f0ec46478808a98385c
Cr-Commit-Position: refs/heads/master@{#374716}
==========

[commit-bot: I haz the power, 2016-02-16 22:30:52]

Patchset 3 (id:??) landed as
https://crrev.com/835b8cdd89f21d32915d5f0ec46478808a98385c
Cr-Commit-Position: refs/heads/master@{#374716}