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

Side by Side Diff: chrome/browser/cocoa/tab_view_picker_table_unittest.mm

Issue 2932009: Mac: Tweak content settings window. (Closed)
Patch Set: fffffffffffffffff Created 10 years, 5 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
« no previous file with comments | « chrome/browser/cocoa/tab_view_picker_table.mm ('k') | no next file » | 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 #import <Cocoa/Cocoa.h> 5 #import <Cocoa/Cocoa.h>
6 6
7 #import "chrome/browser/cocoa/tab_view_picker_table.h" 7 #import "chrome/browser/cocoa/tab_view_picker_table.h"
8 8
9 #import "base/cocoa_protocols_mac.h" 9 #import "base/cocoa_protocols_mac.h"
10 #include "base/scoped_nsobject.h" 10 #include "base/scoped_nsobject.h"
(...skipping 77 matching lines...) Expand 10 before | Expand all | Expand 10 after
88 } 88 }
89 89
90 TEST_F(TabViewPickerTableTest, TestDelegateStillWorks) { 90 TEST_F(TabViewPickerTableTest, TestDelegateStillWorks) {
91 EXPECT_FALSE(ping_.get()->didSelectItemCalled_); 91 EXPECT_FALSE(ping_.get()->didSelectItemCalled_);
92 [tabView_ selectTabViewItemAtIndex:0]; 92 [tabView_ selectTabViewItemAtIndex:0];
93 EXPECT_TRUE(ping_.get()->didSelectItemCalled_); 93 EXPECT_TRUE(ping_.get()->didSelectItemCalled_);
94 } 94 }
95 95
96 TEST_F(TabViewPickerTableTest, RowsCorrect) { 96 TEST_F(TabViewPickerTableTest, RowsCorrect) {
97 EXPECT_EQ(2, [view_ numberOfRows]); 97 EXPECT_EQ(2, [view_ numberOfRows]);
98 EXPECT_EQ(2, [[view_ dataSource] numberOfRowsInTableView:view_]); 98 EXPECT_EQ(2,
99 [[view_ dataSource] outlineView:view_ numberOfChildrenOfItem:nil]);
99 100
101 id item;
102 item = [[view_ dataSource] outlineView:view_ child:0 ofItem:nil];
100 EXPECT_TRUE([@"label 1" isEqualToString:[[view_ dataSource] 103 EXPECT_TRUE([@"label 1" isEqualToString:[[view_ dataSource]
101 tableView:view_ 104 outlineView:view_
102 objectValueForTableColumn:nil // ignored 105 objectValueForTableColumn:nil // ignored
103 row:0]]); 106 byItem:item]]);
107 item = [[view_ dataSource] outlineView:view_ child:1 ofItem:nil];
104 EXPECT_TRUE([@"label 2" isEqualToString:[[view_ dataSource] 108 EXPECT_TRUE([@"label 2" isEqualToString:[[view_ dataSource]
105 tableView:view_ 109 outlineView:view_
106 objectValueForTableColumn:nil // ignored 110 objectValueForTableColumn:nil // ignored
107 row:1]]); 111 byItem:item]]);
108 } 112 }
109 113
110 TEST_F(TabViewPickerTableTest, TestListUpdatesTabView) { 114 TEST_F(TabViewPickerTableTest, TestListUpdatesTabView) {
111 [view_ selectRowIndexes:[NSIndexSet indexSetWithIndex:0] 115 [view_ selectRowIndexes:[NSIndexSet indexSetWithIndex:0]
112 byExtendingSelection:NO]; 116 byExtendingSelection:NO];
113 EXPECT_EQ(0, [view_ selectedRow]); // sanity 117 EXPECT_EQ(0, [view_ selectedRow]); // sanity
114 EXPECT_EQ(0, [tabView_ indexOfTabViewItem:[tabView_ selectedTabViewItem]]); 118 EXPECT_EQ(0, [tabView_ indexOfTabViewItem:[tabView_ selectedTabViewItem]]);
115 } 119 }
116 120
121 TEST_F(TabViewPickerTableTest, TestWithHeadingNotEmpty) {
122 [view_ setHeading:@"disregard this"];
123
124 EXPECT_EQ(2, [view_ selectedRow]);
125
126 [tabView_ selectTabViewItemAtIndex:0];
127 EXPECT_EQ(1, [view_ selectedRow]);
128 [tabView_ selectTabViewItemAtIndex:1];
129 EXPECT_EQ(2, [view_ selectedRow]);
130
131 [view_ selectRowIndexes:[NSIndexSet indexSetWithIndex:1]
132 byExtendingSelection:NO];
133 EXPECT_EQ(1, [view_ selectedRow]); // sanity
134 EXPECT_EQ(0, [tabView_ indexOfTabViewItem:[tabView_ selectedTabViewItem]]);
135 }
136
117 } // namespace 137 } // namespace
OLDNEW
« no previous file with comments | « chrome/browser/cocoa/tab_view_picker_table.mm ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698