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

Unified Diff: tools/telemetry/telemetry/core/local_server.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/core/local_server.py
diff --git a/tools/telemetry/telemetry/core/local_server.py b/tools/telemetry/telemetry/core/local_server.py
index 0a3d77dead607fa4996225d9000926c3467a880d..38dcf510732ce6095739946b8020d7a42d0de63f 100644
--- a/tools/telemetry/telemetry/core/local_server.py
+++ b/tools/telemetry/telemetry/core/local_server.py
@@ -201,7 +201,8 @@ def _LocalServerBackendMain(args):
# Note: This message is scraped by the parent process'
# _GetNamedPortsFromBackend(). Do **not** change it.
print 'LocalServerBackend started: %s' % json.dumps(
- [pair._asdict() for pair in named_ports]) # pylint: disable=W0212
+ [pair._asdict() # pylint: disable=protected-access
+ for pair in named_ports])
sys.stdout.flush()
return server.ServeForever()
@@ -211,6 +212,7 @@ if __name__ == '__main__':
# This trick is needed because local_server.NamedPort is not the
# same as sys.modules['__main__'].NamedPort. The module itself is loaded
# twice, basically.
- from telemetry.core import local_server # pylint: disable=W0406
- sys.exit(local_server._LocalServerBackendMain( # pylint: disable=W0212
+ from telemetry.core import local_server # pylint: disable=import-self
+ sys.exit(
+ local_server._LocalServerBackendMain( # pylint: disable=protected-access
sys.argv[1:]))
« no previous file with comments | « tools/perf/profile_creators/profile_generator_unittest.py ('k') | tools/telemetry/telemetry/core/os_version.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698