refactor: sync IsKillURL() with upstream impl in extension_tab_util.cc (#47580)

Use base::MakeFixedFlatSet()
This commit is contained in:
Charles Kerr
2025-06-27 15:49:41 -05:00
committed by GitHub
parent 2ba8fe9140
commit 79cd6a2614

View File

@@ -11,7 +11,7 @@
#include <vector>
#include "base/command_line.h"
#include "base/containers/contains.h"
#include "base/containers/fixed_flat_set.h"
#include "base/strings/pattern.h"
#include "base/types/expected_macros.h"
#include "chrome/common/url_constants.h"
@@ -497,13 +497,16 @@ bool IsKillURL(const GURL& url) {
}
// Also disallow a few more hosts which are not covered by the check above.
static const char* const kKillHosts[] = {
chrome::kChromeUIDelayedHangUIHost, chrome::kChromeUIHangUIHost,
chrome::kChromeUIQuitHost, chrome::kChromeUIRestartHost,
content::kChromeUIBrowserCrashHost, content::kChromeUIMemoryExhaustHost,
};
constexpr auto kKillHosts = base::MakeFixedFlatSet<std::string_view>({
chrome::kChromeUIDelayedHangUIHost,
chrome::kChromeUIHangUIHost,
chrome::kChromeUIQuitHost,
chrome::kChromeUIRestartHost,
content::kChromeUIBrowserCrashHost,
content::kChromeUIMemoryExhaustHost,
});
return base::Contains(kKillHosts, url.host_piece());
return kKillHosts.contains(url.host_piece());
}
GURL ResolvePossiblyRelativeURL(const std::string& url_string,