Enable smoothness.tough_pinch_zoom_cases on Chrome OS

Enable smoothness.tough_pinch_zoom_cases on Chrome OS

Also, fix a bug in decorators.py where every element in |enabled_strings| except the first one is ignored.

BUG=443981
TEST=tools/perf/run_benchmark --browser=cros-chrome --remote=${host} smoothness.tough_pinch_zoom_cases

Committed: https://crrev.com/26e59874b04d1ecdf2e373c335d315f76352014d
Cr-Commit-Position: refs/heads/master@{#309339}

[hashimoto, 2014-12-19 12:19:14]

hashimoto@chromium.org changed reviewers:
+ nednguyen@google.com

[nednguyen, 2014-12-19 14:29:49]

https://codereview.chromium.org/820433003/diff/1/tools/telemetry/telemetry/de...
File tools/telemetry/telemetry/decorators.py (left):

https://codereview.chromium.org/820433003/diff/1/tools/telemetry/telemetry/de...
tools/telemetry/telemetry/decorators.py:133: def ShouldSkip(test,
possible_browser):
Can you add unittest for this method?

[hashimoto, 2014-12-19 17:14:46]

https://codereview.chromium.org/820433003/diff/1/tools/telemetry/telemetry/de...
File tools/telemetry/telemetry/decorators.py (left):

https://codereview.chromium.org/820433003/diff/1/tools/telemetry/telemetry/de...
tools/telemetry/telemetry/decorators.py:133: def ShouldSkip(test,
possible_browser):
On 2014/12/19 14:29:48, nednguyen wrote:
> Can you add unittest for this method?

Added very basic tests.

[tonyg, 2014-12-19 17:24:02]

tonyg@chromium.org changed reviewers:
+ tonyg@chromium.org

[tonyg, 2014-12-19 17:24:03]

https://codereview.chromium.org/820433003/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/decorators_unittest.py (right):

https://codereview.chromium.org/820433003/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/decorators_unittest.py:1: # Copyright 2014 The
Chromium Authors. All rights reserved.
Thanks for adding these. FYI, there are also some unittests for this in
unittest_util/run_tests_unittests.py. I defer to Ned on this review, but suggest
making sure the coverage is normalized.

[nednguyen, 2014-12-20 01:18:22]

LGTM

https://codereview.chromium.org/820433003/diff/20001/tools/telemetry/telemetr...
File tools/telemetry/telemetry/decorators_unittest.py (right):

https://codereview.chromium.org/820433003/diff/20001/tools/telemetry/telemetr...
tools/telemetry/telemetry/decorators_unittest.py:1: # Copyright 2014 The
Chromium Authors. All rights reserved.
On 2014/12/19 17:24:03, tonyg (OOO til Jan 5) wrote:
> Thanks for adding these. FYI, there are also some unittests for this in
> unittest_util/run_tests_unittests.py. I defer to Ned on this review, but
suggest
> making sure the coverage is normalized.

A bunch of test cases in run_tests_unittest do not have clear intention and are
not as well coverage as compared to these tests hashimoto are adding (tests are
devided by platform rather than test cases to enable/disable matching logic) . 

So I would suggest Hashimoto remove all of them and leave just 1 testcase left
as an end-to-end test.

[hashimoto, 2014-12-20 03:50:31]

The CQ bit was checked by hashimoto@chromium.org

[commit-bot: I haz the power, 2014-12-20 03:51:23]

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

[commit-bot: I haz the power, 2014-12-20 05:25:46]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2014-12-20 05:26:34]

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

[nednguyen, 2014-12-20 14:45:28]

On 2014/12/20 05:26:34, I haz the power (commit-bot) wrote:
> Patchset 2 (id:??) landed as
> https://crrev.com/26e59874b04d1ecdf2e373c335d315f76352014d
> Cr-Commit-Position: refs/heads/master@{#309339}

Hi Hashimoto, do you mind deleting some test cases in
unittest_util/run_tests_unittest that are already overlap with
decorator_unitttests?

[hashimoto, 2014-12-22 15:52:52]

On 2014/12/20 14:45:28, nednguyen wrote:
> On 2014/12/20 05:26:34, I haz the power (commit-bot) wrote:
> > Patchset 2 (id:??) landed as
> > https://crrev.com/26e59874b04d1ecdf2e373c335d315f76352014d
> > Cr-Commit-Position: refs/heads/master@{#309339}
> 
> Hi Hashimoto, do you mind deleting some test cases in
> unittest_util/run_tests_unittest that are already overlap with
> decorator_unitttests?

Sorry, I'll be ooo until Jan. 5.
It'd be appreciated if you could take care of it.
If you are overloaded, let me remove them as soon as I get back to work.

[nednguyen, 2014-12-22 16:05:21]

On 2014/12/22 15:52:52, hashimoto (ooo - Jan.4) wrote:
> On 2014/12/20 14:45:28, nednguyen wrote:
> > On 2014/12/20 05:26:34, I haz the power (commit-bot) wrote:
> > > Patchset 2 (id:??) landed as
> > > https://crrev.com/26e59874b04d1ecdf2e373c335d315f76352014d
> > > Cr-Commit-Position: refs/heads/master@{#309339}
> > 
> > Hi Hashimoto, do you mind deleting some test cases in
> > unittest_util/run_tests_unittest that are already overlap with
> > decorator_unitttests?
> 
> Sorry, I'll be ooo until Jan. 5.
> It'd be appreciated if you could take care of it.
> If you are overloaded, let me remove them as soon as I get back to work.

I am overloaded, so removing them as soon as you get back to work sounds good to
me.