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

Side by Side Diff: chrome/renderer/chrome_render_thread_observer.cc

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 7 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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/renderer/chrome_render_thread_observer.h" 5 #include "chrome/renderer/chrome_render_thread_observer.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <limits> 9 #include <limits>
10 #include <memory> 10 #include <memory>
(...skipping 216 matching lines...) Expand 10 before | Expand all | Expand 10 after
227 int workers_to_go_; 227 int workers_to_go_;
228 base::WeakPtr<ChromeRenderThreadObserver> observer_; 228 base::WeakPtr<ChromeRenderThreadObserver> observer_;
229 229
230 base::WeakPtrFactory<ResourceUsageReporterImpl> weak_factory_; 230 base::WeakPtrFactory<ResourceUsageReporterImpl> weak_factory_;
231 231
232 DISALLOW_COPY_AND_ASSIGN(ResourceUsageReporterImpl); 232 DISALLOW_COPY_AND_ASSIGN(ResourceUsageReporterImpl);
233 }; 233 };
234 234
235 void CreateResourceUsageReporter( 235 void CreateResourceUsageReporter(
236 base::WeakPtr<ChromeRenderThreadObserver> observer, 236 base::WeakPtr<ChromeRenderThreadObserver> observer,
237 mojo::InterfaceRequest<chrome::mojom::ResourceUsageReporter> request) { 237 const service_manager::BindSourceInfo& source_info,
238 chrome::mojom::ResourceUsageReporterRequest request) {
238 mojo::MakeStrongBinding(base::MakeUnique<ResourceUsageReporterImpl>(observer), 239 mojo::MakeStrongBinding(base::MakeUnique<ResourceUsageReporterImpl>(observer),
239 std::move(request)); 240 std::move(request));
240 } 241 }
241 242
242 } // namespace 243 } // namespace
243 244
244 bool ChromeRenderThreadObserver::is_incognito_process_ = false; 245 bool ChromeRenderThreadObserver::is_incognito_process_ = false;
245 246
246 ChromeRenderThreadObserver::ChromeRenderThreadObserver() 247 ChromeRenderThreadObserver::ChromeRenderThreadObserver()
247 : visited_link_slave_(new visitedlink::VisitedLinkSlave), 248 : visited_link_slave_(new visitedlink::VisitedLinkSlave),
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 322
322 void ChromeRenderThreadObserver::OnRendererConfigurationAssociatedRequest( 323 void ChromeRenderThreadObserver::OnRendererConfigurationAssociatedRequest(
323 chrome::mojom::RendererConfigurationAssociatedRequest request) { 324 chrome::mojom::RendererConfigurationAssociatedRequest request) {
324 renderer_configuration_bindings_.AddBinding(this, std::move(request)); 325 renderer_configuration_bindings_.AddBinding(this, std::move(request));
325 } 326 }
326 327
327 const RendererContentSettingRules* 328 const RendererContentSettingRules*
328 ChromeRenderThreadObserver::content_setting_rules() const { 329 ChromeRenderThreadObserver::content_setting_rules() const {
329 return &content_setting_rules_; 330 return &content_setting_rules_;
330 } 331 }
OLDNEW
« no previous file with comments | « chrome/renderer/chrome_render_frame_observer.cc ('k') | chrome/renderer/content_settings_observer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698