Add a doc in the presubmit script checking if layout tests use eventSender

Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a 
doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340
Review-Url: https://codereview.chromium.org/2844803005
Cr-Commit-Position: refs/heads/master@{#468323}
Committed: https://chromium.googlesource.com/chromium/src/+/3e1724e5ccd3d41d417d13487c4b1376a6815a93

[lanwei, 2017-04-28 19:11:19]

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

[commit-bot: I haz the power, 2017-04-28 19:12:18]

Dry run: CQ is trying da patch.

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

[lanwei, 2017-04-28 19:34:17]

Description was changed from

==========
comments


hover

BUG=
==========

to

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender still be used for new layout tests. We add a doc to
explain why we want to replace EventSender and give some examples on how to
use this new API in testing.

BUG=711340
==========

[lanwei, 2017-04-28 19:34:17]

lanwei@chromium.org changed reviewers:
+ dtapuska@chromium.org, tdresser@chromium.org

[lanwei, 2017-04-28 19:34:27]

Patchset #1 (id:1) has been deleted

[lanwei, 2017-04-28 19:35:34]

Description was changed from

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender still be used for new layout tests. We add a doc to
explain why we want to replace EventSender and give some examples on how to
use this new API in testing.

BUG=711340
==========

to

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a doc to
explain why we want to replace EventSender and give some examples on how to
use this new API in testing.

BUG=711340
==========

[lanwei, 2017-04-28 19:36:03]

Description was changed from

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a doc to
explain why we want to replace EventSender and give some examples on how to
use this new API in testing.

BUG=711340
==========

to

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add 
a doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340
==========

[lanwei, 2017-04-28 19:36:22]

Description was changed from

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add 
a doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340
==========

to

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a 
doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340
==========

[commit-bot: I haz the power, 2017-04-28 20:46:45]

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

[commit-bot: I haz the power, 2017-04-28 20:46:46]

Dry run: This issue passed the CQ dry run.

[dtapuska, 2017-04-28 21:13:17]

https://codereview.chromium.org/2844803005/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/PRESUBMIT.py (right):

https://codereview.chromium.org/2844803005/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/PRESUBMIT.py:101: 'Files that still use
eventSender, please use chrome.gpuBenchmarking.pointerActionSequence instead: '
+
Not sure this is a complete sentence anymore. Perhaps

"eventSender is deprecated, please use.... ' 
'(see....' +
'Files: %s:%d....'

[tdresser, 2017-05-01 13:20:41]

LGTM

[lanwei, 2017-05-01 14:50:17]

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

[commit-bot: I haz the power, 2017-05-01 14:50:25]

Dry run: CQ is trying da patch.

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

[lanwei, 2017-05-01 14:51:46]

Patchset #2 (id:40001) has been deleted

[lanwei, 2017-05-01 15:53:10]

https://codereview.chromium.org/2844803005/diff/20001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/PRESUBMIT.py (right):

https://codereview.chromium.org/2844803005/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/PRESUBMIT.py:101: 'Files that still use
eventSender, please use chrome.gpuBenchmarking.pointerActionSequence instead: '
+
On 2017/04/28 21:13:17, dtapuska wrote:
> Not sure this is a complete sentence anymore. Perhaps
> 
> "eventSender is deprecated, please use.... ' 
> '(see....' +
> 'Files: %s:%d....'

Done.

[dtapuska, 2017-05-01 15:54:33]

On 2017/05/01 15:53:10, lanwei wrote:
>
https://codereview.chromium.org/2844803005/diff/20001/third_party/WebKit/Layo...
> File third_party/WebKit/LayoutTests/PRESUBMIT.py (right):
> 
>
https://codereview.chromium.org/2844803005/diff/20001/third_party/WebKit/Layo...
> third_party/WebKit/LayoutTests/PRESUBMIT.py:101: 'Files that still use
> eventSender, please use chrome.gpuBenchmarking.pointerActionSequence instead:
'
> +
> On 2017/04/28 21:13:17, dtapuska wrote:
> > Not sure this is a complete sentence anymore. Perhaps
> > 
> > "eventSender is deprecated, please use.... ' 
> > '(see....' +
> > 'Files: %s:%d....'
> 
> Done.

lgtm

[lanwei, 2017-05-01 16:12:59]

The CQ bit was unchecked by lanwei@chromium.org

[lanwei, 2017-05-01 16:13:01]

The CQ bit was checked by lanwei@chromium.org

[lanwei, 2017-05-01 16:13:01]

The patchset sent to the CQ was uploaded after l-g-t-m from
tdresser@chromium.org
Link to the patchset: https://codereview.chromium.org/2844803005/#ps60001
(title: "comments")

[commit-bot: I haz the power, 2017-05-01 16:13:12]

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, 2017-05-01 16:24:16]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1493655181100710,
"parent_rev": "7ad7f6d7a7478d21b30a505dd00e473a609b687d", "commit_rev":
"3e1724e5ccd3d41d417d13487c4b1376a6815a93"}

[commit-bot: I haz the power, 2017-05-01 16:24:26]

Description was changed from

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a 
doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340
==========

to

==========
Add a doc in the presubmit script checking if layout tests use eventSender

In https://codereview.chromium.org/2810943005/, we added a presubmit script
to check if eventSender is still used in new layout tests. Here, We add a 
doc to explain why we want to replace EventSender and give some examples
on how to use this new API in testing.

BUG=711340

Review-Url: https://codereview.chromium.org/2844803005
Cr-Commit-Position: refs/heads/master@{#468323}
Committed:
https://chromium.googlesource.com/chromium/src/+/3e1724e5ccd3d41d417d13487c4b...
==========

[commit-bot: I haz the power, 2017-05-01 16:24:27]

Committed patchset #2 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/3e1724e5ccd3d41d417d13487c4b...