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

Side by Side Diff: chrome/browser/notifications/notification_exceptions_table_model_unittest.cc

Issue 6051012: Revert 70271 - Remove wstring from l10n_util. Part 3.... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years, 11 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
Property Changes:
Deleted: svn:eol-style
- LF
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 "chrome/browser/notifications/notification_exceptions_table_model.h" 5 #include "chrome/browser/notifications/notification_exceptions_table_model.h"
6 6
7 #include "app/l10n_util.h" 7 #include "app/l10n_util.h"
8 #include "base/utf_string_conversions.h"
9 #include "chrome/browser/browser_thread.h" 8 #include "chrome/browser/browser_thread.h"
10 #include "chrome/browser/renderer_host/test/test_render_view_host.h" 9 #include "chrome/browser/renderer_host/test/test_render_view_host.h"
11 #include "chrome/test/testing_profile.h" 10 #include "chrome/test/testing_profile.h"
12 #include "grit/generated_resources.h" 11 #include "grit/generated_resources.h"
13 #include "testing/gtest/include/gtest/gtest.h" 12 #include "testing/gtest/include/gtest/gtest.h"
14 13
15 class NotificationExceptionsTableModelTest : public RenderViewHostTestHarness { 14 class NotificationExceptionsTableModelTest : public RenderViewHostTestHarness {
16 public: 15 public:
17 NotificationExceptionsTableModelTest() 16 NotificationExceptionsTableModelTest()
18 : ui_thread_(BrowserThread::UI, MessageLoop::current()) { 17 : ui_thread_(BrowserThread::UI, MessageLoop::current()) {
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 EXPECT_EQ(0u, service_->GetAllowedOrigins().size()); 68 EXPECT_EQ(0u, service_->GetAllowedOrigins().size());
70 EXPECT_EQ(0u, service_->GetBlockedOrigins().size()); 69 EXPECT_EQ(0u, service_->GetBlockedOrigins().size());
71 } 70 }
72 71
73 TEST_F(NotificationExceptionsTableModelTest, AlphabeticalOrder) { 72 TEST_F(NotificationExceptionsTableModelTest, AlphabeticalOrder) {
74 FillData(); 73 FillData();
75 EXPECT_EQ(5, model_->RowCount()); 74 EXPECT_EQ(5, model_->RowCount());
76 75
77 EXPECT_EQ(L"allowed.com", 76 EXPECT_EQ(L"allowed.com",
78 model_->GetText(0, IDS_EXCEPTIONS_HOSTNAME_HEADER)); 77 model_->GetText(0, IDS_EXCEPTIONS_HOSTNAME_HEADER));
79 EXPECT_EQ(UTF16ToWideHack( 78 EXPECT_EQ(l10n_util::GetString(IDS_EXCEPTIONS_ALLOW_BUTTON),
80 l10n_util::GetStringUTF16(IDS_EXCEPTIONS_ALLOW_BUTTON)),
81 model_->GetText(0, IDS_EXCEPTIONS_ACTION_HEADER)); 79 model_->GetText(0, IDS_EXCEPTIONS_ACTION_HEADER));
82 80
83 EXPECT_EQ(L"denied.com", 81 EXPECT_EQ(L"denied.com",
84 model_->GetText(1, IDS_EXCEPTIONS_HOSTNAME_HEADER)); 82 model_->GetText(1, IDS_EXCEPTIONS_HOSTNAME_HEADER));
85 EXPECT_EQ(UTF16ToWideHack( 83 EXPECT_EQ(l10n_util::GetString(IDS_EXCEPTIONS_BLOCK_BUTTON),
86 l10n_util::GetStringUTF16(IDS_EXCEPTIONS_BLOCK_BUTTON)),
87 model_->GetText(1, IDS_EXCEPTIONS_ACTION_HEADER)); 84 model_->GetText(1, IDS_EXCEPTIONS_ACTION_HEADER));
88 85
89 EXPECT_EQ(L"denied2.com", 86 EXPECT_EQ(L"denied2.com",
90 model_->GetText(2, IDS_EXCEPTIONS_HOSTNAME_HEADER)); 87 model_->GetText(2, IDS_EXCEPTIONS_HOSTNAME_HEADER));
91 EXPECT_EQ(UTF16ToWideHack( 88 EXPECT_EQ(l10n_util::GetString(IDS_EXCEPTIONS_BLOCK_BUTTON),
92 l10n_util::GetStringUTF16(IDS_EXCEPTIONS_BLOCK_BUTTON)),
93 model_->GetText(2, IDS_EXCEPTIONS_ACTION_HEADER)); 89 model_->GetText(2, IDS_EXCEPTIONS_ACTION_HEADER));
94 90
95 EXPECT_EQ(L"e-allowed2.com", 91 EXPECT_EQ(L"e-allowed2.com",
96 model_->GetText(3, IDS_EXCEPTIONS_HOSTNAME_HEADER)); 92 model_->GetText(3, IDS_EXCEPTIONS_HOSTNAME_HEADER));
97 EXPECT_EQ(UTF16ToWideHack( 93 EXPECT_EQ(l10n_util::GetString(IDS_EXCEPTIONS_ALLOW_BUTTON),
98 l10n_util::GetStringUTF16(IDS_EXCEPTIONS_ALLOW_BUTTON)),
99 model_->GetText(3, IDS_EXCEPTIONS_ACTION_HEADER)); 94 model_->GetText(3, IDS_EXCEPTIONS_ACTION_HEADER));
100 95
101 EXPECT_EQ(L"f-denied3.com", 96 EXPECT_EQ(L"f-denied3.com",
102 model_->GetText(4, IDS_EXCEPTIONS_HOSTNAME_HEADER)); 97 model_->GetText(4, IDS_EXCEPTIONS_HOSTNAME_HEADER));
103 EXPECT_EQ(UTF16ToWideHack( 98 EXPECT_EQ(l10n_util::GetString(IDS_EXCEPTIONS_BLOCK_BUTTON),
104 l10n_util::GetStringUTF16(IDS_EXCEPTIONS_BLOCK_BUTTON)),
105 model_->GetText(4, IDS_EXCEPTIONS_ACTION_HEADER)); 99 model_->GetText(4, IDS_EXCEPTIONS_ACTION_HEADER));
106 } 100 }
107 101
108 TEST_F(NotificationExceptionsTableModelTest, RemoveRows) { 102 TEST_F(NotificationExceptionsTableModelTest, RemoveRows) {
109 FillData(); 103 FillData();
110 EXPECT_EQ(5, model_->RowCount()); 104 EXPECT_EQ(5, model_->RowCount());
111 105
112 { 106 {
113 RemoveRowsTableModel::Rows rows; 107 RemoveRowsTableModel::Rows rows;
114 rows.insert(0); // allowed.com 108 rows.insert(0); // allowed.com
115 rows.insert(3); // e-allowed2.com 109 rows.insert(3); // e-allowed2.com
116 model_->RemoveRows(rows); 110 model_->RemoveRows(rows);
117 } 111 }
118 EXPECT_EQ(3, model_->RowCount()); 112 EXPECT_EQ(3, model_->RowCount());
119 EXPECT_EQ(0u, service_->GetAllowedOrigins().size()); 113 EXPECT_EQ(0u, service_->GetAllowedOrigins().size());
120 EXPECT_EQ(3u, service_->GetBlockedOrigins().size()); 114 EXPECT_EQ(3u, service_->GetBlockedOrigins().size());
121 115
122 { 116 {
123 RemoveRowsTableModel::Rows rows; 117 RemoveRowsTableModel::Rows rows;
124 rows.insert(0); 118 rows.insert(0);
125 rows.insert(1); 119 rows.insert(1);
126 rows.insert(2); 120 rows.insert(2);
127 model_->RemoveRows(rows); 121 model_->RemoveRows(rows);
128 } 122 }
129 EXPECT_EQ(0, model_->RowCount()); 123 EXPECT_EQ(0, model_->RowCount());
130 EXPECT_EQ(0u, service_->GetAllowedOrigins().size()); 124 EXPECT_EQ(0u, service_->GetAllowedOrigins().size());
131 EXPECT_EQ(0u, service_->GetBlockedOrigins().size()); 125 EXPECT_EQ(0u, service_->GetBlockedOrigins().size());
132 } 126 }
OLDNEW
« no previous file with comments | « chrome/browser/notifications/notification_exceptions_table_model.cc ('k') | chrome/browser/omnibox_search_hint.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698