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

Side by Side Diff: chrome/browser/tabs/tab_strip_model_unittest.cc

Issue 1141005: Revert r42156, r42157, r42160. Allow dynamic switching in and out of sidetabs... (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
« no previous file with comments | « chrome/browser/tabs/tab_strip_model.cc ('k') | chrome/browser/views/frame/browser_frame.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 #include "app/system_monitor.h" 5 #include "app/system_monitor.h"
6 #include "base/file_path.h" 6 #include "base/file_path.h"
7 #include "base/file_util.h" 7 #include "base/file_util.h"
8 #include "base/path_service.h" 8 #include "base/path_service.h"
9 #include "base/string_util.h" 9 #include "base/string_util.h"
10 #include "base/stl_util-inl.h" 10 #include "base/stl_util-inl.h"
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
63 virtual void CloseFrameAfterDragSession() {} 63 virtual void CloseFrameAfterDragSession() {}
64 virtual void CreateHistoricalTab(TabContents* contents) {} 64 virtual void CreateHistoricalTab(TabContents* contents) {}
65 virtual bool RunUnloadListenerBeforeClosing(TabContents* contents) { 65 virtual bool RunUnloadListenerBeforeClosing(TabContents* contents) {
66 return run_unload_; 66 return run_unload_;
67 } 67 }
68 virtual bool CanRestoreTab() { return false; } 68 virtual bool CanRestoreTab() { return false; }
69 virtual void RestoreTab() {} 69 virtual void RestoreTab() {}
70 virtual bool CanCloseContentsAt(int index) { return can_close_ ; } 70 virtual bool CanCloseContentsAt(int index) { return can_close_ ; }
71 virtual bool CanBookmarkAllTabs() const { return false; } 71 virtual bool CanBookmarkAllTabs() const { return false; }
72 virtual void BookmarkAllTabs() {} 72 virtual void BookmarkAllTabs() {}
73 virtual bool UseVerticalTabs() const { return false; }
74 virtual void ToggleUseVerticalTabs() {}
75 73
76 private: 74 private:
77 // A dummy TabContents we give to callers that expect us to actually build a 75 // A dummy TabContents we give to callers that expect us to actually build a
78 // Destinations tab for them. 76 // Destinations tab for them.
79 TabContents* dummy_contents_; 77 TabContents* dummy_contents_;
80 78
81 // Whether tabs can be closed. 79 // Whether tabs can be closed.
82 bool can_close_; 80 bool can_close_;
83 81
84 // Whether to report that we need to run an unload listener before closing. 82 // Whether to report that we need to run an unload listener before closing.
(...skipping 1660 matching lines...) Expand 10 before | Expand all | Expand 10 after
1745 state.src_contents = old_contents3; 1743 state.src_contents = old_contents3;
1746 EXPECT_TRUE(observer.StateEquals(1, state)); 1744 EXPECT_TRUE(observer.StateEquals(1, state));
1747 EXPECT_TRUE(observer.empty()); 1745 EXPECT_TRUE(observer.empty());
1748 1746
1749 observer.ClearStates(); 1747 observer.ClearStates();
1750 } 1748 }
1751 1749
1752 // Clean up the phantom tabs. 1750 // Clean up the phantom tabs.
1753 tabstrip.CloseAllTabs(); 1751 tabstrip.CloseAllTabs();
1754 } 1752 }
OLDNEW
« no previous file with comments | « chrome/browser/tabs/tab_strip_model.cc ('k') | chrome/browser/views/frame/browser_frame.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698