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 unittest
 
 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
@@ skipping 85 matching lines @@
 
   def SetUpCycler(self, page_repeat=1, pageset_repeat=10, cold_load_percent=50,
                   record_v8_object_stats=False, report_speed_index=False,
                   setup_memory_module=False):
     cycler = page_cycler.PageCycler(
         page_repeat = page_repeat,
         pageset_repeat = pageset_repeat,
         cold_load_percent = cold_load_percent,
         record_v8_object_stats = record_v8_object_stats,
         report_speed_index = report_speed_index)
+    cycler.SetInUnitTest(True)

[dtu, 2014/12/05 01:12:54]

Why can't we have this enabled in unit tests as well?

[erikchen, 2014/12/08 23:41:25]

I don't understand your comment. My intention is for all unit tests that use a
PageCycler to call this method. This appears to be the best place to do so. I
don't know of any unit tests that I've missed. 

[dtu, 2014/12/09 00:25:03]

Sorry, I mean that I think it's generally dangerous and messy to have a custom
code path just for unit tests. Why are you disabling the metric for unit tests?

[erikchen, 2014/12/09 01:26:03]

Good point. I've removed the method "SetInUnitTest" and its corresponding
member. I've introduced platform-dependent code into the unit test to account
for the different behavior of the class on Mac vs. non-Mac.

     options = browser_options.BrowserFinderOptions()
     options.browser_options.platform = FakePlatform()
     parser = options.CreateParser()
     user_story_runner.AddCommandLineArgs(parser)
     args = ['--page-repeat=%i' % page_repeat,
             '--pageset-repeat=%i' % pageset_repeat]
     parser.parse_args(args)
     user_story_runner.ProcessCommandLineArgs(parser, options)
     cycler.CustomizeBrowserOptions(options.browser_options)
 
@@ skipping 120 matching lines @@
     # 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)