Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(28)

Unified Diff: tools/perf/page_sets/login_helpers/facebook_login.py

Issue 2168743004: [system health] Add media browsing stories. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: disable facebook on desktop Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tools/perf/page_sets/login_helpers/facebook_login.py
diff --git a/tools/perf/page_sets/login_helpers/facebook_login.py b/tools/perf/page_sets/login_helpers/facebook_login.py
index 360cf28da6f7f6fdaa77f10ba8b2c04a3ab068cf..a07ddff40d22b5c2c59bb01a46af141d0ab57bf1 100644
--- a/tools/perf/page_sets/login_helpers/facebook_login.py
+++ b/tools/perf/page_sets/login_helpers/facebook_login.py
@@ -33,3 +33,11 @@ def LoginWithMobileSite(
action_runner.WaitForElement(text='OK')
action_runner.ClickElement(text='OK')
action_runner.WaitForNavigate()
+
+
+def LoginWithDesktopSite(
+ action_runner, credential,
+ credentials_path=login_utils.DEFAULT_CREDENTIAL_PATH):
+ # Currently we use the mobile login page also on Desktop because the
+ # Desktop version requires enabled cookies and rejects the login.
+ return LoginWithMobileSite(action_runner, credential, credentials_path)
« no previous file with comments | « tools/perf/page_sets/data/system_health_mobile_040.wpr.sha1 ('k') | tools/perf/page_sets/system_health/browsing_stories.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698