Move handling of DraggableRegionsChanged notification from "view" to "frame".

extensions/browser/app_window/app_window_contents.cc

Index: extensions/browser/app_window/app_window_contents.cc
diff --git a/extensions/browser/app_window/app_window_contents.cc b/extensions/browser/app_window/app_window_contents.cc
index 2478ddeba03723437966037bc5b854f3b7e9aeee..819090ac537ad86eabd51f930421fed1bd9d32fc 100644
--- a/extensions/browser/app_window/app_window_contents.cc
+++ b/extensions/browser/app_window/app_window_contents.cc
@@ -97,9 +97,11 @@ WindowController* AppWindowContentsImpl::GetWindowController() const {
   return nullptr;
 }
 
-bool AppWindowContentsImpl::OnMessageReceived(const IPC::Message& message) {
+bool AppWindowContentsImpl::OnMessageReceived(
+    const IPC::Message& message,
+    content::RenderFrameHost* sender) {
   bool handled = true;
-  IPC_BEGIN_MESSAGE_MAP(AppWindowContentsImpl, message)
+  IPC_BEGIN_MESSAGE_MAP_WITH_PARAM(AppWindowContentsImpl, message, sender)
     IPC_MESSAGE_HANDLER(ExtensionHostMsg_UpdateDraggableRegions,
                         UpdateDraggableRegions)
     IPC_MESSAGE_UNHANDLED(handled = false)
@@ -114,6 +116,7 @@ void AppWindowContentsImpl::ReadyToCommitNavigation(
 }
 
 void AppWindowContentsImpl::UpdateDraggableRegions(
+    content::RenderFrameHost* sender,
     const std::vector<DraggableRegion>& regions) {

[Łukasz Anforowicz, 2017/06/07 18:25:03]

Test coverage of this CL is not that great:

- There is *some* test coverage - some tests crash if I insert a CHECK(false)
here (e.g. LaunchWebAuthFlowFunctionTest).

- OTOH, the test coverage is superficial - no tests fail if I just return early
here.


I hope it is okay to depend on the limited test coverage described above and
then monitor crash rates (which should hopefully go down after this CL for
https://crbug.com/645472 - blink::WebDocument::draggableRegions).

I guess that in theory we could say that we'll add test coverage in a follow-up
CL, but I'd rather not sign up for that and so the promise of the follow-up test
coverage would be rather hollow... :-/

   host_->UpdateDraggableRegions(regions);
 }