Add delay promise and change automation scripts

Add delay promise and change automation scripts

Also update the scroll an element into the view
as it may cause some internal div to scroll
and fail the test.

BUG=643215
Committed: https://crrev.com/d916caa601149ec4a2a09d588dae8bb6777b321b
Cr-Commit-Position: refs/heads/master@{#417598}

[Navid Zolghadr, 2016-09-07 16:04:42]

The CQ bit was checked by nzolghadr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 16:04:58]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Navid Zolghadr, 2016-09-07 16:09:40]

nzolghadr@chromium.org changed reviewers:
+ dtapuska@chromium.org

[Navid Zolghadr, 2016-09-07 16:13:23]

The CQ bit was checked by nzolghadr@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-07 16:13:55]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-07 17:25:03]

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

[commit-bot: I haz the power, 2016-09-07 17:25:04]

Dry run: This issue passed the CQ dry run.

[Navid Zolghadr, 2016-09-08 15:20:31]

On 2016/09/07 17:55:44, Navid Zolghadr wrote:

dtapuska@ since you are adding automation scripts for new tests maybe it is
better to land this one sooner. As I changed the way scroll into view worked, I
was able to remove bunch of "> p" from the selectors in the similar test
automation that I believe you are going to use.

[Navid Zolghadr, 2016-09-08 15:20:32]

On 2016/09/07 17:55:44, Navid Zolghadr wrote:

dtapuska@ since you are adding automation scripts for new tests maybe it is
better to land this one sooner. As I changed the way scroll into view worked, I
was able to remove bunch of "> p" from the selectors in the similar test
automation that I believe you are going to use.

[Navid Zolghadr, 2016-09-09 14:28:15]

On 2016/09/08 15:20:32, Navid Zolghadr wrote:
> On 2016/09/07 17:55:44, Navid Zolghadr wrote:
> 
> dtapuska@ since you are adding automation scripts for new tests maybe it is
> better to land this one sooner. As I changed the way scroll into view worked,
I
> was able to remove bunch of "> p" from the selectors in the similar test
> automation that I believe you are going to use.

ping

[dtapuska, 2016-09-09 14:57:46]

On 2016/09/09 14:28:15, Navid Zolghadr wrote:
> On 2016/09/08 15:20:32, Navid Zolghadr wrote:
> > On 2016/09/07 17:55:44, Navid Zolghadr wrote:
> > 
> > dtapuska@ since you are adding automation scripts for new tests maybe it is
> > better to land this one sooner. As I changed the way scroll into view
worked,
> I
> > was able to remove bunch of "> p" from the selectors in the similar test
> > automation that I believe you are going to use.
> 
> ping

lgtm. this might give merge problems with
https://codereview.chromium.org/2322083002/ which is in the CQ now.

[Navid Zolghadr, 2016-09-09 14:59:01]

On 2016/09/09 14:57:46, dtapuska wrote:
> On 2016/09/09 14:28:15, Navid Zolghadr wrote:
> > On 2016/09/08 15:20:32, Navid Zolghadr wrote:
> > > On 2016/09/07 17:55:44, Navid Zolghadr wrote:
> > > 
> > > dtapuska@ since you are adding automation scripts for new tests maybe it
is
> > > better to land this one sooner. As I changed the way scroll into view
> worked,
> > I
> > > was able to remove bunch of "> p" from the selectors in the similar test
> > > automation that I believe you are going to use.
> > 
> > ping
> 
> lgtm. this might give merge problems with
> https://codereview.chromium.org/2322083002/ which is in the CQ now.

I'll rebase if that happens.

[Navid Zolghadr, 2016-09-09 14:59:07]

The CQ bit was checked by nzolghadr@chromium.org

[commit-bot: I haz the power, 2016-09-09 14:59:38]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-09 16:11:06]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-09 16:14:31]

Description was changed from

==========
Add delay promise and change automation scripts

Also update the scroll an element into the view
as it may cause some internal div to scroll
and fail the test.

BUG=643215
==========

to

==========
Add delay promise and change automation scripts

Also update the scroll an element into the view
as it may cause some internal div to scroll
and fail the test.

BUG=643215

Committed: https://crrev.com/d916caa601149ec4a2a09d588dae8bb6777b321b
Cr-Commit-Position: refs/heads/master@{#417598}
==========

[commit-bot: I haz the power, 2016-09-09 16:14:32]

Patchset 2 (id:??) landed as
https://crrev.com/d916caa601149ec4a2a09d588dae8bb6777b321b
Cr-Commit-Position: refs/heads/master@{#417598}