Chromium Code Reviews| Index: tools/perf/benchmarks/page_cycler.py |
| diff --git a/tools/perf/benchmarks/page_cycler.py b/tools/perf/benchmarks/page_cycler.py |
| index 21f3449e76b48a3f95bcbcb2731e488fd8d04bb2..c1100bce73e5106cc8a041f85c546438d014c140 100644 |
| --- a/tools/perf/benchmarks/page_cycler.py |
| +++ b/tools/perf/benchmarks/page_cycler.py |
| @@ -117,6 +117,7 @@ class PageCyclerIntlKoThVi(_PageCycler): |
| @benchmark.Enabled('android') |
| +@benchmark.Disabled('all') # crbug.com/633222 |
|
shatch
2016/08/02 16:02:14
Is this right? This is kinda weird, since enabled
nednguyen
2016/08/02 16:54:30
Yeah, Enabled is white-listing, whereas Disabled i
|
| class PageCyclerTop10Mobile(_PageCycler): |
| """Page load time benchmark for the top 10 mobile web pages. |
| @@ -135,11 +136,6 @@ class PageCyclerTop10Mobile(_PageCycler): |
| stories.RemoveStory(found) |
| return stories |
| - @classmethod |
| - def ShouldDisable(cls, possible_browser): |
| - # http://crbug.com/633222 (Android Nexus 5). |
| - return possible_browser.platform.GetDeviceTypeName() == 'Nexus 5' |
| - |
| @benchmark.Disabled('android') # crbug.com/357326 |
| class PageCyclerToughLayoutCases(_PageCycler): |