fix: handle nullish WebContentsView in UpdateDraggableRegions (#30558)

* fix: handle nullish WebContentsView in UpdateDraggableRegions * build: nogncheck on webcontentsimpl include Co-authored-by: Samuel Attard <samuel.r.attard@gmail.com>
electron · Aug 19, 2021 · 87399cc · 87399cc
1 parent 33631b4
commit 87399cc
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/shell/browser/api/electron_api_browser_window_views.cc b/shell/browser/api/electron_api_browser_window_views.cc
@@ -4,6 +4,7 @@
 
 #include "shell/browser/api/electron_api_browser_window.h"
 
+#include "content/browser/web_contents/web_contents_impl.h"  // nogncheck
 #include "shell/browser/native_window_views.h"
 #include "ui/aura/window.h"
 
@@ -17,8 +18,10 @@ void BrowserWindow::UpdateDraggableRegions(
     return;
 
   if (&draggable_regions_ != &regions) {
-    auto* nv = web_contents()->GetNativeView();
-    if (nv) {
+    auto* view =
+        static_cast<content::WebContentsImpl*>(web_contents())->GetView();
+    if (view) {
+      const gfx::NativeView nv = view->GetNativeView();
       auto const offset = nv->GetBoundsInRootWindow();
       auto snapped_regions = mojo::Clone(regions);
       for (auto& snapped_region : snapped_regions) {