Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(4646)

Unified Diff: chrome/browser/ui/browser.cc

Issue 1164873003: Show origin for hosted apps that navigate away from their start origin. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: feedback for the stig Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/ui/browser.h ('k') | chrome/browser/ui/browser_browsertest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/browser.cc
diff --git a/chrome/browser/ui/browser.cc b/chrome/browser/ui/browser.cc
index 6ccb52cab2b020b29baaca9c8b71cbd22edf7350..1cc78f27f2e4b6fd4fa11efd14c1bd64a4bf9825 100644
--- a/chrome/browser/ui/browser.cc
+++ b/chrome/browser/ui/browser.cc
@@ -107,7 +107,7 @@
#include "chrome/browser/ui/chrome_select_file_policy.h"
#include "chrome/browser/ui/exclusive_access/fullscreen_controller.h"
#include "chrome/browser/ui/exclusive_access/mouse_lock_controller.h"
-#include "chrome/browser/ui/extensions/bookmark_app_browser_controller.h"
+#include "chrome/browser/ui/extensions/hosted_app_browser_controller.h"
#include "chrome/browser/ui/fast_unload_controller.h"
#include "chrome/browser/ui/find_bar/find_bar.h"
#include "chrome/browser/ui/find_bar/find_bar_controller.h"
@@ -418,9 +418,9 @@ Browser::Browser(const CreateParams& params)
if (chrome::IsInstantExtendedAPIEnabled() && is_type_tabbed())
instant_controller_.reset(new BrowserInstantController(this));
- if (extensions::BookmarkAppBrowserController::IsForBookmarkApp(this)) {
- bookmark_app_controller_.reset(
- new extensions::BookmarkAppBrowserController(this));
+ if (extensions::HostedAppBrowserController::IsForHostedApp(this)) {
+ hosted_app_controller_.reset(
+ new extensions::HostedAppBrowserController(this));
}
UpdateBookmarkBarState(BOOKMARK_BAR_STATE_CHANGE_INIT);
@@ -429,8 +429,8 @@ Browser::Browser(const CreateParams& params)
window_ = params.window ? params.window : CreateBrowserWindow(this);
- if (bookmark_app_controller_)
- bookmark_app_controller_->UpdateLocationBarVisibility(false);
+ if (hosted_app_controller_)
+ hosted_app_controller_->UpdateLocationBarVisibility(false);
// Create the extension window controller before sending notifications.
extension_window_controller_.reset(
@@ -1438,8 +1438,8 @@ void Browser::NavigationStateChanged(WebContents* source,
content::INVALIDATE_TYPE_LOAD))
command_controller_->TabStateChanged();
- if (bookmark_app_controller_)
- bookmark_app_controller_->UpdateLocationBarVisibility(true);
+ if (hosted_app_controller_)
+ hosted_app_controller_->UpdateLocationBarVisibility(true);
}
void Browser::VisibleSSLStateChanged(const WebContents* source) {
@@ -2429,8 +2429,8 @@ bool Browser::SupportsLocationBar() const {
if (!is_app())
return !is_trusted_source();
- if (bookmark_app_controller_)
- return bookmark_app_controller_->SupportsLocationBar();
+ if (hosted_app_controller_)
+ return hosted_app_controller_->SupportsLocationBar();
return false;
}
@@ -2441,8 +2441,8 @@ bool Browser::ShouldUseWebAppFrame() const {
if (!is_app())
return false;
- if (bookmark_app_controller_)
- return bookmark_app_controller_->should_use_web_app_frame();
+ if (hosted_app_controller_)
+ return hosted_app_controller_->should_use_web_app_frame();
return false;
}
« no previous file with comments | « chrome/browser/ui/browser.h ('k') | chrome/browser/ui/browser_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698