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

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

Issue 3012042: Revert 54560 - ... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 10 years, 4 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.h ('k') | chrome/browser/views/frame/browser_view.cc » ('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) 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 #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/scoped_ptr.h" 9 #include "base/scoped_ptr.h"
10 #include "base/string_util.h" 10 #include "base/string_util.h"
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 virtual bool RunUnloadListenerBeforeClosing(TabContents* contents) { 73 virtual bool RunUnloadListenerBeforeClosing(TabContents* contents) {
74 return run_unload_; 74 return run_unload_;
75 } 75 }
76 virtual bool CanRestoreTab() { return false; } 76 virtual bool CanRestoreTab() { return false; }
77 virtual void RestoreTab() {} 77 virtual void RestoreTab() {}
78 virtual bool CanCloseContentsAt(int index) { return can_close_ ; } 78 virtual bool CanCloseContentsAt(int index) { return can_close_ ; }
79 virtual bool CanBookmarkAllTabs() const { return false; } 79 virtual bool CanBookmarkAllTabs() const { return false; }
80 virtual void BookmarkAllTabs() {} 80 virtual void BookmarkAllTabs() {}
81 virtual bool UseVerticalTabs() const { return false; } 81 virtual bool UseVerticalTabs() const { return false; }
82 virtual void ToggleUseVerticalTabs() {} 82 virtual void ToggleUseVerticalTabs() {}
83 virtual bool LargeIconsPermitted() const { return true; }
84 83
85 private: 84 private:
86 // A dummy TabContents we give to callers that expect us to actually build a 85 // A dummy TabContents we give to callers that expect us to actually build a
87 // Destinations tab for them. 86 // Destinations tab for them.
88 TabContents* dummy_contents_; 87 TabContents* dummy_contents_;
89 88
90 // Whether tabs can be closed. 89 // Whether tabs can be closed.
91 bool can_close_; 90 bool can_close_;
92 91
93 // Whether to report that we need to run an unload listener before closing. 92 // Whether to report that we need to run an unload listener before closing.
(...skipping 1832 matching lines...) Expand 10 before | Expand all | Expand 10 after
1926 // Deleting the strip should delete the phanton tabcontents and result in one 1925 // Deleting the strip should delete the phanton tabcontents and result in one
1927 // TAB_CONTENTS_DESTROYED. 1926 // TAB_CONTENTS_DESTROYED.
1928 EXPECT_CALL(notification_observer, Observe(_, _, _)).Times(1); 1927 EXPECT_CALL(notification_observer, Observe(_, _, _)).Times(1);
1929 1928
1930 // Delete the tabstrip. 1929 // Delete the tabstrip.
1931 strip.reset(); 1930 strip.reset();
1932 1931
1933 // And there should have been no methods sent to the TabStripModelObserver. 1932 // And there should have been no methods sent to the TabStripModelObserver.
1934 EXPECT_EQ(0, tabstrip_observer.GetStateCount()); 1933 EXPECT_EQ(0, tabstrip_observer.GetStateCount());
1935 } 1934 }
OLDNEW
« no previous file with comments | « chrome/browser/tabs/tab_strip_model.h ('k') | chrome/browser/views/frame/browser_view.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698