Browse Source

fix: allow docking DevTools with WCO (#35764)

fix: allow for docking devtools with WCO

Co-authored-by: Shelley Vohr <[email protected]>
trop[bot] 2 years ago
parent
commit
25521f6286
1 changed files with 9 additions and 8 deletions
  1. 9 8
      shell/browser/api/electron_api_web_contents.cc

+ 9 - 8
shell/browser/api/electron_api_web_contents.cc

@@ -2410,14 +2410,6 @@ void WebContents::OpenDevTools(gin::Arguments* args) {
       !owner_window()) {
     state = "detach";
   }
-  bool activate = true;
-  if (args && args->Length() == 1) {
-    gin_helper::Dictionary options;
-    if (args->GetNext(&options)) {
-      options.Get("mode", &state);
-      options.Get("activate", &activate);
-    }
-  }
 
 #if BUILDFLAG(IS_WIN)
   auto* win = static_cast<NativeWindowViews*>(owner_window());
@@ -2427,6 +2419,15 @@ void WebContents::OpenDevTools(gin::Arguments* args) {
     state = "detach";
 #endif
 
+  bool activate = true;
+  if (args && args->Length() == 1) {
+    gin_helper::Dictionary options;
+    if (args->GetNext(&options)) {
+      options.Get("mode", &state);
+      options.Get("activate", &activate);
+    }
+  }
+
   DCHECK(inspectable_web_contents_);
   inspectable_web_contents_->SetDockState(state);
   inspectable_web_contents_->ShowDevTools(activate);