Convert SetVisuallyDeemphasized IPC to mojo

chrome/browser/ui/apps/chrome_app_delegate.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/apps/chrome_app_delegate.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/macros.h"
@@ skipping 16 matching lines @@
 #include "chrome/browser/ui/browser_tabstrip.h"
 #include "chrome/browser/ui/browser_window.h"
 #include "chrome/browser/ui/scoped_tabbed_browser_displayer.h"
 #include "chrome/browser/ui/web_contents_sizer.h"
 #include "chrome/common/extensions/chrome_extension_messages.h"
 #include "components/zoom/zoom_controller.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/host_zoom_map.h"
 #include "content/public/browser/notification_service.h"
-#include "content/public/browser/render_view_host.h"
+#include "content/public/browser/render_frame_host.h"
+#include "content/public/browser/render_process_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "extensions/common/constants.h"
 #include "printing/features/features.h"
+#include "services/service_manager/public/cpp/interface_provider.h"
 
 #if defined(USE_ASH)
 #include "ash/common/shelf/shelf_constants.h"  // nogncheck
 #endif
 
 #if BUILDFLAG(ENABLE_PRINTING)
 #if BUILDFLAG(ENABLE_PRINT_PREVIEW)
 #include "chrome/browser/printing/print_preview_message_handler.h"
 #include "chrome/browser/printing/print_view_manager.h"
 #else
@@ skipping 246 matching lines @@
   return extension_misc::EXTENSION_ICON_SMALL;
 #endif
 }
 
 void ChromeAppDelegate::SetWebContentsBlocked(
     content::WebContents* web_contents,
     bool blocked) {
   if (!blocked)
     web_contents->Focus();
   // RenderViewHost may be NULL during shutdown.
-  content::RenderViewHost* host = web_contents->GetRenderViewHost();
+  content::RenderFrameHost* host = web_contents->GetMainFrame();
   if (host) {
-    host->Send(new ChromeViewMsg_SetVisuallyDeemphasized(host->GetRoutingID(),
-                                                         blocked));
+    host->GetProcess()->GetRemoteInterfaces()->GetInterface(
+        &extensions_frame_observer_);

[Devlin, 2017/03/04 02:19:41]

This re-gets the interface each time we send the message.  Seems like we should
be either caching the interface in a local variable, rather than a member
variable, or else checking whether or not its initialized before trying to get
it.  I think the former makes more sense here, since it appears the
ChromeAppDelegate can be used across multiple WebContents/RenderFrameHosts.

[catmullings, 2017/03/06 21:20:40]

Done.

+    extensions_frame_observer_->SetVisuallyDeemphasized(blocked);
   }
 }
 
 bool ChromeAppDelegate::IsWebContentsVisible(
     content::WebContents* web_contents) {
   return platform_util::IsVisible(web_contents->GetNativeView());
 }
 
 void ChromeAppDelegate::SetTerminatingCallback(const base::Closure& callback) {
   terminating_callback_ = callback;
@@ skipping 22 matching lines @@
                                         KeepAliveRestartOption::DISABLED));
 }
 
 void ChromeAppDelegate::Observe(int type,
                                 const content::NotificationSource& source,
                                 const content::NotificationDetails& details) {
   DCHECK_EQ(chrome::NOTIFICATION_APP_TERMINATING, type);
   if (!terminating_callback_.is_null())
     terminating_callback_.Run();
 }