Commit 3f22e50b authored by Eric Willigers's avatar Eric Willigers Committed by Commit Bot

desktop-pwas: Avoid needless namespace qualifier

The code in app_browser_controller.cc is within the web_app namespace,
so qualifying names with 'web:app::' is redundant.

Change-Id: I8852005ec93db644aa9034115d5f6dcafe2f9ac1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888238
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Commit-Queue: Alexey Baskakov <loyso@chromium.org>
Auto-Submit: Eric Willigers <ericwilligers@chromium.org>
Reviewed-by: default avatarAlexey Baskakov <loyso@chromium.org>
Cr-Commit-Position: refs/heads/master@{#710687}
parent f5fa338f
...@@ -38,15 +38,14 @@ ...@@ -38,15 +38,14 @@
namespace web_app { namespace web_app {
// static // static
std::unique_ptr<web_app::AppBrowserController> std::unique_ptr<AppBrowserController>
AppBrowserController::MaybeCreateWebAppController(Browser* browser) { AppBrowserController::MaybeCreateWebAppController(Browser* browser) {
#if BUILDFLAG(ENABLE_EXTENSIONS) #if BUILDFLAG(ENABLE_EXTENSIONS)
const AppId app_id = const AppId app_id = GetAppIdFromApplicationName(browser->app_name());
web_app::GetAppIdFromApplicationName(browser->app_name());
if (base::FeatureList::IsEnabled(features::kDesktopPWAsWithoutExtensions)) { if (base::FeatureList::IsEnabled(features::kDesktopPWAsWithoutExtensions)) {
auto* provider = web_app::WebAppProvider::Get(browser->profile()); auto* provider = WebAppProvider::Get(browser->profile());
if (provider && provider->registrar().IsInstalled(app_id)) if (provider && provider->registrar().IsInstalled(app_id))
return std::make_unique<web_app::WebAppBrowserController>(browser); return std::make_unique<WebAppBrowserController>(browser);
} }
const extensions::Extension* extension = const extensions::Extension* extension =
extensions::ExtensionRegistry::Get(browser->profile()) extensions::ExtensionRegistry::Get(browser->profile())
...@@ -55,7 +54,7 @@ AppBrowserController::MaybeCreateWebAppController(Browser* browser) { ...@@ -55,7 +54,7 @@ AppBrowserController::MaybeCreateWebAppController(Browser* browser) {
if (base::FeatureList::IsEnabled( if (base::FeatureList::IsEnabled(
features::kDesktopPWAsUnifiedUiController) && features::kDesktopPWAsUnifiedUiController) &&
extension->from_bookmark()) { extension->from_bookmark()) {
return std::make_unique<web_app::WebAppBrowserController>(browser); return std::make_unique<WebAppBrowserController>(browser);
} }
return std::make_unique<extensions::HostedAppBrowserController>(browser); return std::make_unique<extensions::HostedAppBrowserController>(browser);
} }
...@@ -217,8 +216,7 @@ bool AppBrowserController::IsHostedApp() const { ...@@ -217,8 +216,7 @@ bool AppBrowserController::IsHostedApp() const {
return false; return false;
} }
web_app::WebAppBrowserController* WebAppBrowserController* AppBrowserController::AsWebAppBrowserController() {
AppBrowserController::AsWebAppBrowserController() {
return nullptr; return nullptr;
} }
...@@ -240,7 +238,7 @@ bool AppBrowserController::IsForSystemWebApp() const { ...@@ -240,7 +238,7 @@ bool AppBrowserController::IsForSystemWebApp() const {
if (!GetAppId()) if (!GetAppId())
return false; return false;
return web_app::WebAppProvider::Get(browser()->profile()) return WebAppProvider::Get(browser()->profile())
->system_web_app_manager() ->system_web_app_manager()
.IsSystemWebApp(*GetAppId()); .IsSystemWebApp(*GetAppId());
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment