Reland: [Telemetry] Fix image_decoding_measurement timeout.

Reland: [Telemetry] Fix image_decoding_measurement timeout.

This was reverted because it caused some timeouts. The reland is the same as the
original but I now preseved the original timeout values of Page.enable and
Page.disable.

> util.WaitFor polls which is not appropriate for PerformActionAndWaitForNavigate.
>
> Since DispatchNotifications blocks in recv until it gets some data, there is no
> need to poll at an interval. Instead, we should call recv again as soon as
> possible after dispatching a notification.
>
> In the case of the image decoding measurement, it runs timeline recording. There
> were so many timeline messages to receive that we were not always getting to the
> important Page.navigate message before timing out.
>
> Example:
> http://build.chromium.org/p/chromium.perf/builders/Linux%20Perf%20%281%29/builds/27098
>
> BUG=314375
>
> Review URL: https://codereview.chromium.org/47013004

TBR=kbr@chromium.org
BUG=

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=233543

[commit-bot: I haz the power, 2013-11-06 03:57:54]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tonyg@chromium.org/61483002/1

[commit-bot: I haz the power, 2013-11-06 04:45:17]

Step "update" is always a major failure.
Look at the try server FAQ for more details.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win_x64_re...

[commit-bot: I haz the power, 2013-11-06 05:17:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tonyg@chromium.org/61483002/1

[Ken Russell (switch to Gerrit), 2013-11-06 08:12:14]

I'd like to point out that sending this CL to the win_gpu, linux_gpu, mac_gpu,
and mac_gpu_retina try servers would be a good way of checking that it doesn't
reintroduce the regression. These machines are debug builders and testers.

[commit-bot: I haz the power, 2013-11-06 09:54:32]

Retried try job too often on linux_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_aura...

[tonyg, 2013-11-06 14:34:59]

O nice, didn't realize there were trybots. I've added them.

[commit-bot: I haz the power, 2013-11-06 14:38:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tonyg@chromium.org/61483002/1

[commit-bot: I haz the power, 2013-11-06 18:41:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tonyg@chromium.org/61483002/390001

[tonyg, 2013-11-06 18:42:44]

Unfortunately, the gpu trybots (or maybe all trybots?) are busted:
https://code.google.com/p/chromium/issues/detail?id=315777

Back in the CQ.

On Wed, Nov 6, 2013 at 10:41 AM,  <commit-bot@chromium.org> wrote:
> CQ is trying da patch. Follow status at
> https://chromium-status.appspot.com/cq/tonyg@chromium.org/61483002/390001
>
>
> https://codereview.chromium.org/61483002/

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[commit-bot: I haz the power, 2013-11-07 08:12:43]

Change committed as 233543