summaryrefslogtreecommitdiffstats
path: root/chrome/browser/cocoa/preferences_window_controller.mm
diff options
context:
space:
mode:
authorthakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-12-03 21:45:06 +0000
committerthakis@chromium.org <thakis@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2009-12-03 21:45:06 +0000
commit37d39aa90e3121030f129407a82d53e50a85837d (patch)
tree1be396760ac136abcd0d1397b3d8b8a127c59bc0 /chrome/browser/cocoa/preferences_window_controller.mm
parent340943147a0cae34ab3dd20f0b97def51abab24e (diff)
downloadchromium_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.mm15
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];
}