Support close request from web

CefView · Nov 6, 2023 · e6e2cd6 · e6e2cd6
1 parent f10e668
commit e6e2cd6
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 6 deletions.
diff --git a/include/CefViewBrowserClient.h b/include/CefViewBrowserClient.h
@@ -49,6 +49,7 @@ class CefViewBrowserClient
 
 private:
   bool is_closing_;
+  bool close_by_native_;
   bool initial_navigation_;
 
   std::unordered_map<int, CefRefPtr<CefBrowser>> browser_map_;

diff --git a/include/CefViewBrowserClientDelegate.h b/include/CefViewBrowserClientDelegate.h
@@ -141,6 +141,8 @@ class CefViewBrowserClientDelegateInterface
 
   virtual bool doClose(CefRefPtr<CefBrowser> browser) = 0;
 
+  virtual bool requestClose(CefRefPtr<CefBrowser> browser) = 0;
+
   virtual void onBeforeClose(CefRefPtr<CefBrowser> browser) = 0;
 #pragma endregion
 

diff --git a/src/CefView/CefBrowserApp/CefViewBrowserClient.cpp b/src/CefView/CefBrowserApp/CefViewBrowserClient.cpp
@@ -16,6 +16,7 @@
 CefViewBrowserClient::CefViewBrowserClient(CefRefPtr<CefViewBrowserApp> app,
                                            CefViewBrowserClientDelegateInterface::RefPtr delegate)
   : is_closing_(false)
+  , close_by_native_(false)
   , initial_navigation_(true)
   , app_(app)
   , client_delegate_(delegate)
@@ -39,6 +40,7 @@ CefViewBrowserClient::~CefViewBrowserClient()
 void
 CefViewBrowserClient::CloseAllBrowsers()
 {
+  close_by_native_ = true;
   auto browsers = browser_map_;
   for (auto& kv : browsers) {
     kv.second->StopLoad();

diff --git a/src/CefView/CefBrowserApp/CefViewBrowserClient_LifeSpanHandler.cpp b/src/CefView/CefBrowserApp/CefViewBrowserClient_LifeSpanHandler.cpp
@@ -89,17 +89,29 @@ CefViewBrowserClient::DoClose(CefRefPtr<CefBrowser> browser)
 {
   CEF_REQUIRE_UI_THREAD();
 
-  if (!browser->IsPopup()) {
-    is_closing_ = true;
+  bool ignoreClose = false;
+
+  if (browser->IsPopup()) {
+    return ignoreClose;
   }
 
   auto delegate = client_delegate_.lock();
   if (delegate)
-    return delegate->doClose(browser);
+  {
+    if (close_by_native_) {
+      // close by native
+      ignoreClose = delegate->doClose(browser);
+    } else {
+      // close request from web
+      ignoreClose = delegate->requestClose(browser);
+    }
+  }
+
+  if (!ignoreClose) {
+    is_closing_ = true;
+  }
 
-  // Allow the close. For windowed browsers this will result in the OS close
-  // event being sent.
-  return false;
+  return ignoreClose;
 }
 
 void