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

third_party/WebKit/Source/core/frame/LocalFrameView.cpp

Index: third_party/WebKit/Source/core/frame/LocalFrameView.cpp
diff --git a/third_party/WebKit/Source/core/frame/LocalFrameView.cpp b/third_party/WebKit/Source/core/frame/LocalFrameView.cpp
index ab4eeb4512b421d2b4390744847448881398114f..90488c05a93a1d5c725984cad0ad80ed05eb3d63 100644
--- a/third_party/WebKit/Source/core/frame/LocalFrameView.cpp
+++ b/third_party/WebKit/Source/core/frame/LocalFrameView.cpp
@@ -2823,8 +2823,8 @@ void LocalFrameView::UpdateDocumentAnnotatedRegions() const {
   if (new_regions == document->AnnotatedRegions())
     return;
   document->SetAnnotatedRegions(new_regions);
-  if (Page* page = frame_->GetPage())
-    page->GetChromeClient().AnnotatedRegionsChanged();
+  if (frame_->Client())

[dcheng, 2017/06/07 20:22:35]

I strongly suspect that this should be a DCHECK--this is getting called on the
FrameView, which is set to null during detach (so this wouldn't be reachable)

[Łukasz Anforowicz, 2017/06/08 16:29:24]

Done (I've also added a comment - please shout if the comment includes too much
information [i.e. if the comment is likely to become stale / out-of-sync] and if
I should remove or reword the comment).

+    frame_->Client()->AnnotatedRegionsChanged();
 }
 
 void LocalFrameView::DidAttachDocument() {