Stop passing raw pointers to base::Value API in c/b/ui

chrome/browser/ui/webui/chromeos/login/app_launch_splash_screen_handler.cc

 // Copyright 2013 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/ui/webui/chromeos/login/app_launch_splash_screen_handler.h"
 

[jdoerrie, 2017/04/13 09:08:51]

#include <utility>

#include "base/memory/ptr_util.h"
#include "base/values.h"

[vabr (Chromium), 2017/04/13 12:03:14]

Done.

 #include "chrome/browser/chromeos/app_mode/kiosk_app_manager.h"
 #include "chrome/browser/chromeos/login/oobe_screen.h"
 #include "chrome/browser/chromeos/login/screens/network_error.h"
 #include "chrome/browser/ui/webui/chromeos/login/oobe_ui.h"
 #include "chrome/grit/chrome_unscaled_resources.h"
 #include "chrome/grit/chromium_strings.h"
 #include "chrome/grit/generated_resources.h"
 #include "chromeos/network/network_state.h"
 #include "chromeos/network/network_state_handler.h"
 #include "components/login/localized_values_builder.h"
@@ skipping 61 matching lines @@
   app_id_ = app_id;
   if (!page_is_ready()) {
     show_on_init_ = true;
     return;
   }
 
   base::DictionaryValue data;
   data.SetBoolean("shortcutEnabled",
                   !KioskAppManager::Get()->GetDisableBailoutShortcut());
 
-  // |data| will take ownership of |app_info|.
-  base::DictionaryValue *app_info = new base::DictionaryValue();
-  PopulateAppInfo(app_info);
-  data.Set("appInfo", app_info);
+  auto app_info = base::MakeUnique<base::DictionaryValue>();
+  PopulateAppInfo(app_info.get());
+  data.Set("appInfo", std::move(app_info));
 
   SetLaunchText(l10n_util::GetStringUTF8(GetProgressMessageFromState(state_)));
   ShowScreenWithData(kScreenId, &data);
 }
 
 void AppLaunchSplashScreenHandler::RegisterMessages() {
   AddCallback("configureNetwork",
               &AppLaunchSplashScreenHandler::HandleConfigureNetwork);
   AddCallback("cancelAppLaunch",
               &AppLaunchSplashScreenHandler::HandleCancelAppLaunch);
@@ skipping 171 matching lines @@
   DCHECK(online_state_);
   if (delegate_ && online_state_) {
     network_config_requested_ = false;
     network_config_done_ = true;
     delegate_->OnNetworkConfigRequested(false);
     Show(app_id_);
   }
 }
 
 }  // namespace chromeos