Remove many instances of CreateBooleanValue.

Remove many instances of CreateBooleanValue.

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

[Evan Stade, 2014-07-15 21:56:38]

zelidrag:
  chrome/browser/chromeos/*
thestig:
  the rest

[Lei Zhang, 2014-07-15 22:00:14]

lgtm

[zel, 2014-07-15 22:31:43]

lgtm

[Evan Stade, 2014-07-15 22:32:41]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-07-15 22:35:20]

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

[commit-bot: I haz the power, 2014-07-16 06:07:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-07-16 06:07:03]

Failed to apply patch for
chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file
chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
  Hunk #1 FAILED at 275.
  1 out of 2 hunks FAILED -- saving rejects to file
chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc.rej

Patch:      
chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
Index:
chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
diff --git
a/chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
b/chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
index
59eae7d2d79241f01bb910c8fe874eade7beff4d..0656b9738ff6e210b519edf699bb7c322daefd62
100644
--- a/chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
+++ b/chrome/browser/ui/webui/performance_monitor/performance_monitor_handler.cc
@@ -275,9 +275,9 @@ void DoGetEvents(base::ListValue* results,
         // The property 'eventType' is set in HandleGetEvents as part of the
         // entire result set, so we don't need to include this here in the
         // event.
-        if (data.key() == "eventType")
+        if (data.key() == "eventType") {
           continue;
-        else if (data.key() == "time") {
+        } else if (data.key() == "time") {
           // The property 'time' is also used computationally, but must be
           // converted to JS-style time.
           double time = 0.0;
@@ -451,10 +451,10 @@ void PerformanceMonitorHandler::HandleGetFlagEnabled(
     const base::ListValue* args) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   CHECK_EQ(0u, args->GetSize());
-  scoped_ptr<base::Value> value(base::Value::CreateBooleanValue(
+  base::FundamentalValue value(
       CommandLine::ForCurrentProcess()->HasSwitch(
-          switches::kPerformanceMonitorGathering)));
-  ReturnResults("PerformanceMonitor.getFlagEnabledCallback", value.get());
+          switches::kPerformanceMonitorGathering));
+  ReturnResults("PerformanceMonitor.getFlagEnabledCallback", &value);
 }
 
 void PerformanceMonitorHandler::HandleGetAggregationTypes(

[Evan Stade, 2014-07-16 22:31:17]

The CQ bit was checked by estade@chromium.org

[commit-bot: I haz the power, 2014-07-16 22:34:58]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/estade@chromium.org/399493002/20001

[commit-bot: I haz the power, 2014-07-17 06:28:13]

Change committed as 283659