[Telemetry refactor] Migrate clients to new JavaScript API (batch 3)

tools/android/loading/devtools_monitor.py

 # Copyright (c) 2016 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Library handling DevTools websocket interaction.
 """
 
 import datetime
 import httplib
 import json
@@ skipping 270 matching lines @@
 
   def ExecuteJavaScript(self, expression):
     """Run JavaScript expression.
 
     Args:
       expression: JavaScript expression to run.
 
     Returns:
       The return value from the JavaScript expression.
     """
+    # Note: Clients may be tempted to do naive string interpolation to inject
+    # Python values into the JavaScript expression, which could lead to syntax
+    # errors during evaluation (e.g. injecting strings with special characters).
+    # If this becomes an issue, consider extending the interface of this method
+    # as in: https://github.com/catapult-project/catapult/issues/3028
     response = self.SyncRequest('Runtime.evaluate', {
         'expression': expression,
         'returnByValue': True})
     if 'error' in response:
       raise Exception(response['error']['message'])
     if 'wasThrown' in response['result'] and response['result']['wasThrown']:
       raise Exception(response['error']['result']['description'])
     if response['result']['result']['type'] == 'undefined':
       return None
     return response['result']['result']['value']
@@ skipping 179 matching lines @@
       json_data: (dict) Parsed from a JSON file using the json module.
 
     Returns:
       a Track instance.
     """
     # There is no sensible way to deserialize this abstract class, but
     # subclasses are not required to define a deserialization method. For
     # example, for testing we have a FakeRequestTrack which is never
     # deserialized; instead fake instances are deserialized as RequestTracks.
     assert False