mac: Expose keychain access frequency to Telemetry.

tools/perf/measurements/page_cycler_unittest.py

 # Copyright 2013 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 sys
 import unittest
 
+from metrics import keychain_metric
 from telemetry.core import browser_options
 from telemetry.results import page_test_results
 from telemetry.unittest_util import simple_mock
 from telemetry.user_story import user_story_runner
 
 from measurements import page_cycler
 
 
 # Allow testing protected members in the unit test.
 # pylint: disable=W0212
@@ skipping 25 matching lines @@
 
 
 class FakeTab(object):
   """Used to mock a browser tab."""
   def __init__(self):
     self.clear_cache_calls = 0
     self.navigated_urls = []
   def ClearCache(self, force=False):
     assert force
     self.clear_cache_calls += 1
-  def EvaluateJavaScript(self, _):
+  def EvaluateJavaScript(self, script):
+    # If the page cycler invokes javascript to measure the number of keychain
+    # accesses, return a valid JSON dictionary.
+    get_keychain_script = 'statsCollectionController.getBrowserHistogram' \
+        '("{0}")'.format(keychain_metric.KeychainMetric.HISTOGRAM_NAME)
+    if script == get_keychain_script:

[jeremy, 2014/12/09 14:57:16]

# Fake data for keychain metric.
if keychain_histogram_name in script:
  return ....

[erikchen, 2014/12/13 02:39:58]

Done.

+      return '{{ "{0}" : 0 }}'.format(
+          keychain_metric.KeychainMetric.HISTOGRAM_NAME)
+
     return 1
   def Navigate(self, url):
     self.navigated_urls.append(url)
   def WaitForJavaScriptExpression(self, _, __):
     pass
   @property
   def browser(self):
     return FakeBrowser()
 
 class FakeBrowser(object):
@@ skipping 146 matching lines @@
 
     for i in range(2):
       for page in pages:
         results = page_test_results.PageTestResults()
         results.WillRunPage(page)
         cycler.WillNavigateToPage(page, tab)
         cycler.ValidateAndMeasurePage(page, tab, results)
         results.DidRunPage(page)
 
         values = results.all_page_specific_values
-        self.assertEqual(4, len(values))
+
+        # On Mac, there is an additional measurement: the number of keychain
+        # accesses.
+        value_count = 4
+        if sys.platform =='darwin':

[jeremy, 2014/12/09 14:57:16]

space after ==

[erikchen, 2014/12/13 02:39:58]

Done.

+          value_count = value_count + 1

[jeremy, 2014/12/09 14:57:16]

value_count += 1

[erikchen, 2014/12/13 02:39:58]

Done.

+        self.assertEqual(value_count, len(values))
 
         self.assertEqual(values[0].page, page)
         chart_name = 'cold_times' if i == 0 else 'warm_times'
         self.assertEqual(values[0].name, '%s.page_load_time' % chart_name)
         self.assertEqual(values[0].units, 'ms')
 
-        for value, expected in zip(values[1:], ['gpu', 'renderer', 'browser']):
+        for value, expected in zip(values[1:4], ['gpu', 'renderer', 'browser']):

[jeremy, 2014/12/09 14:57:16]

I don't like the hardcoded 4 here, how about:

foo = ['gpu', ...]
for ... in zip(values[1:len(foo)+1], foo)

[erikchen, 2014/12/13 02:39:58]

Done.

           self.assertEqual(value.page, page)
           self.assertEqual(value.name,
                            'cpu_utilization.cpu_utilization_%s' % expected)
           self.assertEqual(value.units, '%')
 
         cycler.DidNavigateToPage(page, tab)
 
   def testLegacyPagesAvoidCrossRenderNavigation(self):
     # For legacy page cyclers with file URLs, verify that WillNavigateToPage
     # does an initial navigate to avoid paying for a cross-renderer navigation.
     cycler = self.SetUpCycler(setup_memory_module=True)
     pages = [FakePage('file://fakepage1.com'), FakePage('file://fakepage2.com')]
     tab = FakeTab()
 
     self.assertEqual([], tab.navigated_urls)
     for page in pages * 2:
       cycler.WillNavigateToPage(page, tab)
       self.assertEqual(
           ['http://fakeserver:99999/nonexistent.html'], tab.navigated_urls)