diff options
author | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-08 21:49:15 +0000 |
---|---|---|
committer | jhawkins@chromium.org <jhawkins@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-03-08 21:49:15 +0000 |
commit | 1cb2dac467c6b4816b6913e303e75b8853ab9bc4 (patch) | |
tree | 18c24b70b6c96667d751b283c60b7cd80a12778e | |
parent | de29ea1e64e8cd1a7b662b3aca4aaca33ec0879e (diff) | |
download | chromium_src-1cb2dac467c6b4816b6913e303e75b8853ab9bc4.zip chromium_src-1cb2dac467c6b4816b6913e303e75b8853ab9bc4.tar.gz chromium_src-1cb2dac467c6b4816b6913e303e75b8853ab9bc4.tar.bz2 |
chrome 2: string_util.h -> utf_string_conversions.h fix.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/672007
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@40945 0039d316-1c4b-4281-b951-d872f2087c98
20 files changed, 39 insertions, 17 deletions
diff --git a/chrome/browser/alternate_nav_url_fetcher.cc b/chrome/browser/alternate_nav_url_fetcher.cc index ee8c7b6..bdca7a8 100644 --- a/chrome/browser/alternate_nav_url_fetcher.cc +++ b/chrome/browser/alternate_nav_url_fetcher.cc @@ -6,6 +6,7 @@ #include "app/l10n_util.h" #include "app/resource_bundle.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/intranet_redirect_detector.h" #include "chrome/browser/profile.h" #include "chrome/browser/tab_contents/navigation_controller.h" diff --git a/chrome/browser/autocomplete/autocomplete_edit.cc b/chrome/browser/autocomplete/autocomplete_edit.cc index 74fd71f..95b4b42 100644 --- a/chrome/browser/autocomplete/autocomplete_edit.cc +++ b/chrome/browser/autocomplete/autocomplete_edit.cc @@ -7,7 +7,7 @@ #include <string> #include "base/basictypes.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/app/chrome_dll_resource.h" #include "chrome/browser/autocomplete/autocomplete_edit_view.h" #include "chrome/browser/autocomplete/autocomplete_popup_model.h" diff --git a/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc b/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc index 083f6a2..407ed10 100644 --- a/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc +++ b/chrome/browser/autocomplete/autocomplete_edit_view_gtk.cc @@ -15,7 +15,7 @@ #include "app/gfx/gtk_util.h" #include "app/l10n_util.h" #include "base/logging.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/app/chrome_dll_resource.h" #include "chrome/browser/autocomplete/autocomplete_edit.h" #include "chrome/browser/autocomplete/autocomplete_popup_model.h" diff --git a/chrome/browser/autocomplete/autocomplete_popup_view_gtk.cc b/chrome/browser/autocomplete/autocomplete_popup_view_gtk.cc index 773a809..8d42889 100644 --- a/chrome/browser/autocomplete/autocomplete_popup_view_gtk.cc +++ b/chrome/browser/autocomplete/autocomplete_popup_view_gtk.cc @@ -7,6 +7,7 @@ #include <gtk/gtk.h> #include <algorithm> +#include <string> #include "app/gfx/font.h" #include "app/gfx/gtk_util.h" @@ -15,7 +16,7 @@ #include "base/basictypes.h" #include "base/gfx/rect.h" #include "base/logging.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/autocomplete/autocomplete.h" #include "chrome/browser/autocomplete/autocomplete_edit.h" #include "chrome/browser/autocomplete/autocomplete_edit_view_gtk.h" diff --git a/chrome/browser/autocomplete/autocomplete_unittest.cc b/chrome/browser/autocomplete/autocomplete_unittest.cc index 0e8671a..2507310 100644 --- a/chrome/browser/autocomplete/autocomplete_unittest.cc +++ b/chrome/browser/autocomplete/autocomplete_unittest.cc @@ -4,6 +4,7 @@ #include "base/message_loop.h" #include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/autocomplete/autocomplete.h" #include "chrome/common/notification_registrar.h" #include "chrome/common/notification_service.h" @@ -63,7 +64,7 @@ void TestProvider::Start(const AutocompleteInput& input, } void TestProvider::Run() { - DCHECK(num_results_per_provider > 0); + DCHECK_GT(num_results_per_provider, 0U); AddResults(1, num_results_per_provider); done_ = true; DCHECK(listener_); diff --git a/chrome/browser/autocomplete/search_provider.cc b/chrome/browser/autocomplete/search_provider.cc index 2683e2c..044431b 100644 --- a/chrome/browser/autocomplete/search_provider.cc +++ b/chrome/browser/autocomplete/search_provider.cc @@ -4,11 +4,13 @@ #include "chrome/browser/autocomplete/search_provider.h" +#include <algorithm> + #include "app/l10n_util.h" #include "base/callback.h" #include "base/i18n/icu_string_conversions.h" #include "base/message_loop.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/autocomplete/keyword_provider.h" #include "chrome/browser/browser_process.h" #include "chrome/browser/google_util.h" @@ -138,7 +140,7 @@ void SearchProvider::Run() { } // We should only get here if we have a suggest url for the keyword or default // providers. - DCHECK(suggest_results_pending_ > 0); + DCHECK_GT(suggest_results_pending_, 0); } void SearchProvider::Stop() { @@ -155,7 +157,7 @@ void SearchProvider::OnURLFetchComplete(const URLFetcher* source, const std::string& data) { DCHECK(!done_); suggest_results_pending_--; - DCHECK(suggest_results_pending_ >= 0); // Should never go negative. + DCHECK_GE(suggest_results_pending_, 0); // Should never go negative. const net::HttpResponseHeaders* const response_headers = source->response_headers(); std::string json_data(data); diff --git a/chrome/browser/bug_report_util.cc b/chrome/browser/bug_report_util.cc index b238713..1f8628e 100644 --- a/chrome/browser/bug_report_util.cc +++ b/chrome/browser/bug_report_util.cc @@ -6,7 +6,7 @@ #include "app/l10n_util.h" #include "base/file_version_info.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/browser_process_impl.h" #include "chrome/browser/net/url_fetcher.h" #include "chrome/browser/profile.h" diff --git a/chrome/browser/importer/firefox2_importer.cc b/chrome/browser/importer/firefox2_importer.cc index 276144e..b56bee7 100644 --- a/chrome/browser/importer/firefox2_importer.cc +++ b/chrome/browser/importer/firefox2_importer.cc @@ -4,6 +4,9 @@ #include "chrome/browser/importer/firefox2_importer.h" +#include <string> +#include <vector> + #include "app/l10n_util.h" #include "base/file_path.h" #include "base/file_util.h" @@ -11,7 +14,7 @@ #include "base/message_loop.h" #include "base/path_service.h" #include "base/stl_util-inl.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "base/values.h" #include "chrome/browser/history/history_types.h" #include "chrome/browser/importer/firefox_importer_utils.h" diff --git a/chrome/browser/importer/firefox_importer_unittest.cc b/chrome/browser/importer/firefox_importer_unittest.cc index c2e0210..42c23ac 100644 --- a/chrome/browser/importer/firefox_importer_unittest.cc +++ b/chrome/browser/importer/firefox_importer_unittest.cc @@ -7,6 +7,7 @@ #include "base/file_path.h" #include "base/file_util.h" #include "base/path_service.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/importer/firefox2_importer.h" #include "chrome/browser/importer/firefox_importer_unittest_utils.h" #include "chrome/browser/importer/firefox_importer_utils.h" diff --git a/chrome/browser/importer/firefox_importer_utils.cc b/chrome/browser/importer/firefox_importer_utils.cc index 833892f..5ee8a13 100644 --- a/chrome/browser/importer/firefox_importer_utils.cc +++ b/chrome/browser/importer/firefox_importer_utils.cc @@ -5,10 +5,13 @@ #include "chrome/browser/importer/firefox_importer_utils.h" #include <algorithm> +#include <map> +#include <string> #include "base/file_util.h" #include "base/logging.h" #include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "base/values.h" #include "chrome/browser/search_engines/template_url.h" #include "chrome/browser/search_engines/template_url_model.h" diff --git a/chrome/browser/importer/importer_list.cc b/chrome/browser/importer/importer_list.cc index 935cc3e..4369dd6 100644 --- a/chrome/browser/importer/importer_list.cc +++ b/chrome/browser/importer/importer_list.cc @@ -8,6 +8,7 @@ #include "base/file_util.h" #include "base/stl_util-inl.h" #include "base/values.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/first_run.h" #include "chrome/browser/importer/firefox2_importer.h" #include "chrome/browser/importer/firefox3_importer.h" diff --git a/chrome/browser/importer/toolbar_importer.cc b/chrome/browser/importer/toolbar_importer.cc index 3dc301c..47b0974 100644 --- a/chrome/browser/importer/toolbar_importer.cc +++ b/chrome/browser/importer/toolbar_importer.cc @@ -7,8 +7,8 @@ #include <limits> #include "app/l10n_util.h" -#include "base/string_util.h" #include "base/rand_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/chrome_thread.h" #include "chrome/browser/first_run.h" #include "chrome/browser/importer/importer_bridge.h" diff --git a/chrome/browser/language_order_table_model.cc b/chrome/browser/language_order_table_model.cc index e5bd403e..e9c5aac 100644 --- a/chrome/browser/language_order_table_model.cc +++ b/chrome/browser/language_order_table_model.cc @@ -5,6 +5,7 @@ #include "chrome/browser/language_order_table_model.h" #include "app/l10n_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/browser_process.h" LanguageOrderTableModel::LanguageOrderTableModel() diff --git a/chrome/browser/possible_url_model.cc b/chrome/browser/possible_url_model.cc index 1f5fada..bfe3c1a 100644 --- a/chrome/browser/possible_url_model.cc +++ b/chrome/browser/possible_url_model.cc @@ -9,6 +9,7 @@ #include "app/resource_bundle.h" #include "app/table_model_observer.h" #include "base/callback.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/cancelable_request.h" #include "chrome/browser/favicon_service.h" #include "chrome/browser/pref_service.h" diff --git a/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc b/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc index 114473c..6d62b5e 100644 --- a/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc +++ b/chrome/browser/safe_browsing/safe_browsing_blocking_page.cc @@ -6,10 +6,12 @@ #include "chrome/browser/safe_browsing/safe_browsing_blocking_page.h" +#include <string> + #include "app/l10n_util.h" #include "app/resource_bundle.h" #include "base/histogram.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "base/values.h" #include "chrome/browser/chrome_thread.h" #include "chrome/browser/dom_operation_notification_details.h" diff --git a/chrome/browser/spellcheck_host.cc b/chrome/browser/spellcheck_host.cc index b7e9574..4814ba4 100644 --- a/chrome/browser/spellcheck_host.cc +++ b/chrome/browser/spellcheck_host.cc @@ -10,7 +10,7 @@ #include "base/file_util.h" #include "base/logging.h" #include "base/path_service.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/profile.h" #include "chrome/browser/net/url_request_context_getter.h" #include "chrome/browser/pref_member.h" diff --git a/chrome/browser/sync/glue/autofill_change_processor.cc b/chrome/browser/sync/glue/autofill_change_processor.cc index 2045351..bd561af 100755 --- a/chrome/browser/sync/glue/autofill_change_processor.cc +++ b/chrome/browser/sync/glue/autofill_change_processor.cc @@ -4,7 +4,10 @@ #include "chrome/browser/sync/glue/autofill_change_processor.h" -#include "base/string_util.h" +#include <string> +#include <vector> + +#include "base/utf_string_conversions.h" #include "chrome/browser/profile.h" #include "chrome/browser/sync/glue/autofill_model_associator.h" #include "chrome/browser/sync/profile_sync_service.h" @@ -161,7 +164,6 @@ void AutofillChangeProcessor::ApplyChangesFromSyncModel( std::vector<AutofillEntry> new_entries; for (int i = 0; i < change_count; ++i) { - sync_api::ReadNode sync_node(trans); if (!sync_node.InitByIdLookup(changes[i].id)) { LOG(ERROR) << "Autofill node lookup failed."; diff --git a/chrome/browser/sync/glue/preference_change_processor.cc b/chrome/browser/sync/glue/preference_change_processor.cc index 4f86c75..d363c06 100644 --- a/chrome/browser/sync/glue/preference_change_processor.cc +++ b/chrome/browser/sync/glue/preference_change_processor.cc @@ -4,8 +4,11 @@ #include "chrome/browser/sync/glue/preference_change_processor.h" +#include <set> +#include <string> + #include "base/json/json_reader.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/profile.h" #include "chrome/browser/sync/glue/preference_model_associator.h" #include "chrome/browser/sync/profile_sync_service.h" diff --git a/chrome/browser/sync/glue/preference_model_associator.cc b/chrome/browser/sync/glue/preference_model_associator.cc index 761c2db..4c72a0f 100644 --- a/chrome/browser/sync/glue/preference_model_associator.cc +++ b/chrome/browser/sync/glue/preference_model_associator.cc @@ -7,6 +7,7 @@ #include "base/json/json_reader.h" #include "base/logging.h" #include "base/values.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/pref_service.h" #include "chrome/browser/profile.h" #include "chrome/browser/sync/engine/syncapi.h" @@ -34,7 +35,6 @@ PreferenceModelAssociator::PreferenceModelAssociator( } bool PreferenceModelAssociator::AssociateModels() { - // TODO(albertb): Attempt to load the model association from storage. PrefService* pref_service = sync_service_->profile()->GetPrefs(); diff --git a/chrome/browser/sync/sync_ui_util.cc b/chrome/browser/sync/sync_ui_util.cc index 1c20d0b..a04bfe3 100644 --- a/chrome/browser/sync/sync_ui_util.cc +++ b/chrome/browser/sync/sync_ui_util.cc @@ -5,7 +5,7 @@ #include "chrome/browser/sync/sync_ui_util.h" #include "app/l10n_util.h" -#include "base/string_util.h" +#include "base/utf_string_conversions.h" #include "chrome/browser/google_service_auth_error.h" #include "chrome/browser/profile.h" #include "chrome/browser/sync/profile_sync_service.h" |