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

Unified Diff: extensions/renderer/extension_helper.cc

Issue 2929493003: Move handling of DraggableRegionsChanged notification from "view" to "frame". (Closed)
Patch Set: Also add main frame filtering to the browser-side. Created 3 years, 6 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: extensions/renderer/extension_helper.cc
diff --git a/extensions/renderer/extension_helper.cc b/extensions/renderer/extension_helper.cc
index d90a34d7066fcc1186acd604aed54accda9a574f..e062da75438b4a35edc7590caea613d2f0a0cb0e 100644
--- a/extensions/renderer/extension_helper.cc
+++ b/extensions/renderer/extension_helper.cc
@@ -45,20 +45,6 @@ void ExtensionHelper::OnDestruct() {
delete this;
}
-void ExtensionHelper::DraggableRegionsChanged(blink::WebFrame* frame) {
- blink::WebVector<blink::WebDraggableRegion> webregions =
- frame->GetDocument().DraggableRegions();
- std::vector<DraggableRegion> regions;
- for (size_t i = 0; i < webregions.size(); ++i) {
- DraggableRegion region;
- render_view()->ConvertViewportToWindowViaWidget(&webregions[i].bounds);
- region.bounds = webregions[i].bounds;
- region.draggable = webregions[i].draggable;
- regions.push_back(region);
- }
- Send(new ExtensionHostMsg_UpdateDraggableRegions(routing_id(), regions));
-}
-
void ExtensionHelper::OnSetFrameName(const std::string& name) {
blink::WebView* web_view = render_view()->GetWebView();
if (web_view)

Powered by Google App Engine
This is Rietveld 408576698