From 93ffa77370d34ea72a9d0dee1d3c172762ac1e66 Mon Sep 17 00:00:00 2001 From: Timothy Flynn Date: Tue, 8 Aug 2023 16:03:42 -0400 Subject: [PATCH] Ladybird: Remove workaround for initial page load for WebDriver The issue noted in the removed comment no longer seems to apply. The URL loaded here is triggered before WebDriver even issues the POST /session command. --- Ladybird/Qt/BrowserWindow.cpp | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/Ladybird/Qt/BrowserWindow.cpp b/Ladybird/Qt/BrowserWindow.cpp index ea19770026..291686af38 100644 --- a/Ladybird/Qt/BrowserWindow.cpp +++ b/Ladybird/Qt/BrowserWindow.cpp @@ -495,12 +495,8 @@ Tab& BrowserWindow::new_tab(QString const& url, Web::HTML::ActivateTab activate_ tab_ptr->focus_location_editor(); - // We *don't* load the initial page if we are connected to a WebDriver, as the Set URL command may come in very - // quickly, and become replaced by this load. - if (m_webdriver_content_ipc_path.is_empty()) { - // We make it HistoryNavigation so that the initial page doesn't get added to the history. - tab_ptr->navigate(url, Tab::LoadType::HistoryNavigation); - } + // We make it HistoryNavigation so that the initial page doesn't get added to the history. + tab_ptr->navigate(url, Tab::LoadType::HistoryNavigation); return *tab_ptr; }