Telemetry: extends Platform abstraction.

tools/telemetry/telemetry/core/browser.py

Index: tools/telemetry/telemetry/core/browser.py
diff --git a/tools/telemetry/telemetry/core/browser.py b/tools/telemetry/telemetry/core/browser.py
index 2f39b65d5ed1cb782712958682b71ffcca91fa04..d81144bbc2974fcb5d4501dec7f0e3e5880bf873 100644
--- a/tools/telemetry/telemetry/core/browser.py
+++ b/tools/telemetry/telemetry/core/browser.py
@@ -35,6 +35,7 @@ class Browser(object):
     self.credentials = browser_credentials.BrowserCredentials()
 
   def __enter__(self):
+    self._platform.SetFullPerformanceModeEnabled(True)
     return self
 
   def __exit__(self, *args):
@@ -89,6 +90,7 @@ class Browser(object):
 
   def Close(self):
     """Closes this browser."""
+    self._platform.SetFullPerformanceModeEnabled(False)

[tonyg, 2013/02/22 17:19:09]

It feels really strange to have one in __enter__ and the other in Close. I
recommend __enter__ and __exit__ or else __init__ and Close. Not sure which is
more appropriate.

[bulach, 2013/02/22 17:43:48]

yeah, it does look weird... I went for __init__ and Close() so enter/exit are
unchanged..

     if self._wpr_server:
       self._wpr_server.Close()
       self._wpr_server = None