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

Unified Diff: extensions/renderer/extensions_render_frame_observer.cc

Issue 2724933002: Convert SetVisuallyDeemphasized IPC to mojo (Closed)
Patch Set: Addressed Ken's comments Created 3 years, 9 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
« no previous file with comments | « extensions/renderer/extensions_render_frame_observer.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/renderer/extensions_render_frame_observer.cc
diff --git a/extensions/renderer/extensions_render_frame_observer.cc b/extensions/renderer/extensions_render_frame_observer.cc
index be6dbccd2f1e12e36e90bbc7021516595a1dbbec..a96c45f5ef5f3f3d8d078d5fcb4f0382258b9453 100644
--- a/extensions/renderer/extensions_render_frame_observer.cc
+++ b/extensions/renderer/extensions_render_frame_observer.cc
@@ -9,9 +9,12 @@
#include "base/strings/string_split.h"
#include "base/strings/utf_string_conversions.h"
#include "content/public/renderer/render_frame.h"
+#include "content/public/renderer/render_view.h"
#include "extensions/common/extension_messages.h"
#include "extensions/common/stack_frame.h"
+#include "services/service_manager/public/cpp/interface_registry.h"
#include "third_party/WebKit/public/web/WebLocalFrame.h"
+#include "third_party/WebKit/public/web/WebView.h"
namespace extensions {
@@ -76,12 +79,32 @@ StackTrace GetStackTraceFromMessage(base::string16* message,
ExtensionsRenderFrameObserver::ExtensionsRenderFrameObserver(
content::RenderFrame* render_frame)
- : content::RenderFrameObserver(render_frame) {
+ : content::RenderFrameObserver(render_frame),
+ webview_visually_deemphasized_(false) {
+ render_frame->GetInterfaceRegistry()->AddInterface(
+ base::Bind(&ExtensionsRenderFrameObserver::BindAppWindowRequest,
+ base::Unretained(this)));
}
ExtensionsRenderFrameObserver::~ExtensionsRenderFrameObserver() {
}
+void ExtensionsRenderFrameObserver::BindAppWindowRequest(
+ mojom::AppWindowRequest request) {
+ bindings_.AddBinding(this, std::move(request));
+}
+
+void ExtensionsRenderFrameObserver::SetVisuallyDeemphasized(bool deemphasized) {
+ if (webview_visually_deemphasized_ == deemphasized)
+ return;
+
+ webview_visually_deemphasized_ = deemphasized;
+
+ SkColor color =
+ deemphasized ? SkColorSetARGB(178, 0, 0, 0) : SK_ColorTRANSPARENT;
+ render_frame()->GetRenderView()->GetWebView()->setPageOverlayColor(color);
+}
+
void ExtensionsRenderFrameObserver::DetailedConsoleMessageAdded(
const base::string16& message,
const base::string16& source,
« no previous file with comments | « extensions/renderer/extensions_render_frame_observer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698