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

Side by Side Diff: chrome/browser/tab_contents/tab_contents.h

Issue 249023: Use favicon for application shortcut icon. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: fixed unittests, handle null favicon Created 11 years, 2 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) 2009 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009 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_TAB_CONTENTS_TAB_CONTENTS_H_ 5 #ifndef CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_
6 #define CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_ 6 #define CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_
7 7
8 #include "build/build_config.h" 8 #include "build/build_config.h"
9 9
10 #include <map> 10 #include <map>
(...skipping 204 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 // bar. Normally this is true so you can see the URL. This is set to false 215 // bar. Normally this is true so you can see the URL. This is set to false
216 // for the new tab page and related pages so that the URL bar is empty and 216 // for the new tab page and related pages so that the URL bar is empty and
217 // the user is invited to type into it. 217 // the user is invited to type into it.
218 virtual bool ShouldDisplayURL(); 218 virtual bool ShouldDisplayURL();
219 219
220 // Returns the favicon for this tab, or an isNull() bitmap if the tab does not 220 // Returns the favicon for this tab, or an isNull() bitmap if the tab does not
221 // have a favicon. The default implementation uses the current navigation 221 // have a favicon. The default implementation uses the current navigation
222 // entry. 222 // entry.
223 SkBitmap GetFavIcon() const; 223 SkBitmap GetFavIcon() const;
224 224
225 // Returns true if we are not using the default favicon.
226 bool FavIconIsValid() const;
227
225 // Returns whether the favicon should be displayed. If this returns false, no 228 // Returns whether the favicon should be displayed. If this returns false, no
226 // space is provided for the favicon, and the favicon is never displayed. 229 // space is provided for the favicon, and the favicon is never displayed.
227 virtual bool ShouldDisplayFavIcon(); 230 virtual bool ShouldDisplayFavIcon();
228 231
229 // Returns a human-readable description the tab's loading state. 232 // Returns a human-readable description the tab's loading state.
230 virtual std::wstring GetStatusText() const; 233 virtual std::wstring GetStatusText() const;
231 234
232 // Return whether this tab contents is loading a resource. 235 // Return whether this tab contents is loading a resource.
233 bool is_loading() const { return is_loading_; } 236 bool is_loading() const { return is_loading_; }
234 237
(...skipping 916 matching lines...) Expand 10 before | Expand all | Expand 10 after
1151 // If this tab was created from a renderer using window.open, this will be 1154 // If this tab was created from a renderer using window.open, this will be
1152 // non-NULL and represent the DOMUI of the opening renderer. 1155 // non-NULL and represent the DOMUI of the opening renderer.
1153 DOMUITypeID opener_dom_ui_type_; 1156 DOMUITypeID opener_dom_ui_type_;
1154 1157
1155 // --------------------------------------------------------------------------- 1158 // ---------------------------------------------------------------------------
1156 1159
1157 DISALLOW_COPY_AND_ASSIGN(TabContents); 1160 DISALLOW_COPY_AND_ASSIGN(TabContents);
1158 }; 1161 };
1159 1162
1160 #endif // CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_ 1163 #endif // CHROME_BROWSER_TAB_CONTENTS_TAB_CONTENTS_H_
OLDNEW
« no previous file with comments | « chrome/browser/shell_integration_unittest.cc ('k') | chrome/browser/tab_contents/tab_contents.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698