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

Side by Side Diff: chrome/browser/cocoa/location_bar_view_mac.h

Issue 1075006: Eliminate all UI thread decoding of extension images.... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 9 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2010 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 #ifndef CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_ 5 #ifndef CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_
6 #define CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_ 6 #define CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_
7 7
8 #include <string> 8 #include <string>
9 #include <map> 9 #include <map>
10 #include <vector> 10 #include <vector>
(...skipping 223 matching lines...) Expand 10 before | Expand all | Expand 10 after
234 // When a new page action is created, all the icons are destroyed and 234 // When a new page action is created, all the icons are destroyed and
235 // recreated; at this point we need to calculate sizes to lay out the 235 // recreated; at this point we need to calculate sizes to lay out the
236 // icons even though no images are available yet. For this case, we return 236 // icons even though no images are available yet. For this case, we return
237 // the default image size for a page icon. 237 // the default image size for a page icon.
238 virtual NSSize GetPreferredImageSize(); 238 virtual NSSize GetPreferredImageSize();
239 239
240 // Either notify listeners or show a popup depending on the Page Action. 240 // Either notify listeners or show a popup depending on the Page Action.
241 virtual void OnMousePressed(NSRect bounds); 241 virtual void OnMousePressed(NSRect bounds);
242 242
243 // Overridden from ImageLoadingTracker. 243 // Overridden from ImageLoadingTracker.
244 virtual void OnImageLoaded(SkBitmap* image, size_t index); 244 virtual void OnImageLoaded(SkBitmap* image, int index);
245 245
246 // Called to notify the Page Action that it should determine whether to be 246 // Called to notify the Page Action that it should determine whether to be
247 // visible or hidden. |contents| is the TabContents that is active, |url| 247 // visible or hidden. |contents| is the TabContents that is active, |url|
248 // is the current page URL. 248 // is the current page URL.
249 void UpdateVisibility(TabContents* contents, const GURL& url); 249 void UpdateVisibility(TabContents* contents, const GURL& url);
250 250
251 // Sets the tooltip for this Page Action image. 251 // Sets the tooltip for this Page Action image.
252 void SetToolTip(NSString* tooltip); 252 void SetToolTip(NSString* tooltip);
253 void SetToolTip(std::string tooltip); 253 void SetToolTip(std::string tooltip);
254 254
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
428 // The transition type to use for the navigation. 428 // The transition type to use for the navigation.
429 PageTransition::Type transition_; 429 PageTransition::Type transition_;
430 430
431 // Used to register for notifications received by NotificationObserver. 431 // Used to register for notifications received by NotificationObserver.
432 NotificationRegistrar registrar_; 432 NotificationRegistrar registrar_;
433 433
434 DISALLOW_COPY_AND_ASSIGN(LocationBarViewMac); 434 DISALLOW_COPY_AND_ASSIGN(LocationBarViewMac);
435 }; 435 };
436 436
437 #endif // CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_ 437 #endif // CHROME_BROWSER_COCOA_LOCATION_BAR_VIEW_MAC_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698