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

Side by Side Diff: chrome/browser/views/tabs/tab_renderer.h

Issue 63029: Clarify the documentation for loading_only on TabRenderer::UpdateData. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 years, 8 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) 2006-2008 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2006-2008 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_VIEWS_TABS_TAB_RENDERER_H__ 5 #ifndef CHROME_BROWSER_VIEWS_TABS_TAB_RENDERER_H__
6 #define CHROME_BROWSER_VIEWS_TABS_TAB_RENDERER_H__ 6 #define CHROME_BROWSER_VIEWS_TABS_TAB_RENDERER_H__
7 7
8 #include "base/gfx/point.h" 8 #include "base/gfx/point.h"
9 #include "chrome/common/animation.h" 9 #include "chrome/common/animation.h"
10 #include "chrome/common/slide_animation.h" 10 #include "chrome/common/slide_animation.h"
(...skipping 19 matching lines...) Expand all
30 enum AnimationState { 30 enum AnimationState {
31 ANIMATION_NONE, 31 ANIMATION_NONE,
32 ANIMATION_WAITING, 32 ANIMATION_WAITING,
33 ANIMATION_LOADING 33 ANIMATION_LOADING
34 }; 34 };
35 35
36 TabRenderer(); 36 TabRenderer();
37 virtual ~TabRenderer(); 37 virtual ~TabRenderer();
38 38
39 // Updates the data the Tab uses to render itself from the specified 39 // Updates the data the Tab uses to render itself from the specified
40 // TabContents. If only the loading state was updated, the loading_only flag 40 // TabContents.
41 // should be specified. If other things change, set this flag to false to 41 //
42 // update everything. 42 // See TabStripModel::TabChangedAt documentation for what loading_only means.
43 void UpdateData(TabContents* contents, bool loading_only); 43 void UpdateData(TabContents* contents, bool loading_only);
44 44
45 // Updates the display to reflect the contents of this TabRenderer's model. 45 // Updates the display to reflect the contents of this TabRenderer's model.
46 void UpdateFromModel(); 46 void UpdateFromModel();
47 47
48 // Returns true if the Tab is selected, false otherwise. 48 // Returns true if the Tab is selected, false otherwise.
49 virtual bool IsSelected() const; 49 virtual bool IsSelected() const;
50 50
51 // Advance the Loading Animation to the next frame, or hide the animation if 51 // Advance the Loading Animation to the next frame, or hide the animation if
52 // the tab isn't loading. 52 // the tab isn't loading.
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
189 189
190 bool should_display_crashed_favicon_; 190 bool should_display_crashed_favicon_;
191 191
192 static void InitClass(); 192 static void InitClass();
193 static bool initialized_; 193 static bool initialized_;
194 194
195 DISALLOW_EVIL_CONSTRUCTORS(TabRenderer); 195 DISALLOW_EVIL_CONSTRUCTORS(TabRenderer);
196 }; 196 };
197 197
198 #endif // CHROME_BROWSER_VIEWS_TABS_TAB_RENDERER_H__ 198 #endif // CHROME_BROWSER_VIEWS_TABS_TAB_RENDERER_H__
OLDNEW
« chrome/browser/tabs/tab_strip_model.h ('K') | « chrome/browser/tabs/tab_strip_model.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698