Remove many instances of CreateBooleanValue.

chrome/browser/guest_view/web_view/web_view_permission_helper.cc

 // Copyright 2014 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/guest_view/web_view/web_view_permission_helper.h"
 
 #include "chrome/browser/content_settings/tab_specific_content_settings.h"
 #include "chrome/browser/geolocation/geolocation_permission_context.h"
 #include "chrome/browser/geolocation/geolocation_permission_context_factory.h"
 #include "chrome/browser/guest_view/web_view/web_view_constants.h"
@@ skipping 245 matching lines @@
   }
 }
 
 #endif  // defined(ENABLE_PLUGINS)
 
 void WebViewPermissionHelper::RequestMediaAccessPermission(
     content::WebContents* source,
     const content::MediaStreamRequest& request,
     const content::MediaResponseCallback& callback) {
   base::DictionaryValue request_info;
-  request_info.Set(guestview::kUrl,
-                   new base::StringValue(request.security_origin.spec()));
+  request_info.SetString(guestview::kUrl, request.security_origin.spec());
   RequestPermission(WEB_VIEW_PERMISSION_TYPE_MEDIA,
                     request_info,
                     base::Bind(&WebViewPermissionHelper::
                                OnMediaPermissionResponse,
                                base::Unretained(this),
                                request,
                                callback),
                     false /* allowed_by_default */);
 }
 
@@ skipping 17 matching lines @@
                                    request,
                                    callback);
 }
 
 void WebViewPermissionHelper::CanDownload(
     content::RenderViewHost* render_view_host,
     const GURL& url,
     const std::string& request_method,
     const base::Callback<void(bool)>& callback) {
   base::DictionaryValue request_info;
-  request_info.Set(guestview::kUrl, new base::StringValue(url.spec()));
+  request_info.SetString(guestview::kUrl, url.spec());
   RequestPermission(
       WEB_VIEW_PERMISSION_TYPE_DOWNLOAD,
       request_info,
       base::Bind(&WebViewPermissionHelper::OnDownloadPermissionResponse,
                  base::Unretained(this),
                  callback),
       false /* allowed_by_default */);
 }
 
 void WebViewPermissionHelper::OnDownloadPermissionResponse(
     const base::Callback<void(bool)>& callback,
     bool allow,
     const std::string& user_input) {
   callback.Run(allow && web_view_guest_->attached());
 }
 
 void WebViewPermissionHelper::RequestPointerLockPermission(
     bool user_gesture,
     bool last_unlocked_by_target,
     const base::Callback<void(bool)>& callback) {
   base::DictionaryValue request_info;
-  request_info.Set(guestview::kUserGesture,
-                   base::Value::CreateBooleanValue(user_gesture));
-  request_info.Set(webview::kLastUnlockedBySelf,
-                   base::Value::CreateBooleanValue(last_unlocked_by_target));
-  request_info.Set(guestview::kUrl,
-                   new base::StringValue(
-                       web_contents()->GetLastCommittedURL().spec()));
+  request_info.SetBoolean(guestview::kUserGesture, user_gesture);
+  request_info.SetBoolean(webview::kLastUnlockedBySelf,
+                          last_unlocked_by_target);
+  request_info.SetString(guestview::kUrl,
+                         web_contents()->GetLastCommittedURL().spec());
 
   RequestPermission(
       WEB_VIEW_PERMISSION_TYPE_POINTER_LOCK,
       request_info,
       base::Bind(
           &WebViewPermissionHelper::OnPointerLockPermissionResponse,
           base::Unretained(this),
           callback),
       false /* allowed_by_default */);
 }
 
 void WebViewPermissionHelper::OnPointerLockPermissionResponse(
     const base::Callback<void(bool)>& callback,
     bool allow,
     const std::string& user_input) {
   callback.Run(allow && web_view_guest_->attached());
 }
 
 void WebViewPermissionHelper::RequestGeolocationPermission(
     int bridge_id,
     const GURL& requesting_frame,
     bool user_gesture,
     const base::Callback<void(bool)>& callback) {
   base::DictionaryValue request_info;
-  request_info.Set(guestview::kUrl,
-                   new base::StringValue(requesting_frame.spec()));
-  request_info.Set(guestview::kUserGesture,
-                   base::Value::CreateBooleanValue(user_gesture));
+  request_info.SetString(guestview::kUrl, requesting_frame.spec());
+  request_info.SetBoolean(guestview::kUserGesture, user_gesture);
 
   // It is safe to hold an unretained pointer to WebViewPermissionHelper because
   // this callback is called from WebViewPermissionHelper::SetPermission.
   const PermissionResponseCallback permission_callback =
       base::Bind(
           &WebViewPermissionHelper::OnGeolocationPermissionResponse,
           base::Unretained(this),
           bridge_id,
           user_gesture,
           callback);
@@ skipping 57 matching lines @@
   int request_id = bridge_itr->second;
   bridge_id_to_request_id_map_.erase(bridge_itr);
   return request_id;
 }
 
 void WebViewPermissionHelper::RequestFileSystemPermission(
     const GURL& url,
     bool allowed_by_default,
     const base::Callback<void(bool)>& callback) {
   base::DictionaryValue request_info;
-  request_info.Set(guestview::kUrl, new base::StringValue(url.spec()));
+  request_info.SetString(guestview::kUrl, url.spec());
   RequestPermission(
       WEB_VIEW_PERMISSION_TYPE_FILESYSTEM,
       request_info,
       base::Bind(
           &WebViewPermissionHelper::OnFileSystemPermissionResponse,
           base::Unretained(this),
           callback),
       allowed_by_default);
 }
 
@@ skipping 145 matching lines @@
     const PermissionResponseCallback& callback,
     WebViewPermissionType permission_type,
     bool allowed_by_default)
     : callback(callback),
       permission_type(permission_type),
       allowed_by_default(allowed_by_default) {
 }
 
 WebViewPermissionHelper::PermissionResponseInfo::~PermissionResponseInfo() {
 }