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

Unified Diff: tools/telemetry/telemetry/internal/browser/browser_credentials.py

Issue 1458083003: [Telemetry + tools/perf] Modify the pylint disable message to use symbolic name (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 1 month 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/telemetry/telemetry/internal/browser/browser_credentials.py
diff --git a/tools/telemetry/telemetry/internal/browser/browser_credentials.py b/tools/telemetry/telemetry/internal/browser/browser_credentials.py
index ac9cc45a57a278f27b0a52231ff8b84dd2346592..54b8fd106108eb43bafd4b1a4a3ca4e4f522c687 100644
--- a/tools/telemetry/telemetry/internal/browser/browser_credentials.py
+++ b/tools/telemetry/telemetry/internal/browser/browser_credentials.py
@@ -69,11 +69,11 @@ class BrowserCredentials(object):
self._backends[credentials_type].LoginNoLongerNeeded(tab)
@property
- def credentials_path(self): # pylint: disable=E0202
+ def credentials_path(self):
return self._credentials_path
@credentials_path.setter
- def credentials_path(self, credentials_path): # pylint: disable=E0202
+ def credentials_path(self, credentials_path):
self._credentials_path = credentials_path
self._RebuildCredentials()
@@ -90,7 +90,8 @@ class BrowserCredentials(object):
Should only be used in unit tests to simulate --dont-override-profile.
"""
for backend in self._backends.keys():
- self._backends[backend]._ResetLoggedInState() # pylint: disable=W0212
+ # pylint: disable=protected-access
+ self._backends[backend]._ResetLoggedInState()
def _RebuildCredentials(self):
credentials = {}

Powered by Google App Engine
This is Rietveld 408576698