Disable page_cycler.basic_oopif on zenbook bot

tools/perf/benchmarks/page_cycler.py

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
+import os

[petrcermak, 2016/03/18 15:49:13]

nit: I think that there should be an empty line above this one.

[nednguyen, 2016/03/18 15:52:23]

Done.

 
 from core import perf_benchmark
 
 from measurements import page_cycler
 import page_sets
 from telemetry import benchmark
 
 
 class _PageCycler(perf_benchmark.PerfBenchmark):
   options = {'pageset_repeat': 6}
@@ skipping 229 matching lines @@
   """ A benchmark measuring performance of out-of-process iframes. """
   page_set = page_sets.OopifBasicPageSet
 
   @classmethod
   def Name(cls):
     return 'page_cycler_site_isolation.basic_oopif'
 
   def SetExtraBrowserOptions(self, options):
     options.AppendExtraBrowserArgs(['--site-per-process'])
 
+  @classmethod
+  def ShouldDisable(cls, possible_browser):  # http://crbug.com/596067
+    # This test is flaky on zenbook laptops of chromium perf lab.

[petrcermak, 2016/03/18 15:49:13]

nit: s/zenbook/Zenbook/, s/of chromium/in the Chromium/

[nednguyen, 2016/03/18 15:52:23]

Done.

+    return 'Zenbook' in os.getenv('BUILDBOT_BUILDERNAME', '')
+
 
 @benchmark.Disabled('reference')  # crbug.com/523346
 class PageCyclerBasicOopif(_PageCycler):
   """ A benchmark measuring performance of the out-of-process iframes page
   set, without running in out-of-process iframes mode.. """
   page_set = page_sets.OopifBasicPageSet
 
   @classmethod
   def Name(cls):
     return 'page_cycler.basic_oopif'
 
 
 @benchmark.Disabled('all')  # crbug.com/443730
 class PageCyclerBigJs(_PageCycler):
   page_set = page_sets.BigJsPageSet
 
   @classmethod
   def Name(cls):
     return 'page_cycler.big_js'