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

Issue 176843022: Move UTF16ToASCII, remove WideToASCII. (Closed)

Created:
6 years, 9 months ago by brettw
Modified:
6 years, 9 months ago
Reviewers:
sky
CC:
chromium-reviews, asanka, vandebo (ex-Chrome), skanuj+watch_chromium.org, sadrul, nkostylev+watch_chromium.org, chromoting-reviews_chromium.org, tburkard+watch_chromium.org, dhollowa+watch_chromium.org, dougw+watch_chromium.org, browser-components-watch_chromium.org, chrome-speed-team+watch_google.com, yukishiino+watch_chromium.org, rginda+watch_chromium.org, joi+watch-content_chromium.org, pedrosimonetti+watch_chromium.org, aandrey+blink_chromium.org, stevenjb+watch_chromium.org, yusukes+watch_chromium.org, tim+watch_chromium.org, extensions-reviews_chromium.org, cbentzel+watch_chromium.org, vsevik, melevin+watch_chromium.org, gavinp+disk_chromium.org, estade+watch_chromium.org, benjhayden+dwatch_chromium.org, Ilya Sherman, jam, dyu1, dominich, nona+watch_chromium.org, darin-cc_chromium.org, benquan, dbeam+watch-options_chromium.org, rouslan+autofillwatch_chromium.org, devtools-reviews_chromium.org, dennis_jeffrey, chromium-apps-reviews_chromium.org, erikwright+watch_chromium.org, Dane Wallinga, dbeam+watch-ntp_chromium.org, wjia+watch_chromium.org, samarth+watch_chromium.org, fischman+watch_chromium.org, yoshiki+watch_chromium.org, haitaol+watch_chromium.org, gavinp+prer_chromium.org, paulirish+reviews_chromium.org, oshima+watch_chromium.org, kalyank, piman+watch_chromium.org, pfeldman, Jered, maniscalco+watch_chromium.org, davidben+watch_chromium.org, feature-media-reviews_chromium.org, Lei Zhang, tfarina, donnd+watch_chromium.org, mcasas+watch_chromium.org, yurys, mmenke, tommycli, dominich+watch_chromium.org, jfweitz+watch_chromium.org, David Black, ben+ash_chromium.org, Greg Billock, grt+watch_chromium.org, kmadhusu+watch_chromium.org, James Su, anantha, jshin+watch_chromium.org, davemoore+watch_chromium.org
Visibility:
Public.

Description

Move UTF16ToASCII, remove WideToASCII. This removes WideToASCII and changes all callers to use UTF16ToASCII instead. Moves UTF16ToASCII from base/strings/string_util.h to base/strings/utf_string_conversions.h and into the base namespace. Convert a few related string_util functions to take a StringPiece16 instead of a string16. Remove IsStringASCII(std::wstring) which was unused. Updates callers' includes and namespace usage accordingly. TBR=sky Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=257200

Patch Set 1 #

Patch Set 2 : #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Patch Set 6 : #

Patch Set 7 : #

Patch Set 8 : #

Patch Set 9 : #

Patch Set 10 : try failures #

