DevTools: deprecate InspectorTest.runAfterPendingDispatches

DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
BUG=602843
Committed: https://crrev.com/cb18158bf1f1ef145d2aa6ebe7481b9bc14c1b4d
Cr-Commit-Position: refs/heads/master@{#387101}

[lushnikov, 2016-04-13 00:05:44]

please, take a look.

This is to avoid the temptation of using the method in future.

[pfeldman, 2016-04-13 00:10:51]

On 2016/04/13 00:05:44, lushnikov wrote:
> please, take a look.
> 
> This is to avoid the temptation of using the method in future.

Although being a bad / discouraged method, it has never contributed to the
flakiness. I wonder what has changed.

[lushnikov, 2016-04-13 00:17:36]

Description was changed from

==========
DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
==========

to

==========
DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
BUG=602843
==========

[lushnikov, 2016-04-13 00:19:01]

I have a history of flakiness produced by this method.
The latest example is here: 
https://codereview.chromium.org/1878123003/

[dgozman, 2016-04-13 19:39:03]

lgtm, but we have to work on removing it gradually.

[dgozman, 2016-04-13 19:39:09]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-04-13 19:39:18]

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

[commit-bot: I haz the power, 2016-04-13 19:53:02]

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

[commit-bot: I haz the power, 2016-04-13 19:53:03]

Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)

[lushnikov, 2016-04-13 21:07:47]

The CQ bit was checked by lushnikov@chromium.org

[commit-bot: I haz the power, 2016-04-13 21:08:06]

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

[commit-bot: I haz the power, 2016-04-13 21:28:55]

Description was changed from

==========
DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
BUG=602843
==========

to

==========
DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
BUG=602843
==========

[commit-bot: I haz the power, 2016-04-13 21:28:56]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-13 21:29:59]

Description was changed from

==========
DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
BUG=602843
==========

to

==========
DevTools: deprecate InspectorTest.runAfterPendingDispatches

The InspectorTest.runAfterPendingDispatches results in an eventual test
flakiness, increasing maintanence burden.

Clients should consider other ways instead of using
InspectorTest.runAfterPendingDispatches, e.g. adding a specific
_methodForTest method which will be sniffed.

R=dgozman
BUG=602843

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

[commit-bot: I haz the power, 2016-04-13 21:30:01]

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