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

Unified Diff: chrome/browser/ui/views/frame/browser_view_layout.cc

Issue 2690443002: cros-md: Remove the non-MD immersive mode code paths (Closed)
Patch Set: rebase & code comments Created 3 years, 10 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
Index: chrome/browser/ui/views/frame/browser_view_layout.cc
diff --git a/chrome/browser/ui/views/frame/browser_view_layout.cc b/chrome/browser/ui/views/frame/browser_view_layout.cc
index 8ad5c754eacdec1469e07bfbb11791d3857d876a..f781e84368037b6f2a95bb376a281e686bdbfadd 100644
--- a/chrome/browser/ui/views/frame/browser_view_layout.cc
+++ b/chrome/browser/ui/views/frame/browser_view_layout.cc
@@ -448,12 +448,10 @@ int BrowserViewLayout::LayoutBookmarkBar(int top) {
int BrowserViewLayout::LayoutInfoBar(int top) {
// In immersive fullscreen, the infobar always starts near the top of the
- // screen, just under the "light bar" rectangular stripes.
- if (immersive_mode_controller_->IsEnabled()) {
+ // screen.
+ if (immersive_mode_controller_->IsEnabled())
top = browser_view_->y();
- if (!immersive_mode_controller_->ShouldHideTabIndicators())
- top += TabStrip::GetImmersiveHeight();
- }
+
infobar_container_->SetVisible(InfobarVisible());
infobar_container_->SetBounds(
vertical_layout_rect_.x(), top, vertical_layout_rect_.width(),

Powered by Google App Engine
This is Rietveld 408576698