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

Unified Diff: chrome/renderer/chrome_content_renderer_client.cc

Issue 2393513004: Convert app banners to use Mojo. (Closed)
Patch Set: Rebase Created 4 years, 2 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: chrome/renderer/chrome_content_renderer_client.cc
diff --git a/chrome/renderer/chrome_content_renderer_client.cc b/chrome/renderer/chrome_content_renderer_client.cc
index 6f0950ee21f02f21eb09159fc1c96a336dc1c3a6..39f0423865b2c937b0a53d43ed816ae4f8e091f7 100644
--- a/chrome/renderer/chrome_content_renderer_client.cc
+++ b/chrome/renderer/chrome_content_renderer_client.cc
@@ -35,7 +35,6 @@
#include "chrome/grit/locale_settings.h"
#include "chrome/grit/renderer_resources.h"
#include "chrome/renderer/app_categorizer.h"
-#include "chrome/renderer/banners/app_banner_client.h"
#include "chrome/renderer/benchmarking_extension.h"
#include "chrome/renderer/chrome_render_frame_observer.h"
#include "chrome/renderer/chrome_render_thread_observer.h"
@@ -97,6 +96,7 @@
#include "ppapi/c/private/ppb_pdf.h"
#include "ppapi/shared_impl/ppapi_switches.h"
#include "services/shell/public/cpp/interface_provider.h"
+#include "services/shell/public/cpp/interface_registry.h"
#include "third_party/WebKit/public/platform/URLConversion.h"
#include "third_party/WebKit/public/platform/WebCachePolicy.h"
#include "third_party/WebKit/public/platform/WebSecurityOrigin.h"
@@ -434,7 +434,12 @@ void ChromeContentRendererClient::RenderThreadStarted() {
void ChromeContentRendererClient::RenderFrameCreated(
content::RenderFrame* render_frame) {
- new ChromeRenderFrameObserver(render_frame);
+ ChromeRenderFrameObserver* observer =
+ new ChromeRenderFrameObserver(render_frame);
+
+ render_frame->GetInterfaceRegistry()->AddInterface(
+ base::Bind(&ChromeRenderFrameObserver::BindBannerClient,
+ base::Unretained(observer)));
bool should_whitelist_for_content_settings =
base::CommandLine::ForCurrentProcess()->HasSwitch(
@@ -1336,13 +1341,6 @@ void ChromeContentRendererClient::RecordRapporURL(const std::string& metric,
RenderThread::Get()->Send(new ChromeViewHostMsg_RecordRapporURL(metric, url));
}
-std::unique_ptr<blink::WebAppBannerClient>
-ChromeContentRendererClient::CreateAppBannerClient(
- content::RenderFrame* render_frame) {
- return std::unique_ptr<blink::WebAppBannerClient>(
- new AppBannerClient(render_frame));
-}
-
void ChromeContentRendererClient::AddImageContextMenuProperties(
const WebURLResponse& response,
std::map<std::string, std::string>* properties) {

Powered by Google App Engine
This is Rietveld 408576698