| OLD | NEW |
| 1 // Copyright (c) 2009 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2009 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 "base/scoped_nsobject.h" | 5 #include "base/scoped_nsobject.h" |
| 6 #include "base/scoped_ptr.h" | 6 #include "base/scoped_ptr.h" |
| 7 #include "base/string_util.h" | 7 #include "base/string_util.h" |
| 8 #include "chrome/browser/cocoa/location_bar_view_mac.h" | 8 #include "chrome/browser/cocoa/location_bar_view_mac.h" |
| 9 #include "testing/gtest/include/gtest/gtest.h" | 9 #include "testing/gtest/include/gtest/gtest.h" |
| 10 | 10 |
| 11 // TODO(shess): Figure out how to unittest this. The code below was |
| 12 // testing the hacked-up behavior so you didn't have to be pedantic |
| 13 // WRT http://. But that approach is completely and utterly wrong in |
| 14 // a world where omnibox is running. |
| 15 // http://code.google.com/p/chromium/issues/detail?id=9977 |
| 16 |
| 17 #if 0 |
| 11 class LocationBarViewMacTest : public testing::Test { | 18 class LocationBarViewMacTest : public testing::Test { |
| 12 public: | 19 public: |
| 13 LocationBarViewMacTest() | 20 LocationBarViewMacTest() |
| 14 : field_([[NSTextField alloc] init]), | 21 : field_([[NSTextField alloc] init]), |
| 15 locationBarView_(new LocationBarViewMac(field_)) { | 22 locationBarView_(new LocationBarViewMac(field_, NULL, NULL)) { |
| 16 } | 23 } |
| 17 | 24 |
| 18 scoped_nsobject<NSTextField> field_; | 25 scoped_nsobject<NSTextField> field_; |
| 19 scoped_ptr<LocationBarViewMac> locationBarView_; | 26 scoped_ptr<LocationBarViewMac> locationBarView_; |
| 20 }; | 27 }; |
| 21 | 28 |
| 22 TEST_F(LocationBarViewMacTest, GetInputString) { | 29 TEST_F(LocationBarViewMacTest, GetInputString) { |
| 23 // Test a few obvious cases to make sure things work end-to-end, but | 30 // Test a few obvious cases to make sure things work end-to-end, but |
| 24 // trust url_fixer_upper_unittest.cc to do the bulk of the work. | 31 // trust url_fixer_upper_unittest.cc to do the bulk of the work. |
| 25 [field_ setStringValue:@"ahost"]; | 32 [field_ setStringValue:@"ahost"]; |
| 26 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://ahost/")); | 33 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://ahost/")); |
| 27 | 34 |
| 28 [field_ setStringValue:@"bhost\n"]; | 35 [field_ setStringValue:@"bhost\n"]; |
| 29 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://bhost/")); | 36 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://bhost/")); |
| 30 | 37 |
| 31 [field_ setStringValue:@"chost/"]; | 38 [field_ setStringValue:@"chost/"]; |
| 32 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://chost/")); | 39 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://chost/")); |
| 33 | 40 |
| 34 [field_ setStringValue:@"www.example.com"]; | 41 [field_ setStringValue:@"www.example.com"]; |
| 35 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://www.example.
com/")); | 42 EXPECT_EQ(locationBarView_->GetInputString(), |
| 43 ASCIIToWide("http://www.example.com/")); |
| 36 | 44 |
| 37 [field_ setStringValue:@"http://example.com"]; | 45 [field_ setStringValue:@"http://example.com"]; |
| 38 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("http://example.com/
")); | 46 EXPECT_EQ(locationBarView_->GetInputString(), |
| 47 ASCIIToWide("http://example.com/")); |
| 39 | 48 |
| 40 [field_ setStringValue:@"https://www.example.com"]; | 49 [field_ setStringValue:@"https://www.example.com"]; |
| 41 EXPECT_EQ(locationBarView_->GetInputString(), ASCIIToWide("https://www.example
.com/")); | 50 EXPECT_EQ(locationBarView_->GetInputString(), |
| 51 ASCIIToWide("https://www.example.com/")); |
| 42 } | 52 } |
| 53 #endif |
| OLD | NEW |