Patch Set 11 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+283 lines, -251 lines) Patch
M ash/extended_desktop_unittest.cc View 1 2 3 4 5 6 3 chunks +6 lines, -5 lines 0 comments Download
M base/command_line.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M base/files/file_path.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M base/strings/string_util.h View 2 3 4 5 6 7 2 chunks +0 lines, -7 lines 0 comments Download
M base/strings/string_util.cc View 1 2 3 4 5 6 7 2 chunks +2 lines, -22 lines 0 comments Download
M base/strings/string_util_unittest.cc View 1 2 chunks +7 lines, -7 lines 0 comments Download
M base/strings/utf_string_conversions.h View 1 2 3 4 5 6 1 chunk +4 lines, -0 lines 0 comments Download
M base/strings/utf_string_conversions.cc View 1 2 3 4 5 6 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/app/client_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/autocomplete/autocomplete_input.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/background/background_contents_service.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/bookmarks/bookmark_model_unittest.cc View 2 chunks +5 lines, -5 lines 0 comments Download
M chrome/browser/chrome_browser_main.cc View 1 2 3 4 5 6 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/login/supervised_user_manager_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/memory/oom_priority_manager.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/profiles/profile_list_chromeos_unittest.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/component_updater/component_patcher_win.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/devtools/adb/android_usb_device.cc View 2 chunks +2 lines, -1 line 0 comments Download
M chrome/browser/download/download_extensions.cc View 2 chunks +2 lines, -1 line 0 comments Download
M chrome/browser/enumerate_modules_model_unittest_win.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/browser/enumerate_modules_model_win.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/extensions/api/proxy/proxy_api_helpers.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/extensions/external_pref_loader.cc View 1 2 3 4 5 6 7 8 9 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/external_registry_loader_win.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M chrome/browser/file_select_helper.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/google/google_util.cc View 1 2 3 4 5 6 7 8 9 10 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/media_galleries/media_galleries_dialog_controller_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/parsers/metadata_parser_filebase_unittest.cc View 1 2 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/profiles/profile_manager.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/rlz/rlz.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/search/search_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/search_engines/template_url_parser.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/search_engines/template_url_service.cc View 1 2 3 4 5 6 7 8 9 10 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/test/integration/search_engines_helper.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/sync/test/integration/sync_test.cc View 1 2 3 4 5 6 8 chunks +22 lines, -15 lines 0 comments Download
M chrome/browser/task_manager/task_manager_browsertest_util.cc View 1 2 3 4 5 6 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/translate/translate_browsertest.cc View 6 chunks +6 lines, -6 lines 0 comments Download
M chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc View 1 2 3 4 5 6 7 8 9 10 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/ui/autofill/data_model_wrapper.cc View 1 2 3 4 5 6 7 8 9 4 chunks +7 lines, -8 lines 0 comments Download
M chrome/browser/ui/search/instant_search_prerenderer_unittest.cc View 3 chunks +9 lines, -7 lines 0 comments Download
M chrome/browser/ui/views/bookmarks/bookmark_editor_view_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/ntp/android/promo_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/options/chromeos/cros_language_options_handler.cc View 1 2 3 chunks +6 lines, -3 lines 0 comments Download
M chrome/browser/ui/webui/options/core_options_handler.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/options/language_options_handler_common.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/signin/inline_login_handler_impl.cc View 4 chunks +5 lines, -4 lines 0 comments Download
M chrome/browser/upgrade_detector_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/chrome_version_info_win.cc View 2 chunks +2 lines, -1 line 0 comments Download
M chrome/common/custom_handlers/protocol_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/api/commands/commands_manifest_unittest.cc View 4 chunks +5 lines, -3 lines 0 comments Download
M chrome/common/extensions/command_unittest.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M chrome/common/extensions/extension_file_util.cc View 1 2 3 4 5 6 1 chunk +1 line, -1 line 0 comments Download
M chrome/installer/setup/install_worker.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/installer/setup/setup_util.cc View 1 2 3 4 5 6 2 chunks +2 lines, -1 line 0 comments Download
M chrome/installer/setup/uninstall.cc View 1 2 3 4 5 6 7 8 9 10 1 chunk +1 line, -1 line 0 comments Download
M chrome/installer/test/alternate_version_generator.cc View 1 2 2 chunks +2 lines, -1 line 0 comments Download
M chrome/installer/util/google_update_settings.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/installer/util/install_util.cc View 3 chunks +3 lines, -2 lines 0 comments Download
M chrome/installer/util/installation_state.cc View 3 chunks +4 lines, -3 lines 0 comments Download
M chrome/installer/util/installer_state.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/installer/util/installer_state_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/installer/util/move_tree_work_item_unittest.cc View 3 chunks +3 lines, -2 lines 0 comments Download
M chrome/installer/util/shell_util.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M chrome/installer/util/user_experiment.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/perf/startup_test.cc View 1 2 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/test/pyautolib/pyautolib.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/autofill/content/browser/wallet/wallet_address.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/autofill/core/browser/address.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M components/autofill/core/browser/autofill_profile.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M components/autofill/core/browser/personal_data_manager.cc View 1 2 3 chunks +3 lines, -3 lines 0 comments Download
M components/autofill/core/browser/phone_number.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/autofill/core/browser/webdata/autofill_table_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/breakpad/app/breakpad_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/wifi/wifi_test.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/browser/session_history_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/child/simple_webmimeregistry_impl.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/common/pepper_plugin_list.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M content/common/plugin_list_win.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M content/public/common/webplugininfo.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/media/android/webmediaplayer_android.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/renderer/media/webcontentdecryptionmodule_impl.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/renderer/media/webcontentdecryptionmodulesession_impl.cc View 2 chunks +2 lines, -1 line 0 comments Download
M content/renderer/media/webmediaplayer_impl.cc View 1 2 2 chunks +2 lines, -1 line 0 comments Download
M content/renderer/npapi/webplugin_impl.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/pepper/pepper_plugin_instance_impl.cc View 1 2 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/render_frame_impl.cc View 1 2 3 4 5 6 7 8 9 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/render_view_browsertest.cc View 1 2 3 2 chunks +3 lines, -2 lines 0 comments Download
M content/renderer/render_view_browsertest_mac.mm View 5 chunks +5 lines, -4 lines 0 comments Download
M content/renderer/renderer_webkitplatformsupport_impl.cc View 1 2 3 4 5 6 2 chunks +5 lines, -4 lines 0 comments Download
M content/test/image_decoder_test.cc View 1 2 2 chunks +2 lines, -1 line 0 comments Download
M device/serial/serial_device_enumerator_win.cc View 2 chunks +2 lines, -1 line 0 comments Download
M gpu/config/gpu_info_collector_win.cc View 7 chunks +9 lines, -8 lines 0 comments Download
M net/cert/x509_certificate_win.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M net/disk_cache/cache_util.cc View 2 chunks +2 lines, -1 line 0 comments Download
M net/dns/dns_config_service_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M net/proxy/proxy_config_service_win.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M net/proxy/proxy_resolver_v8.cc View 1 2 2 chunks +2 lines, -2 lines 0 comments Download
M net/proxy/proxy_resolver_v8_tracing_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M net/proxy/proxy_resolver_winhttp.cc View 1 chunk +1 line, -1 line 0 comments Download
M net/tools/gdig/gdig.cc View 1 2 3 4 5 6 2 chunks +2 lines, -1 line 0 comments Download
M remoting/host/setup/me2me_native_messaging_host.cc View 1 chunk +1 line, -1 line 0 comments Download
M skia/ext/vector_canvas_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M ui/base/l10n/l10n_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M ui/base/l10n/l10n_util_win.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M ui/views/controls/tree/tree_view_unittest.cc View 2 chunks +5 lines, -3 lines 0 comments Download
M ui/views/examples/table_example.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M webkit/browser/database/database_util.cc View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
brettw
6 years, 9 months ago (2014-03-06 18:57:05 UTC) #1
brettw
The CQ bit was checked by brettw@chromium.org
6 years, 9 months ago (2014-03-14 05:00:02 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/brettw@chromium.org/176843022/150001
6 years, 9 months ago (2014-03-14 05:00:58 UTC) #3
commit-bot: I haz the power
The CQ bit was unchecked by commit-bot@chromium.org
6 years, 9 months ago (2014-03-14 05:01:44 UTC) #4
commit-bot: I haz the power
Failed to apply patch for chrome/browser/ui/autofill/autofill_dialog_controller_impl.cc: While running patch -p1 --forward --force --no-backup-if-mismatch; patching file ...
6 years, 9 months ago (2014-03-14 05:01:44 UTC) #5
brettw
6 years, 9 months ago (2014-03-14 21:12:07 UTC) #6
Message was sent while issue was closed.
Committed patchset #11 manually as r257200 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698