diff options
author | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-03 21:45:06 +0000 |
---|---|---|
committer | thakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-12-03 21:45:06 +0000 |
commit | 37d39aa90e3121030f129407a82d53e50a85837d (patch) | |
tree | 1be396760ac136abcd0d1397b3d8b8a127c59bc0 /chrome/browser/cocoa/preferences_window_controller.mm | |
parent | 340943147a0cae34ab3dd20f0b97def51abab24e (diff) | |
download | chromium_src-37d39aa90e3121030f129407a82d53e50a85837d.zip chromium_src-37d39aa90e3121030f129407a82d53e50a85837d.tar.gz chromium_src-37d39aa90e3121030f129407a82d53e50a85837d.tar.bz2 |
Remove the deprecated version of FixupURL and fix the callers.
BUG=None
TEST=None
Review URL: http://codereview.chromium.org/455022
Patch from tfarina.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@33728 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/cocoa/preferences_window_controller.mm')
-rw-r--r-- | chrome/browser/cocoa/preferences_window_controller.mm | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/chrome/browser/cocoa/preferences_window_controller.mm b/chrome/browser/cocoa/preferences_window_controller.mm index bcfb2f1..dbedda9a 100644 --- a/chrome/browser/cocoa/preferences_window_controller.mm +++ b/chrome/browser/cocoa/preferences_window_controller.mm @@ -48,9 +48,8 @@ namespace { -std::wstring GetNewTabUIURLString() { - std::wstring temp = UTF8ToWide(chrome::kChromeUINewTabURL); - return URLFixerUpper::FixupURL(temp, std::wstring()); +std::string GetNewTabUIURLString() { + return URLFixerUpper::FixupURL(chrome::kChromeUINewTabURL, std::string()); } // Helper to remove all but the last view from the view heirarchy. @@ -785,12 +784,12 @@ class PrefObserverBridge : public NotificationObserver, // observers not to fire, which is actually a good thing as we could end up in a // state where setting the homepage to an empty url would automatically reset // the prefs back to using the NTP, so we'd be never be able to change it. -- (void)setHomepage:(const std::wstring&)homepage { +- (void)setHomepage:(const std::string&)homepage { if (homepage.empty() || homepage == GetNewTabUIURLString()) { newTabPageIsHomePage_.SetValue(true); } else { newTabPageIsHomePage_.SetValue(false); - homepage_.SetValue(homepage); + homepage_.SetValue(UTF8ToWide(homepage)); } } @@ -993,9 +992,9 @@ enum { kHomepageNewTabPage, kHomepageURL }; // to something valid ("http://google.com"). if (!urlString) urlString = [NSString stringWithFormat:@"%s", chrome::kChromeUINewTabURL]; - std::wstring temp = base::SysNSStringToWide(urlString); - std::wstring fixedString = URLFixerUpper::FixupURL(temp, std::wstring()); - if (GURL(WideToUTF8(fixedString)).is_valid()) + std::string temp = base::SysNSStringToUTF8(urlString); + std::string fixedString = URLFixerUpper::FixupURL(temp, std::string()); + if (GURL(fixedString).is_valid()) [self setHomepage:fixedString]; } |