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

Side by Side Diff: chrome/browser/ui/cocoa/tabs/tab_strip_controller.mm

Issue 2086273003: [Mac] Reveal Fullscreen Toolbar for Tab Strip Changes (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed test Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #import "chrome/browser/ui/cocoa/tabs/tab_strip_controller.h" 5 #import "chrome/browser/ui/cocoa/tabs/tab_strip_controller.h"
6 6
7 #import <QuartzCore/QuartzCore.h> 7 #import <QuartzCore/QuartzCore.h>
8 8
9 #include <cmath> 9 #include <cmath>
10 #include <limits> 10 #include <limits>
(...skipping 1266 matching lines...) Expand 10 before | Expand all | Expand 10 after
1277 if (!inForeground) { 1277 if (!inForeground) {
1278 [self layoutTabs]; 1278 [self layoutTabs];
1279 } 1279 }
1280 1280
1281 // During normal loading, we won't yet have a favicon and we'll get 1281 // During normal loading, we won't yet have a favicon and we'll get
1282 // subsequent state change notifications to show the throbber, but when we're 1282 // subsequent state change notifications to show the throbber, but when we're
1283 // dragging a tab out into a new window, we have to put the tab's favicon 1283 // dragging a tab out into a new window, we have to put the tab's favicon
1284 // into the right state up front as we won't be told to do it from anywhere 1284 // into the right state up front as we won't be told to do it from anywhere
1285 // else. 1285 // else.
1286 [self updateIconsForContents:contents atIndex:modelIndex]; 1286 [self updateIconsForContents:contents atIndex:modelIndex];
1287
1288 [delegate_ onTabInsertedInForeground:inForeground];
1287 } 1289 }
1288 1290
1289 // Called before |contents| is deactivated. 1291 // Called before |contents| is deactivated.
1290 - (void)tabDeactivatedWithContents:(content::WebContents*)contents { 1292 - (void)tabDeactivatedWithContents:(content::WebContents*)contents {
1291 contents->StoreFocus(); 1293 contents->StoreFocus();
1292 } 1294 }
1293 1295
1294 // Called when a notification is received from the model to select a particular 1296 // Called when a notification is received from the model to select a particular
1295 // tab. Swaps in the toolbar and content area associated with |newContents|. 1297 // tab. Swaps in the toolbar and content area associated with |newContents|.
1296 - (void)activateTabWithContents:(content::WebContents*)newContents 1298 - (void)activateTabWithContents:(content::WebContents*)newContents
(...skipping 1003 matching lines...) Expand 10 before | Expand all | Expand 10 after
2300 2302
2301 - (TabAlertState)alertStateForContents:(content::WebContents*)contents { 2303 - (TabAlertState)alertStateForContents:(content::WebContents*)contents {
2302 return chrome::GetTabAlertStateForContents(contents); 2304 return chrome::GetTabAlertStateForContents(contents);
2303 } 2305 }
2304 2306
2305 - (void)themeDidChangeNotification:(NSNotification*)notification { 2307 - (void)themeDidChangeNotification:(NSNotification*)notification {
2306 [newTabButton_ setImages]; 2308 [newTabButton_ setImages];
2307 } 2309 }
2308 2310
2309 @end 2311 @end
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698