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

Issue 1233043003: Remove some legacy versions of StartsWith and EndsWith. (Closed)

Created:
5 years, 5 months ago by brettw
Modified:
5 years, 5 months ago
Reviewers:
jam
CC:
chromium-reviews, skanuj+watch_chromium.org, oshima+watch_chromium.org, zea+watch_chromium.org, viettrungluu+watch_chromium.org, posciak+watch_chromium.org, dhollowa+watch_chromium.org, dougw+watch_chromium.org, maxbogue+watch_chromium.org, yzshen+watch_chromium.org, dmazzoni+watch_chromium.org, stevenjb+watch_chromium.org, miu+watch_chromium.org, tim+watch_chromium.org, extensions-reviews_chromium.org, cbentzel+watch_chromium.org, melevin+watch_chromium.org, Matt Giuca, Aaron Boodman, aboxhall+watch_chromium.org, grt+watch_chromium.org, chromoting-reviews_chromium.org, jam, ben+mojo_chromium.org, abarth-chromium, pvalenzuela+watch_chromium.org, je_julie, darin-cc_chromium.org, rouslan+autofillwatch_chromium.org, devtools-reviews_chromium.org, chromium-apps-reviews_chromium.org, erikwright+watch_chromium.org, vabr+watchlist_chromium.org, nasko+codewatch_chromium.org, creis+watch_chromium.org, dtseng+watch_chromium.org, tapted, qsr+mojo_chromium.org, yuzo+watch_chromium.org, feature-media-reviews_chromium.org, jfweitz+watch_chromium.org, pam+watch_chromium.org, asvitkine+watch_chromium.org, estade+watch_chromium.org, piman+watch_chromium.org, gcasto+watchlist_chromium.org, pfeldman, Jered, maniscalco+watch_chromium.org, plundblad+watch_chromium.org, mkwst+watchlist-passwords_chromium.org, tfarina, donnd+watch_chromium.org, nektar+watch_chromium.org, mcasas+watch_chromium.org, yurys, plaree+watch_chromium.org, David Black, davemoore+watch_chromium.org, samarth+watch_chromium.org, wfh+watch_chromium.org, kmadhusu+watch_chromium.org, darin (slow to review), James Su, wjia+watch_chromium.org, jshin+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove some legacy versions of StartsWith and EndsWith. This just replaces true -> base::CompareCase::SENSITIVE false -> base::CompareCase::INSENSITIVE_ASCII I checked the insensitive cases to make sure they're not doing anything suspicious. The old version is a sometimes-correct Unicode comparison so converting to INSENSTITIVE_ASCII isn't a no-op. However, generally the prefix/suffix checking is done against a hardcoded string so there were very few cases to actually look at. extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc has a not-quite search-and-replace change where I changed the type of a class variable. BUG=506255 TBR=jam Reland of http://crrev.com/1239493005 Committed: https://crrev.com/a7ff1b291cd328e9584779a4bb9101fa9a3c01ad Cr-Commit-Position: refs/heads/master@{#339071}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+326 lines, -225 lines) Patch
M base/android/library_loader/library_prefetcher.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M base/strings/string_util.h View 1 chunk +0 lines, -13 lines 0 comments Download
M base/strings/string_util_unittest.cc View 1 chunk +69 lines, -45 lines 0 comments Download
M base/test/test_suite.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/app/delay_load_hook_win.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/banners/app_banner_data_fetcher.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/extensions/wallpaper_private_api.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/chromeos/policy/device_local_account.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/component_updater/cld_component_installer_unittest.cc View 2 chunks +6 lines, -3 lines 0 comments Download
M chrome/browser/extensions/api/web_navigation/web_navigation_apitest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/default_apps.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/extensions/updater/local_extension_cache.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/metrics/chromeos_metrics_provider.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/net/pref_proxy_config_tracker_impl.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/safe_browsing/safe_browsing_database.cc View 2 chunks +17 lines, -9 lines 0 comments Download
M chrome/browser/search/iframe_source.cc View 1 chunk +4 lines, -4 lines 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_url_filter.cc View 2 chunks +3 lines, -2 lines 0 comments Download
M chrome/browser/sync/test/integration/bookmarks_helper.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/autofill/autofill_dialog_controller_unittest.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M chrome/browser/ui/cocoa/omnibox/omnibox_popup_cell.mm View 1 chunk +4 lines, -2 lines 0 comments Download
M chrome/browser/ui/libgtk2ui/select_file_dialog_impl_gtk2.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/passwords/manage_passwords_view_utils_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/devtools_ui.cc View 1 chunk +13 lines, -7 lines 0 comments Download
M chrome/browser/ui/webui/print_preview/print_preview_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/profiler_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/web_applications/web_app_mac.mm View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/common/extensions/api/networking_private/networking_private_crypto.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/common/extensions/chrome_extensions_client.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/common/importer/firefox_importer_utils.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/common/service_process_util_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/installer/gcapi/gcapi.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/installer/setup/setup_util_unittest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/installer/util/shell_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/renderer/chrome_content_renderer_client.cc View 2 chunks +10 lines, -5 lines 0 comments Download
M cloud_print/service/win/service_utils.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/devtools_http_handler/devtools_http_handler.cc View 1 chunk +11 lines, -6 lines 0 comments Download
M components/google/core/browser/google_util.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/omnibox/browser/keyword_provider.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/omnibox/browser/search_suggestion_parser.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/pairing/bluetooth_controller_pairing_controller.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/password_manager/core/browser/password_manager.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/plugins/renderer/mobile_youtube_plugin.cc View 1 chunk +4 lines, -2 lines 0 comments Download
M components/plugins/renderer/plugin_placeholder.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M components/search/search.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/storage_monitor/volume_mount_watcher_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/test_runner/web_test_proxy.cc View 1 chunk +1 line, -1 line 0 comments Download
M components/variations/net/variations_http_header_provider.cc View 1 chunk +2 lines, -1 line 0 comments Download
M components/wifi/wifi_service_win.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/accessibility/android_granularity_movement_browsertest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M content/browser/plugin_browsertest.cc View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/plugin_service_impl.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/web_contents/web_contents_impl.cc View 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/webui/web_ui_data_source_impl.cc View 1 chunk +5 lines, -5 lines 0 comments Download
M content/common/set_process_title.cc View 1 chunk +1 line, -1 line 0 comments Download
M dbus/string_util.cc View 2 chunks +4 lines, -6 lines 0 comments Download
M extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc View 3 chunks +6 lines, -5 lines 0 comments Download
M extensions/browser/api/web_request/web_request_permissions.cc View 2 chunks +6 lines, -4 lines 0 comments Download
M extensions/common/user_script.cc View 1 chunk +2 lines, -1 line 0 comments Download
M extensions/utility/unpacker_unittest.cc View 2 chunks +4 lines, -2 lines 0 comments Download
M google_apis/drive/test_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M gpu/command_buffer/service/program_manager.cc View 1 chunk +2 lines, -1 line 0 comments Download
M ios/web/webui/web_ui_ios_data_source_impl.cc View 1 chunk +5 lines, -5 lines 0 comments Download
M media/filters/chunk_demuxer_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/frame_processor_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/source_buffer_stream_unittest.cc View 2 chunks +9 lines, -7 lines 0 comments Download
M media/video/capture/mac/video_capture_device_factory_mac.mm View 2 chunks +5 lines, -3 lines 0 comments Download
M media/video/capture/video_capture_device.cc View 1 chunk +1 line, -1 line 0 comments Download
M mojo/runner/shell_apptest.cc View 1 chunk +2 lines, -1 line 0 comments Download
M net/base/mime_util.cc View 1 chunk +1 line, -1 line 0 comments Download
M net/base/net_util.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M net/http/http_network_transaction.cc View 1 chunk +2 lines, -1 line 0 comments Download
M net/http/http_server_properties_impl.cc View 4 chunks +8 lines, -4 lines 0 comments Download
M net/quic/crypto/quic_crypto_client_config.cc View 1 chunk +2 lines, -1 line 0 comments Download
M net/spdy/spdy_session.cc View 1 chunk +2 lines, -1 line 0 comments Download
M net/spdy/spdy_session_test_util.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M net/tools/flip_server/mem_cache.cc View 1 chunk +1 line, -1 line 0 comments Download
M net/websockets/websocket_stream_cookie_test.cc View 1 chunk +2 lines, -1 line 0 comments Download
M pdf/document_loader.cc View 1 chunk +9 lines, -6 lines 0 comments Download
M remoting/host/it2me/it2me_host.cc View 1 chunk +2 lines, -1 line 0 comments Download
M remoting/host/remoting_me2me_host.cc View 1 chunk +2 lines, -1 line 0 comments Download
M third_party/zlib/google/zip_reader.cc View 1 chunk +2 lines, -1 line 0 comments Download
M tools/gn/ninja_binary_target_writer.cc View 1 chunk +2 lines, -1 line 0 comments Download
M tools/gn/ninja_target_writer.cc View 1 chunk +2 lines, -1 line 0 comments Download
M ui/base/l10n/l10n_util.cc View 1 chunk +3 lines, -2 lines 0 comments Download
M ui/gfx/font_list.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M ui/gfx/font_list_impl.cc View 1 chunk +2 lines, -1 line 0 comments Download

Messages

Total messages: 9 (3 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/1233043003/1
5 years, 5 months ago (2015-07-16 15:16:13 UTC) #2
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 5 months ago (2015-07-16 16:55:19 UTC) #4
brettw
This is the same as the previous patch from https://codereview.chromium.org/1239493005 with the one line change ...
5 years, 5 months ago (2015-07-16 17:34:37 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1233043003/1
5 years, 5 months ago (2015-07-16 17:36:12 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 5 months ago (2015-07-16 17:49:38 UTC) #8
commit-bot: I haz the power
5 years, 5 months ago (2015-07-16 17:50:48 UTC) #9
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/a7ff1b291cd328e9584779a4bb9101fa9a3c01ad
Cr-Commit-Position: refs/heads/master@{#339071}

Powered by Google App Engine
This is Rietveld 408576698