Allow a bit of wiggle room in highlight's bounding box

Allow a bit of wiggle room in highlight's bounding box. This should help reduce the noise on speed index. Previously, we were searching through the frames for when the orange highlight disappears, but with compression artifacts we might see a slight variations in each frame's highlight bounding box.


BUG=

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

[tonyg, 2014-02-13 01:16:40]

lgtm

https://codereview.chromium.org/152643010/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/core/tab.py (right):

https://codereview.chromium.org/152643010/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/core/tab.py:182: if bounds_difference > 4:
We've hardcoded 4 and 8 in this method.

Let's make them optional parameters in the method signature:

def _FindHighlightBoundingBox(..., bounds_tolerance=4, color_tolerance=8)

[shatch, 2014-02-13 01:26:48]

The CQ bit was checked by simonhatch@chromium.org

[commit-bot: I haz the power, 2014-02-13 01:28:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simonhatch@chromium.org/152643010/90001

[commit-bot: I haz the power, 2014-02-13 04:25:20]

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

[commit-bot: I haz the power, 2014-02-13 04:25:21]

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

[tonyg, 2014-02-13 05:35:35]

The CQ bit was checked by tonyg@chromium.org

[commit-bot: I haz the power, 2014-02-13 05:36:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/simonhatch@chromium.org/152643010/90001

[commit-bot: I haz the power, 2014-02-13 15:06:59]

Change committed as 251010