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

Side by Side Diff: chrome/browser/ui/gtk/location_bar_view_gtk.h

Issue 10806058: Move icon fallbacks into ExtensionAction. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix kalman's comments. Created 8 years, 5 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) 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 #ifndef CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 5 #ifndef CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
6 #define CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 6 #define CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
7 7
8 #include <gtk/gtk.h> 8 #include <gtk/gtk.h>
9 9
10 #include <map> 10 #include <map>
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
296 const ExtensionAction::IconAnimation& animation) OVERRIDE; 296 const ExtensionAction::IconAnimation& animation) OVERRIDE;
297 297
298 // The location bar view that owns us. 298 // The location bar view that owns us.
299 LocationBarViewGtk* owner_; 299 LocationBarViewGtk* owner_;
300 300
301 // The PageAction that this view represents. The PageAction is not owned by 301 // The PageAction that this view represents. The PageAction is not owned by
302 // us, it resides in the extension of this particular profile. 302 // us, it resides in the extension of this particular profile.
303 ExtensionAction* page_action_; 303 ExtensionAction* page_action_;
304 304
305 // A cache of all the different icon paths associated with this page action. 305 // A cache of all the different icon paths associated with this page action.
306 typedef std::map<std::string, GdkPixbuf*> PixbufMap; 306 ExtensionAction::PathToIconCache loaded_icons_;
307 PixbufMap pixbufs_;
308
309 // A cache of the last dynamically generated bitmap and the pixbuf that
310 // corresponds to it. We keep track of both so we can free old pixbufs as
311 // their icons are replaced.
312 SkBitmap last_icon_skbitmap_;
313 GdkPixbuf* last_icon_pixbuf_;
314 307
315 // The object that is waiting for the image loading to complete 308 // The object that is waiting for the image loading to complete
316 // asynchronously. 309 // asynchronously.
317 ImageLoadingTracker tracker_; 310 ImageLoadingTracker tracker_;
318 311
319 // The widgets for this page action. 312 // The widgets for this page action.
320 ui::OwnedWidgetGtk event_box_; 313 ui::OwnedWidgetGtk event_box_;
321 ui::OwnedWidgetGtk image_; 314 ui::OwnedWidgetGtk image_;
322 315
323 // The tab id we are currently showing the icon for. 316 // The tab id we are currently showing the icon for.
(...skipping 215 matching lines...) Expand 10 before | Expand all | Expand 10 after
539 BooleanPrefMember edit_bookmarks_enabled_; 532 BooleanPrefMember edit_bookmarks_enabled_;
540 533
541 // Used to remember the URL and title text when drag&drop has begun. 534 // Used to remember the URL and title text when drag&drop has begun.
542 GURL drag_url_; 535 GURL drag_url_;
543 string16 drag_title_; 536 string16 drag_title_;
544 537
545 DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk); 538 DISALLOW_COPY_AND_ASSIGN(LocationBarViewGtk);
546 }; 539 };
547 540
548 #endif // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_ 541 #endif // CHROME_BROWSER_UI_GTK_LOCATION_BAR_VIEW_GTK_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698