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

Unified Diff: components/contextual_search/renderer/overlay_js_render_frame_observer.h

Issue 2851173004: Eliminate bind callback that doesn't take a BindSourceInfo parameter. (Closed)
Patch Set: . Created 3 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: components/contextual_search/renderer/overlay_js_render_frame_observer.h
diff --git a/components/contextual_search/renderer/overlay_js_render_frame_observer.h b/components/contextual_search/renderer/overlay_js_render_frame_observer.h
index 53a7c870be1f475b2f54e8ad4c2a69406d8f90e3..fdaf82222e9e26c3084932339230508656bba879 100644
--- a/components/contextual_search/renderer/overlay_js_render_frame_observer.h
+++ b/components/contextual_search/renderer/overlay_js_render_frame_observer.h
@@ -10,6 +10,7 @@
#include "components/contextual_search/common/overlay_page_notifier_service.mojom.h"
#include "content/public/renderer/render_frame.h"
#include "content/public/renderer/render_frame_observer.h"
+#include "services/service_manager/public/cpp/bind_source_info.h"
#include "third_party/WebKit/public/web/WebLocalFrame.h"
#include "v8/include/v8.h"
@@ -42,7 +43,8 @@ class OverlayJsRenderFrameObserver : public content::RenderFrameObserver {
// Creates the OverlayPageNotifierService connecting the browser to this
// observer.
void CreateOverlayPageNotifierService(
- mojo::InterfaceRequest<mojom::OverlayPageNotifierService> request);
+ const service_manager::BindSourceInfo& source_info,
+ mojom::OverlayPageNotifierServiceRequest request);
// Destroys the OverlayPageNotifierService.
void DestroyOverlayPageNotifierService();

Powered by Google App Engine
This is Rietveld 408576698