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

Issue 1214183008: Update StartsWith calls to use new versions in chrome and content (Closed)

Created:
5 years, 5 months ago by brettw
Modified:
5 years, 5 months ago
CC:
chromium-reviews, skanuj+watch_chromium.org, felt, yusukes+watch_chromium.org, zea+watch_chromium.org, tzik, posciak+watch_chromium.org, phoglund+watch_chromium.org, vabr+watchlist_chromium.org, maxbogue+watch_chromium.org, browser-components-watch_chromium.org, rginda+watch_chromium.org, pvalenzuela+watch_chromium.org, stevenjb+watch_chromium.org, tim+watch_chromium.org, extensions-reviews_chromium.org, cbentzel+watch_chromium.org, melevin+watch_chromium.org, Matt Giuca, shishir+watch_chromium.org, mvanouwerkerk+watch_chromium.org, grt+watch_chromium.org, samuong+watch_chromium.org, dzhioev+watch_chromium.org, jam, plaree+watch_chromium.org, nona+watch_chromium.org, Jered, dhollowa+watch_chromium.org, chromium-apps-reviews_chromium.org, tnakamura+watch_chromium.org, maniscalco+watch_chromium.org, johnme+watch_chromium.org, pedrosimonetti+watch_chromium.org, dbeam+watch-ntp_chromium.org, derat+watch_chromium.org, tapted, tburkard+watch_chromium.org, dougw+watch_chromium.org, nhiroki, feature-media-reviews_chromium.org, jfweitz+watch_chromium.org, pam+watch_chromium.org, oshima+watch_chromium.org, kmadhusu+watch_chromium.org, gcasto+watchlist_chromium.org, rouslan+autofillwatch_chromium.org, mkwst+watchlist-passwords_chromium.org, davidben+watch_chromium.org, devtools-reviews_chromium.org, media-router+watch_chromium.org, Lei Zhang, tfarina, donnd+watch_chromium.org, shuchen+watch_chromium.org, mcasas+watch_chromium.org, yurys, asvitkine+watch_chromium.org, tommycli, gavinp+prer_chromium.org, David Black, pfeldman, samarth+watch_chromium.org, estade+watch_chromium.org, peter+watch_chromium.org, wjia+watch_chromium.org, kinuko+fileapi, davemoore+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Update StartsWith calls to use new versions in chrome and content true -> base::CompareCase::SENSITIVE false with punctuation only -> base::CompareCase::SENSITIVE false with ascii constants -> base::CompareCase::INSENSITIVE_ASCII false with arbitrary string16 -> base::CompareCase::SENSITIVE with base::i18n::ToLower() on the arguments. BUG=506920, 506255 Committed: https://crrev.com/66d1b81be3d04396da1e92026274a03fa627e933 Cr-Commit-Position: refs/heads/master@{#337439}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+418 lines, -271 lines) Patch
M chrome/browser/about_flags.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/android/preferences/website_preference_bridge.cc View 1 chunk +8 lines, -4 lines 0 comments Download
M chrome/browser/apps/guest_view/web_view_browsertest.cc View 4 chunks +10 lines, -5 lines 0 comments Download
M chrome/browser/autofill/autofill_browsertest.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/chromeos_utils.cc View 1 chunk +6 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/file_manager/file_tasks.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/input_method/input_method_engine.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/input_method/input_method_manager_impl.cc View 1 chunk +9 lines, -5 lines 0 comments Download
M chrome/browser/chromeos/input_method/input_method_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/login/wizard_controller_browsertest.cc View 1 chunk +5 lines, -4 lines 0 comments Download
M chrome/browser/chromeos/policy/user_cloud_policy_manager_chromeos_unittest.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/policy/variations_service_policy_browsertest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/power/peripheral_battery_observer.cc View 1 chunk +4 lines, -2 lines 0 comments Download
M chrome/browser/chromeos/proxy_cros_settings_parser.cc View 3 chunks +6 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/settings/cros_settings.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/system_logs/touch_log_source_ozone.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/activity_log/activity_actions.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/browser/extensions/activity_log/activity_log.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/api/commands/command_service.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/api/extension_action/extension_action_api.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/api/feedback_private/feedback_private_api.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/api/identity/gaia_web_auth_flow.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/extensions/api/identity/identity_apitest.cc View 9 chunks +18 lines, -9 lines 0 comments Download
M chrome/browser/extensions/blob_reader.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/chrome_url_request_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/extension_apitest.cc View 4 chunks +8 lines, -4 lines 0 comments Download
M chrome/browser/extensions/extension_install_ui_browsertest.cc View 2 chunks +6 lines, -4 lines 0 comments Download
M chrome/browser/extensions/extension_management.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/extensions/extension_protocols_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/extensions/extension_request_limiting_throttle_browsertest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M chrome/browser/extensions/extension_web_ui.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/isolated_app_browsertest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/google/google_brand.cc View 2 chunks +5 lines, -5 lines 0 comments Download
M chrome/browser/local_discovery/device_description.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/media/router/media_source_helper.cc View 1 chunk +6 lines, -3 lines 0 comments Download
M chrome/browser/media/webrtc_browsertest_base.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/media/webrtc_browsertest_common.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/media_galleries/fileapi/media_file_system_backend.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M chrome/browser/media_galleries/fileapi/native_media_file_util.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/browser/media_galleries/media_folder_finder.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/metrics/variations/variations_service_unittest.cc View 1 chunk +6 lines, -3 lines 0 comments Download
M chrome/browser/net/async_dns_field_trial.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/net/chrome_network_delegate.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/browser/net/safe_search_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/password_manager/password_manager_browsertest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/plugins/plugin_info_message_filter.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/policy/policy_browsertest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/policy/policy_prefs_browsertest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/predictors/autocomplete_action_predictor.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/predictors/autocomplete_action_predictor_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/prefetch/prefetch_field_trial.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/prefs/tracked/pref_hash_browsertest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/prerender/prerender_util.cc View 1 chunk +5 lines, -3 lines 0 comments Download
M chrome/browser/profile_resetter/automatic_profile_resetter.cc View 1 chunk +5 lines, -4 lines 0 comments Download
M chrome/browser/profiles/profile_io_data.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/push_messaging/push_messaging_app_identifier.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate_browsertest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/safe_browsing/protocol_manager.cc View 1 chunk +6 lines, -3 lines 0 comments Download
M chrome/browser/safe_browsing/safe_browsing_service_browsertest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/search/local_ntp_source.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/supervised_user/experimental/supervised_user_async_url_checker.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/supervised_user/supervised_user_settings_service.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/supervised_user/supervised_user_settings_service_unittest.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M chrome/browser/supervised_user/supervised_user_url_filter.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc View 4 chunks +15 lines, -10 lines 0 comments Download
M chrome/browser/sync/test/integration/extensions_helper.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/sync_file_system/drive_backend/drive_backend_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync_file_system/drive_backend/metadata_database_index_on_disk.cc View 5 chunks +9 lines, -8 lines 0 comments Download
M chrome/browser/sync_file_system/drive_backend/metadata_db_migration_util.cc View 1 chunk +24 lines, -14 lines 0 comments Download
M chrome/browser/sync_file_system/drive_backend/metadata_db_migration_util_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/app_list/app_list_syncable_service.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/app_list/search/search_controller_factory.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/ui/browser_navigator_browsertest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/ui/omnibox/omnibox_view.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/prefs/prefs_tab_helper.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M chrome/browser/ui/search/instant_extended_manual_interactive_uitest.cc View 1 chunk +4 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/devtools_ui.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/extensions/chromeos/kiosk_apps_handler.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/interstitials/interstitial_ui.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/browser/ui/webui/media_router/media_router_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/ntp/favicon_webui_handler.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/web_applications/web_app_mac.mm View 2 chunks +4 lines, -3 lines 0 comments Download
M chrome/common/crash_keys.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/common/extensions/api/file_browser_handlers/file_browser_handler.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/common/extensions/chrome_extensions_client.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M chrome/common/extensions/manifest_handlers/settings_overrides_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/extensions/manifest_tests/extension_manifests_homepage_unittest.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M chrome/common/pref_names_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/renderer/chrome_content_renderer_client.cc View 1 chunk +8 lines, -4 lines 0 comments Download
M chrome/renderer/content_settings_observer.cc View 6 chunks +23 lines, -18 lines 0 comments Download
M chrome/renderer/extensions/app_bindings.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/renderer/extensions/extension_localization_peer.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/renderer/extensions/webstore_bindings.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/renderer/page_load_histograms.cc View 1 chunk +10 lines, -5 lines 0 comments Download
M chrome/renderer/searchbox/searchbox_extension.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/service/cloud_print/cloud_print_connector.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/test/chromedriver/chrome/web_view_impl.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/test/chromedriver/performance_logger.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/test/chromedriver/server/http_handler.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/utility/importer/bookmark_html_reader.cc View 7 chunks +9 lines, -7 lines 0 comments Download
M chrome/utility/media_galleries/media_metadata_parser.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M chrome/utility/media_galleries/picasa_album_table_reader.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/omnibox/builtin_provider.cc View 3 chunks +12 lines, -7 lines 0 comments Download
M components/omnibox/shortcuts_backend.cc View 2 chunks +4 lines, -3 lines 0 comments Download
M components/omnibox/shortcuts_provider.cc View 4 chunks +8 lines, -4 lines 0 comments Download
M components/omnibox/url_index_private_data.cc View 1 chunk +4 lines, -1 line 0 comments Download

Messages

Total messages: 11 (4 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1214183008/1
5 years, 5 months ago (2015-07-04 04:43:12 UTC) #2
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 5 months ago (2015-07-04 05:49:24 UTC) #4
brettw
5 years, 5 months ago (2015-07-06 17:37:20 UTC) #6
Ken Rockot(use gerrit already)
lgtm
5 years, 5 months ago (2015-07-06 17:50:49 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1214183008/1
5 years, 5 months ago (2015-07-06 18:15:54 UTC) #9
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 5 months ago (2015-07-06 19:29:56 UTC) #10
commit-bot: I haz the power
5 years, 5 months ago (2015-07-06 19:31:41 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/66d1b81be3d04396da1e92026274a03fa627e933
Cr-Commit-Position: refs/heads/master@{#337439}

Powered by Google App Engine
This is Rietveld 408576698