Use experimentation framework parameters for location sensitivity

chrome/browser/extensions/api/metrics_private/metrics_private_api.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/api/metrics_private/metrics_private_api.h"
 
 #include <algorithm>
 
 #include "base/metrics/field_trial.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/pref_service.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/common/extensions/api/metrics_private.h"
 #include "chrome/common/extensions/extension.h"
+#include "chrome/common/metrics/variations/variations_associated_data.h"
 #include "chrome/common/pref_names.h"
 #include "content/public/browser/user_metrics.h"
 
 #if defined(OS_CHROMEOS)
 #include "chrome/browser/chromeos/settings/cros_settings.h"
 #endif  // OS_CHROMEOS
 
 namespace extensions {
 
 namespace GetIsCrashReportingEnabled =
@@ skipping 44 matching lines @@
 }
 
 bool MetricsPrivateGetFieldTrialFunction::RunImpl() {
   std::string name;
   EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &name));
 
   SetResult(new base::StringValue(base::FieldTrialList::FindFullName(name)));
   return true;
 }
 
+bool MetricsPrivateGetVariationParamsFunction::RunImpl() {
+  std::string name;
+  EXTENSION_FUNCTION_VALIDATE(args_->GetString(0, &name));

[not at google - send to devlin, 2013/08/09 18:09:22]

prefer using generated code here even if it's simple.

[vadimt, 2013/08/09 19:30:19]

Done.

+
+  std::map<std::string, std::string> params;
+  if (chrome_variations::GetVariationParams(name, &params)) {

[not at google - send to devlin, 2013/08/09 18:09:22]

early return here, and set an error message on failure:

if (!chrome_variasions::Ge(..)) {
  error_ = ???
  return false;
}

...
return true.

[vadimt, 2013/08/09 19:30:19]

Done.

+    scoped_ptr<DictionaryValue> settings(new DictionaryValue());
+
+    for (std::map<std::string, std::string>::const_iterator it = params.begin();
+         it != params.end();
+         ++it ) {
+      settings->SetWithoutPathExpansion(

[not at google - send to devlin, 2013/08/09 18:09:22]

SetStringWithoutPathExpansion

[vadimt, 2013/08/09 19:30:19]

Done.

+          it->first, new base::StringValue(it->second));
+    }
+
+    SetResult(settings.release());
+    return true;
+  } else {
+    return false;
+  }
+}
+
 bool MetricsPrivateRecordUserActionFunction::RunImpl() {
   scoped_ptr<RecordUserAction::Params> params(
       RecordUserAction::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
 
   content::RecordComputedAction(params->name);
   return true;
 }
 
 bool MetricsHistogramHelperFunction::GetNameAndSample(std::string* name,
@@ skipping 102 matching lines @@
 bool MetricsPrivateRecordLongTimeFunction::RunImpl() {
   scoped_ptr<RecordLongTime::Params> params(
       RecordLongTime::Params::Create(*args_));
   EXTENSION_FUNCTION_VALIDATE(params.get());
   static const int kOneHourMs = 60 * 60 * 1000;
   return RecordValue(params->metric_name, base::HISTOGRAM,
                      1, kOneHourMs, 50, params->value);
 }
 
 } // namespace extensions