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

Side by Side Diff: chrome/browser/media/router/mojo/media_router_mojo_impl_unittest.cc

Issue 2334613003: Re-write many calls to WrapUnique() with MakeUnique() (Closed)
Patch Set: Changes from review by sky 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include <memory> 8 #include <memory>
9 #include <string> 9 #include <string>
10 #include <utility> 10 #include <utility>
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
164 public: 164 public:
165 explicit TestProcessManager(content::BrowserContext* context) 165 explicit TestProcessManager(content::BrowserContext* context)
166 : extensions::ProcessManager( 166 : extensions::ProcessManager(
167 context, 167 context,
168 context, 168 context,
169 extensions::ExtensionRegistry::Get(context)) {} 169 extensions::ExtensionRegistry::Get(context)) {}
170 ~TestProcessManager() override {} 170 ~TestProcessManager() override {}
171 171
172 static std::unique_ptr<KeyedService> Create( 172 static std::unique_ptr<KeyedService> Create(
173 content::BrowserContext* context) { 173 content::BrowserContext* context) {
174 return base::WrapUnique(new TestProcessManager(context)); 174 return base::MakeUnique<TestProcessManager>(context);
175 } 175 }
176 176
177 MOCK_METHOD1(IsEventPageSuspended, bool(const std::string& ext_id)); 177 MOCK_METHOD1(IsEventPageSuspended, bool(const std::string& ext_id));
178 178
179 MOCK_METHOD2(WakeEventPage, 179 MOCK_METHOD2(WakeEventPage,
180 bool(const std::string& extension_id, 180 bool(const std::string& extension_id,
181 const base::Callback<void(bool)>& callback)); 181 const base::Callback<void(bool)>& callback));
182 182
183 private: 183 private:
184 DISALLOW_COPY_AND_ASSIGN(TestProcessManager); 184 DISALLOW_COPY_AND_ASSIGN(TestProcessManager);
(...skipping 1375 matching lines...) Expand 10 before | Expand all | Expand 10 after
1560 EXPECT_CALL(mock_media_route_provider_, 1560 EXPECT_CALL(mock_media_route_provider_,
1561 UpdateMediaSinks(MediaSourceForDesktop().id())) 1561 UpdateMediaSinks(MediaSourceForDesktop().id()))
1562 .WillOnce(InvokeWithoutArgs([&run_loop2]() { 1562 .WillOnce(InvokeWithoutArgs([&run_loop2]() {
1563 run_loop2.Quit(); 1563 run_loop2.Quit();
1564 })); 1564 }));
1565 1565
1566 run_loop2.Run(); 1566 run_loop2.Run();
1567 } 1567 }
1568 1568
1569 } // namespace media_router 1569 } // namespace media_router
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698