Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(147)

Side by Side Diff: chrome/browser/ui/passwords/manage_passwords_bubble_model_unittest.cc

Issue 2251263003: Re-write many calls to WrapUnique() with MakeUnique() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/ui/passwords/manage_passwords_bubble_model.h" 5 #include "chrome/browser/ui/passwords/manage_passwords_bubble_model.h"
6 6
7 #include <utility> 7 #include <utility>
8 #include <vector> 8 #include <vector>
9 9
10 #include "base/memory/ptr_util.h" 10 #include "base/memory/ptr_util.h"
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 void set_smartlock_enabled(bool smartlock_enabled) { 77 void set_smartlock_enabled(bool smartlock_enabled) {
78 smartlock_enabled_ = smartlock_enabled; 78 smartlock_enabled_ = smartlock_enabled;
79 } 79 }
80 80
81 private: 81 private:
82 bool smartlock_enabled_; 82 bool smartlock_enabled_;
83 }; 83 };
84 84
85 std::unique_ptr<KeyedService> TestingSyncFactoryFunction( 85 std::unique_ptr<KeyedService> TestingSyncFactoryFunction(
86 content::BrowserContext* context) { 86 content::BrowserContext* context) {
87 return base::WrapUnique(new TestSyncService(static_cast<Profile*>(context))); 87 return base::MakeUnique<TestSyncService>(static_cast<Profile*>(context));
88 } 88 }
89 89
90 } // namespace 90 } // namespace
91 91
92 class ManagePasswordsBubbleModelTest : public ::testing::Test { 92 class ManagePasswordsBubbleModelTest : public ::testing::Test {
93 public: 93 public:
94 ManagePasswordsBubbleModelTest() : field_trials_(nullptr) {} 94 ManagePasswordsBubbleModelTest() : field_trials_(nullptr) {}
95 ~ManagePasswordsBubbleModelTest() override = default; 95 ~ManagePasswordsBubbleModelTest() override = default;
96 96
97 void SetUp() override { 97 void SetUp() override {
(...skipping 463 matching lines...) Expand 10 before | Expand all | Expand 10 after
561 ManageLinkTarget::SETTINGS_PAGE}, 561 ManageLinkTarget::SETTINGS_PAGE},
562 {nullptr, SmartLockStatus::DISABLE, ManageLinkTarget::SETTINGS_PAGE}, 562 {nullptr, SmartLockStatus::DISABLE, ManageLinkTarget::SETTINGS_PAGE},
563 {"Default", SmartLockStatus::DISABLE, ManageLinkTarget::SETTINGS_PAGE}, 563 {"Default", SmartLockStatus::DISABLE, ManageLinkTarget::SETTINGS_PAGE},
564 }; 564 };
565 565
566 } // namespace 566 } // namespace
567 567
568 INSTANTIATE_TEST_CASE_P(Default, 568 INSTANTIATE_TEST_CASE_P(Default,
569 ManagePasswordsBubbleModelManageLinkTest, 569 ManagePasswordsBubbleModelManageLinkTest,
570 ::testing::ValuesIn(kManageLinkTestCases)); 570 ::testing::ValuesIn(kManageLinkTestCases));
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698