X-Git-Url: http://review.tizen.org/git/?a=blobdiff_plain;f=src%2Ftools%2Ftelemetry%2Ftelemetry%2Fcore%2Fbrowser_credentials.py;h=5842aa46ceeaabb680a41e5f943f4c8a4029afb8;hb=1afa4dd80ef85af7c90efaea6959db1d92330844;hp=faeadeb49c0faba7a7011bb18bec61f5afacae1f;hpb=90762837333c13ccf56f2ad88e4481fc71e8d281;p=platform%2Fframework%2Fweb%2Fcrosswalk.git diff --git a/src/tools/telemetry/telemetry/core/browser_credentials.py b/src/tools/telemetry/telemetry/core/browser_credentials.py index faeadeb..5842aa4 100644 --- a/src/tools/telemetry/telemetry/core/browser_credentials.py +++ b/src/tools/telemetry/telemetry/core/browser_credentials.py @@ -28,6 +28,7 @@ class BrowserCredentials(object): backends = [ codepen_credentials_backend.CodePenCredentialsBackend(), facebook_credentials_backend.FacebookCredentialsBackend(), + facebook_credentials_backend.FacebookCredentialsBackend2(), google_credentials_backend.GoogleCredentialsBackend()] self._backends = {} @@ -123,28 +124,18 @@ class BrowserCredentials(object): if k in self._extra_credentials: self._credentials[k] = self._extra_credentials[k] - def WarnIfMissingCredentials(self, page_set): - num_pages_missing_login = 0 - missing_credentials = set() - for page in page_set: - if (page.credentials - and not self.CanLogin(page.credentials)): - num_pages_missing_login += 1 - missing_credentials.add(page.credentials) - - if num_pages_missing_login > 0: + def WarnIfMissingCredentials(self, page): + if (page.credentials and not self.CanLogin(page.credentials)): files_to_tweak = [] - if page_set.credentials_path: - files_to_tweak.append( - os.path.relpath(os.path.join(os.path.dirname(page_set.file_path), - page_set.credentials_path))) + if page.credentials_path: + files_to_tweak.append(page.credentials_path) files_to_tweak.append('~/.telemetry-credentials') example_credentials_file = os.path.join( util.GetTelemetryDir(), 'examples', 'credentials_example.json') logging.warning(""" - Credentials for %s were not found. %i pages will not be tested. + Credentials for %s were not found. page %s will not be tested. To fix this, either follow the instructions to authenticate to gsutil here: @@ -153,7 +144,5 @@ class BrowserCredentials(object): or add your own credentials to: %s An example credentials file you can copy from is here: - %s\n""" % (', '.join(missing_credentials), - num_pages_missing_login, - ' or '.join(files_to_tweak), - example_credentials_file)) + %s\n""" % (page.credentials, page, ' or '.join(files_to_tweak), + example_credentials_file))