Record API fix adding FirstRun, and allowing repeat-count in capture (plus some browsertests along …

chrome/browser/extensions/extension_record_api_test.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/extensions/extension_record_api.h"
 
 #include <string>
 
 #include "base/memory/ref_counted.h"
 #include "base/file_path.h"
@@ skipping 19 matching lines @@
 namespace utils = extension_function_test_utils;
 
 namespace {
 
 // Dummy content for mock stats file.
 const std::string kTestStatistics = "Sample Stat 1\nSample Stat 2\n";
 
 // Standard capture parameters, with a mix of good and bad URLs, and
 // a hole for filling in the user data dir.
 const char kCaptureArgs1[] =
-    "[[\"URL 1\", \"URL 2(bad)\", \"URL 3\", \"URL 4(bad)\"], \"%s\"]";
+    "[[\"URL 1\", \"URL 2(bad)\", \"URL 3\", \"URL 4(bad)\"]"
+    ", \"%s\", 2]";
 
 // Standard playback parameters, with the same mix of good and bad URLs
 // as the capture parameters, a hole for filling in the user data dir, and
 // a mocked-up extension path and repeat count (which are used only to
 // verify that they made it into the CommandLine, since extension loading
 // and repeat-counting are hard to emulate in the test ProcessStrategy.
 const char kPlaybackArgs1[] =
     "[[\"URL 1\", \"URL 2(bad)\", \"URL 3\", \"URL 4(bad)\"], \"%s\""
-    ", {\"extensionPath\": \"MockExtension\", \"repeatCount\": 2}]";
+    ", 2, {\"extensionPath\": \"MockExtension\"}]";
 
 // Use this as the value of FilePath switches (e.g. user-data-dir) that
 // should be replaced by the record methods.
 const FilePath::CharType kDummyDirName[] = FILE_PATH_LITERAL("ReplaceMe");
 
 // Use this as the filename for a mock "cache" file in the user-data-dir.
 const FilePath::CharType kMockCacheFile[] = FILE_PATH_LITERAL("MockCache");
 
 // Prefix for temporary user data directory
 const FilePath::CharType kUserDataDirPrefix[]
@@ skipping 278 matching lines @@
       command_line.GetSwitchValueASCII(switches::kVisitURLsCount).compare("2")
       == 0);
 
    // Check for return value with proper stats.
   EXPECT_EQ(kTestStatistics, utils::GetString(result.get(), kStatsKey));
 
   ListValue* errors = NULL;
   EXPECT_TRUE(result->GetList(kErrorsKey, &errors));
   EXPECT_TRUE(VerifyURLHandling(errors, strategy));
 }