| Index: chrome/browser/chromeos/policy/device_status_collector_browsertest.cc
|
| diff --git a/chrome/browser/chromeos/policy/device_status_collector_browsertest.cc b/chrome/browser/chromeos/policy/device_status_collector_browsertest.cc
|
| index 700791d8e454dc110519cc9a8d6e4975918bd63f..fb811a20eba0570cec1ff7879d28662be19470e1 100644
|
| --- a/chrome/browser/chromeos/policy/device_status_collector_browsertest.cc
|
| +++ b/chrome/browser/chromeos/policy/device_status_collector_browsertest.cc
|
| @@ -6,6 +6,8 @@
|
|
|
| #include <stddef.h>
|
| #include <stdint.h>
|
| +
|
| +#include <memory>
|
| #include <string>
|
| #include <utility>
|
| #include <vector>
|
| @@ -14,7 +16,7 @@
|
| #include "base/environment.h"
|
| #include "base/logging.h"
|
| #include "base/macros.h"
|
| -#include "base/memory/scoped_ptr.h"
|
| +#include "base/memory/ptr_util.h"
|
| #include "base/message_loop/message_loop.h"
|
| #include "base/run_loop.h"
|
| #include "base/strings/stringprintf.h"
|
| @@ -71,7 +73,7 @@ const char kKioskAppId[] = "kiosk_app_id";
|
| const char kExternalMountPoint[] = "/a/b/c";
|
| const char kPublicAccountId[] = "public_user@localhost";
|
|
|
| -scoped_ptr<content::Geoposition> mock_position_to_return_next;
|
| +std::unique_ptr<content::Geoposition> mock_position_to_return_next;
|
|
|
| void SetMockPositionToReturnNext(const content::Geoposition &position) {
|
| mock_position_to_return_next.reset(new content::Geoposition(position));
|
| @@ -88,7 +90,7 @@ void MockPositionUpdateRequester(
|
| // harness, the callback is invoked synchronously upon request, leading to a
|
| // request-callback loop. The loop is broken by returning the mock position
|
| // only once.
|
| - scoped_ptr<content::Geoposition> position(
|
| + std::unique_ptr<content::Geoposition> position(
|
| mock_position_to_return_next.release());
|
| callback.Run(*position);
|
| }
|
| @@ -134,15 +136,15 @@ class TestingDeviceStatusCollector : public policy::DeviceStatusCollector {
|
| baseline_offset_periods_ = 0;
|
| }
|
|
|
| - void set_kiosk_account(scoped_ptr<policy::DeviceLocalAccount> account) {
|
| + void set_kiosk_account(std::unique_ptr<policy::DeviceLocalAccount> account) {
|
| kiosk_account_ = std::move(account);
|
| }
|
|
|
| - scoped_ptr<policy::DeviceLocalAccount>
|
| - GetAutoLaunchedKioskSessionInfo() override {
|
| + std::unique_ptr<policy::DeviceLocalAccount> GetAutoLaunchedKioskSessionInfo()
|
| + override {
|
| if (kiosk_account_)
|
| - return make_scoped_ptr(new policy::DeviceLocalAccount(*kiosk_account_));
|
| - return scoped_ptr<policy::DeviceLocalAccount>();
|
| + return base::WrapUnique(new policy::DeviceLocalAccount(*kiosk_account_));
|
| + return std::unique_ptr<policy::DeviceLocalAccount>();
|
| }
|
|
|
| std::string GetAppVersion(const std::string& app_id) override {
|
| @@ -177,7 +179,7 @@ class TestingDeviceStatusCollector : public policy::DeviceStatusCollector {
|
| // The number of simulated periods since the baseline time.
|
| int baseline_offset_periods_;
|
|
|
| - scoped_ptr<policy::DeviceLocalAccount> kiosk_account_;
|
| + std::unique_ptr<policy::DeviceLocalAccount> kiosk_account_;
|
| };
|
|
|
| // Return the total number of active milliseconds contained in a device
|
| @@ -261,12 +263,13 @@ class DeviceStatusCollectorTest : public testing::Test {
|
| std::string() /* kiosk_app_update_url */) {
|
| // Run this test with a well-known timezone so that Time::LocalMidnight()
|
| // returns the same values on all machines.
|
| - scoped_ptr<base::Environment> env(base::Environment::Create());
|
| + std::unique_ptr<base::Environment> env(base::Environment::Create());
|
| env->SetVar("TZ", "UTC");
|
|
|
| // Initialize our mock mounted disk volumes.
|
| - scoped_ptr<chromeos::disks::MockDiskMountManager> mock_disk_mount_manager =
|
| - make_scoped_ptr(new chromeos::disks::MockDiskMountManager());
|
| + std::unique_ptr<chromeos::disks::MockDiskMountManager>
|
| + mock_disk_mount_manager =
|
| + base::WrapUnique(new chromeos::disks::MockDiskMountManager());
|
| AddMountPoint("/mount/volume1");
|
| AddMountPoint("/mount/volume2");
|
| EXPECT_CALL(*mock_disk_mount_manager, mount_points())
|
| @@ -409,11 +412,11 @@ class DeviceStatusCollectorTest : public testing::Test {
|
| chromeos::ScopedTestDeviceSettingsService test_device_settings_service_;
|
| chromeos::ScopedTestCrosSettings test_cros_settings_;
|
| chromeos::ScopedCrosSettingsTestHelper settings_helper_;
|
| - scoped_ptr<chromeos::FakeOwnerSettingsService> owner_settings_service_;
|
| + std::unique_ptr<chromeos::FakeOwnerSettingsService> owner_settings_service_;
|
| chromeos::MockUserManager* user_manager_;
|
| chromeos::ScopedUserManagerEnabler user_manager_enabler_;
|
| em::DeviceStatusReportRequest status_;
|
| - scoped_ptr<TestingDeviceStatusCollector> status_collector_;
|
| + std::unique_ptr<TestingDeviceStatusCollector> status_collector_;
|
| const policy::DeviceLocalAccount fake_device_local_account_;
|
| };
|
|
|
| @@ -986,7 +989,7 @@ TEST_F(DeviceStatusCollectorTest, NoSessionStatusIfNotKioskMode) {
|
| TEST_F(DeviceStatusCollectorTest, NoSessionStatusIfSessionReportingDisabled) {
|
| // Should not report session status if session status reporting is disabled.
|
| settings_helper_.SetBoolean(chromeos::kReportDeviceSessionStatus, false);
|
| - status_collector_->set_kiosk_account(make_scoped_ptr(
|
| + status_collector_->set_kiosk_account(base::WrapUnique(
|
| new policy::DeviceLocalAccount(fake_device_local_account_)));
|
| // Set up a device-local account for single-app kiosk mode.
|
| MockRunningKioskApp(fake_device_local_account_);
|
| @@ -997,7 +1000,7 @@ TEST_F(DeviceStatusCollectorTest, NoSessionStatusIfSessionReportingDisabled) {
|
|
|
| TEST_F(DeviceStatusCollectorTest, ReportSessionStatus) {
|
| settings_helper_.SetBoolean(chromeos::kReportDeviceSessionStatus, true);
|
| - status_collector_->set_kiosk_account(make_scoped_ptr(
|
| + status_collector_->set_kiosk_account(base::WrapUnique(
|
| new policy::DeviceLocalAccount(fake_device_local_account_)));
|
|
|
| // Set up a device-local account for single-app kiosk mode.
|
| @@ -1284,7 +1287,7 @@ TEST_F(DeviceStatusCollectorNetworkInterfacesTest, NoNetworkStateIfNotKiosk) {
|
|
|
| TEST_F(DeviceStatusCollectorNetworkInterfacesTest, NetworkInterfaces) {
|
| // Mock that we are in kiosk mode so we report network state.
|
| - status_collector_->set_kiosk_account(make_scoped_ptr(
|
| + status_collector_->set_kiosk_account(base::WrapUnique(
|
| new policy::DeviceLocalAccount(fake_device_local_account_)));
|
|
|
| // Interfaces should be reported by default.
|
|
|