Skip to content

Commit

Permalink
Actually disable load:tools:gmail
Browse files Browse the repository at this point in the history
The decorator was applied to a base class which doesn't
actually cause anything to be disabled.

BUG=catapult#:3088

Review-Url: https://codereview.chromium.org/2575513002
Cr-Commit-Position: refs/heads/master@{#438149}
  • Loading branch information
chromy authored and Commit bot committed Dec 13, 2016
1 parent 9fb5b60 commit 2e324fe
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion tools/perf/page_sets/system_health/loading_stories.py
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,6 @@ class LoadDocsStory(_LoadingStory):
'https://docs.google.com/document/d/1GvzDP-tTLmJ0myRhUAfTYWs3ZUFilUICg8psNHyccwQ/edit?usp=sharing')


@decorators.Disabled('android') # crbug.com.com/664505
class _LoadGmailBaseStory(_LoadingStory):
NAME = 'load:tools:gmail'
URL = 'https://mail.google.com/mail/'
Expand Down Expand Up @@ -309,6 +308,8 @@ def _DidLoadDocument(self, action_runner):
action_runner.WaitForJavaScriptCondition(
'document.getElementById("loading").style.display === "none"')


@decorators.Disabled('android') # crbug.com.com/664505
class LoadGmailMobileStory(_LoadGmailBaseStory):
SUPPORTED_PLATFORMS = platforms.MOBILE_ONLY

Expand Down

0 comments on commit 2e324fe

Please sign in to comment.