Implementing a Monsoon power monitor trace agent, utilizing the UI infrastructure that the BattOr a…

systrace/profile_chrome/fake_agent_1.py

 # Copyright 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.
 
 import optparse
 import tempfile
 
 from systrace import trace_result
 from systrace import tracing_agents
 
@@ skipping 20 matching lines @@
   def GetResults(self, timeout=None):
     trace_data = open(self._PullTrace()).read()
     return trace_result.TraceResult('fakeData', trace_data)
 
   def _PullTrace(self):
     with tempfile.NamedTemporaryFile(delete=False) as f:
       self.filename = f.name
       f.write(self.contents)
       return f.name
 
   # pylint: disable=no-self-use

[Chris Craik, 2017/09/26 17:57:35]

why override in the fake agents?

[ddeptford, 2017/09/26 18:05:49]

The fake agents don't inherit from the common base class.

+  def IsConnectionOwner(self):
+    return False
+
+  # pylint: disable=no-self-use
   def SupportsExplicitClockSync(self):
     return False
 
   # pylint: disable=unused-argument, no-self-use
   def RecordClockSyncMarker(self, sync_id, did_record_sync_marker_callback):
     print ('Clock sync marker cannot be recorded since explicit clock sync '
            'is not supported.')
 
   def __repr__(self):
     return 'faketrace'
 
 
 class FakeConfig(tracing_agents.TracingConfig):
   def __init__(self):
     tracing_agents.TracingConfig.__init__(self)
 
 
 # pylint: disable=unused-argument
 def try_create_agent(config):
   return FakeAgent()
 
 def add_options(parser):
   options = optparse.OptionGroup(parser, 'Fake options.')
   return options
 
 # pylint: disable=unused-argument
 def get_config(options):
   return FakeConfig()