Telemtry: fixes version detection for mobile and supports_tracing.

Telemtry: fixes version detection for mobile and supports_tracing.

BUG=170284
TEST=tools/telemetry/run_tests --browser=android-content-shell testGotTrace -vv


Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=180681

[bulach, 2013-01-21 19:29:04]

splitting a small bit from https://codereview.chromium.org/11931015/, ptal

[nduca, 2013-01-22 22:13:34]

lgtm

[commit-bot: I haz the power, 2013-01-25 21:30:17]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/12042015/1

[commit-bot: I haz the power, 2013-02-04 11:29:58]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/12042015/1

[commit-bot: I haz the power, 2013-02-04 11:29:59]

Failed to apply patch for tools/telemetry/telemetry/browser_backend.py:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file tools/telemetry/telemetry/browser_backend.py
  Hunk #1 FAILED at 94.
  Hunk #2 succeeded at 140 (offset 16 lines).
  1 out of 2 hunks FAILED -- saving rejects to file
tools/telemetry/telemetry/browser_backend.py.rej

Patch:       tools/telemetry/telemetry/browser_backend.py
Index: tools/telemetry/telemetry/browser_backend.py
diff --git a/tools/telemetry/telemetry/browser_backend.py
b/tools/telemetry/telemetry/browser_backend.py
index
de9e0929bb618401f0e7f3604d303952a279296d..29341b14c057ec509f4a37f0ac0e4b8bbd3041f6
100644
--- a/tools/telemetry/telemetry/browser_backend.py
+++ b/tools/telemetry/telemetry/browser_backend.py
@@ -94,7 +94,8 @@ class BrowserBackend(object):
     resp = json.loads(data)
     if 'Protocol-Version' in resp:
       self._inspector_protocol_version = resp['Protocol-Version']
-      mU = re.search('Chrome/\d+\.\d+\.(\d+)\.\d+ Safari', resp['User-Agent'])
+      mU = re.search('Chrome/\d+\.\d+\.(\d+)\.\d+ (Mobile?) Safari',
+                     resp['User-Agent'])
       mW = re.search('\((trunk)?\@(\d+)\)', resp['WebKit-Version'])
       if mU:
         self._chrome_branch_number = int(mU.group(1))
@@ -124,7 +125,7 @@ class BrowserBackend(object):
 
   @property
   def supports_tracing(self):
-    return True
+    return self.is_content_shell or self._chrome_branch_number >= 1385
 
   def StartTracing(self):
     if self._tracing_backend is None:

[commit-bot: I haz the power, 2013-02-04 20:20:13]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/12042015/21001

[commit-bot: I haz the power, 2013-02-04 21:25:12]

Sorry for I got bad news for ya.
Compile failed with a clobber build on win.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win&number...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2013-02-05 09:53:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/bulach@chromium.org/12042015/21001

[commit-bot: I haz the power, 2013-02-05 10:59:48]

Change committed as 180681