ExtensionFunction: don't pass ownership of base::Value by raw pointer.

extensions/browser/api/idle/idle_manager.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 "extensions/browser/api/idle/idle_manager.h"
 
 #include <utility>
 
+#include "base/memory/ptr_util.h"
 #include "base/stl_util.h"
 #include "content/public/browser/browser_context.h"
 #include "extensions/browser/api/idle/idle_api_constants.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/common/api/idle.h"
 #include "extensions/common/extension.h"
 
 namespace keys = extensions::idle_api_constants;
 namespace idle = extensions::api::idle;
@@ skipping 154 matching lines @@
   DCHECK(thread_checker_.CalledOnValidThread());
   idle_time_provider_->CalculateIdleState(threshold, notify);
 }
 
 void IdleManager::SetThreshold(const std::string& extension_id, int threshold) {
   DCHECK(thread_checker_.CalledOnValidThread());
   GetMonitor(extension_id)->threshold = threshold;
 }
 
 // static
-base::StringValue* IdleManager::CreateIdleValue(ui::IdleState idle_state) {
+std::unique_ptr<base::StringValue> IdleManager::CreateIdleValue(
+    ui::IdleState idle_state) {
   const char* description;
 
   if (idle_state == ui::IDLE_STATE_ACTIVE) {
     description = keys::kStateActive;
   } else if (idle_state == ui::IDLE_STATE_IDLE) {
     description = keys::kStateIdle;
   } else {
     description = keys::kStateLocked;
   }
 
-  return new base::StringValue(description);
+  return base::MakeUnique<base::StringValue>(description);
 }
 
 void IdleManager::SetEventDelegateForTest(
     std::unique_ptr<EventDelegate> event_delegate) {
   DCHECK(thread_checker_.CalledOnValidThread());
   event_delegate_ = std::move(event_delegate);
 }
 
 void IdleManager::SetIdleTimeProviderForTest(
     std::unique_ptr<IdleTimeProvider> idle_time_provider) {
@@ skipping 49 matching lines @@
       event_delegate_->OnStateChanged(it->first, new_state);
     monitor.last_state = new_state;
     listener_count += monitor.listeners;
   }
 
   if (listener_count == 0)
     StopPolling();
 }
 
 }  // namespace extensions