Fixes call to ShouldBlockPlaylistWebUI().

brave · Nov 6, 2024 · 0ae54ec · 0ae54ec
1 parent 0ab239d
commit 0ae54ec
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/browser/ui/BUILD.gn b/browser/ui/BUILD.gn
@@ -1321,6 +1321,10 @@ source_set("ui") {
     ]
   }
 
+  if (enable_playlist) {
+    deps += [ "//brave/components/playlist/common" ]
+  }
+
   if (enable_playlist_webui) {
     sources += [
       "playlist/playlist_browser_finder.cc",

diff --git a/browser/ui/webui/brave_web_ui_controller_factory.cc b/browser/ui/webui/brave_web_ui_controller_factory.cc
@@ -63,6 +63,7 @@
 
 #if BUILDFLAG(ENABLE_PLAYLIST_WEBUI)
 #include "brave/browser/ui/webui/playlist_ui.h"
+#include "brave/components/playlist/common/features.h"
 #endif
 
 #if BUILDFLAG(ENABLE_TOR)
@@ -244,8 +245,10 @@ WebUI::TypeID BraveWebUIControllerFactory::GetWebUIType(
   }
 #endif  // BUILDFLAG(IS_ANDROID)
 #if BUILDFLAG(ENABLE_PLAYLIST_WEBUI)
-  if (playlist::PlaylistUI::ShouldBlockPlaylistWebUI(browser_context, url)) {
-    return WebUI::kNoWebUI;
+  if (base::FeatureList::IsEnabled(playlist::features::kPlaylist)) {
+    if (playlist::PlaylistUI::ShouldBlockPlaylistWebUI(browser_context, url)) {
+      return WebUI::kNoWebUI;
+    }
   }
 #endif