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

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

Issue 1751018: Remove unused local variables now that subclasses don't need to track their o... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 10 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) 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_VIEWS_TABS_TAB_STRIP_H_ 5 #ifndef CHROME_BROWSER_VIEWS_TABS_TAB_STRIP_H_
6 #define CHROME_BROWSER_VIEWS_TABS_TAB_STRIP_H_ 6 #define CHROME_BROWSER_VIEWS_TABS_TAB_STRIP_H_
7 7
8 #include "app/animation_container.h" 8 #include "app/animation_container.h"
9 #include "base/message_loop.h" 9 #include "base/message_loop.h"
10 #include "base/ref_counted.h" 10 #include "base/ref_counted.h"
(...skipping 412 matching lines...) Expand 10 before | Expand all | Expand 10 after
423 423
424 // If this value is nonnegative, it is used in GetDesiredTabWidths() to 424 // If this value is nonnegative, it is used in GetDesiredTabWidths() to
425 // calculate how much space in the tab strip to use for tabs. Most of the 425 // calculate how much space in the tab strip to use for tabs. Most of the
426 // time this will be -1, but while we're handling closing a tab via the mouse, 426 // time this will be -1, but while we're handling closing a tab via the mouse,
427 // we'll set this to the edge of the last tab before closing, so that if we 427 // we'll set this to the edge of the last tab before closing, so that if we
428 // are closing the last tab and need to resize immediately, we'll resize only 428 // are closing the last tab and need to resize immediately, we'll resize only
429 // back to this width, thus once again placing the last tab under the mouse 429 // back to this width, thus once again placing the last tab under the mouse
430 // cursor. 430 // cursor.
431 int available_width_for_tabs_; 431 int available_width_for_tabs_;
432 432
433 // Storage of strings needed for accessibility.
434 std::wstring accessible_name_;
435
436 // The size of the new tab button must be hardcoded because we need to be 433 // The size of the new tab button must be hardcoded because we need to be
437 // able to lay it out before we are able to get its image from the 434 // able to lay it out before we are able to get its image from the
438 // ThemeProvider. It also makes sense to do this, because the size of the 435 // ThemeProvider. It also makes sense to do this, because the size of the
439 // new tab button should not need to be calculated dynamically. 436 // new tab button should not need to be calculated dynamically.
440 static const int kNewTabButtonWidth = 28; 437 static const int kNewTabButtonWidth = 28;
441 static const int kNewTabButtonHeight = 18; 438 static const int kNewTabButtonHeight = 18;
442 439
443 // Valid for the lifetime of a drag over us. 440 // Valid for the lifetime of a drag over us.
444 scoped_ptr<DropInfo> drop_info_; 441 scoped_ptr<DropInfo> drop_info_;
445 442
(...skipping 12 matching lines...) Expand all
458 // Used for stage 1 of new tab animation. 455 // Used for stage 1 of new tab animation.
459 base::OneShotTimer<TabStrip> new_tab_timer_; 456 base::OneShotTimer<TabStrip> new_tab_timer_;
460 457
461 // Set for special animations. 458 // Set for special animations.
462 AnimationType animation_type_; 459 AnimationType animation_type_;
463 460
464 DISALLOW_COPY_AND_ASSIGN(TabStrip); 461 DISALLOW_COPY_AND_ASSIGN(TabStrip);
465 }; 462 };
466 463
467 #endif // CHROME_BROWSER_VIEWS_TABS_TAB_STRIP_H_ 464 #endif // CHROME_BROWSER_VIEWS_TABS_TAB_STRIP_H_
OLDNEW
« no previous file with comments | « chrome/browser/views/tab_contents/tab_contents_container.h ('k') | views/controls/button/button.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698