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

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

Created:
5 years, 5 months ago by yosin_UTC9
Modified:
5 years, 5 months ago
Reviewers:
brettw, 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

Revert of Remove some legacy versions of StartsWith and EndsWith. (patchset #6 id:100001 of https://codereview.chromium.org/1239493005/) Reason for revert: Compilation error on Windows: FAILED: ninja -t msvc -e environment.x86 -- C:\b\build\goma/gomacc "C:\b\depot_tools\win_toolchain\vs2013_files\VC\bin\amd64_x86\cl.exe" /nologo /showIncludes /FC @obj\chrome\installer\gcapi\gcapi_lib.gcapi.obj.rsp /c ..\..\chrome\installer\gcapi\gcapi.cc /Foobj\chrome\installer\gcapi\gcapi_lib.gcapi.obj /Fdobj\chrome\gcapi_lib.cc.pdb c:\b\build\slave\win-latest-rel\build\src\chrome\installer\gcapi\gcapi.cc(365) : error C3083: 'StartsWith': the symbol to the left of a '::' must be a type c:\b\build\slave\win-latest-rel\build\src\chrome\installer\gcapi\gcapi.cc(365) : error C2039: 'INSENSITIVE_ASCII' : is not a member of 'base' c:\b\build\slave\win-latest-rel\build\src\chrome\installer\gcapi\gcapi.cc(365) : error C2065: 'INSENSITIVE_ASCII' : undeclared identifier ninja: build stopped: subcommand failed. Original issue's 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 > > Committed: https://crrev.com/edce9a33027cc5f73c4866d70e34f690f6720a56 > Cr-Commit-Position: refs/heads/master@{#338996} TBR=jam@chromium.org,brettw@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=506255 Committed: https://crrev.com/ddbde8ff8310abbd529ad5cc43e3991e79106a52 Cr-Commit-Position: refs/heads/master@{#338998}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+226 lines, -327 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 +13 lines, -0 lines 0 comments Download
M base/strings/string_util_unittest.cc View 1 chunk +45 lines, -69 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 +1 line, -2 lines 0 comments Download
M chrome/browser/banners/app_banner_data_fetcher.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/chromeos/extensions/wallpaper_private_api.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/chromeos/policy/device_local_account.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/component_updater/cld_component_installer_unittest.cc View 2 chunks +3 lines, -6 lines 0 comments Download
M chrome/browser/extensions/api/web_navigation/web_navigation_apitest.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/extensions/default_apps.cc View 1 chunk +1 line, -2 lines 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 +1 line, -2 lines 0 comments Download
M chrome/browser/net/spdyproxy/data_reduction_proxy_chrome_settings.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/browser/safe_browsing/safe_browsing_database.cc View 2 chunks +9 lines, -17 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 +2 lines, -3 lines 0 comments Download
M chrome/browser/sync/test/integration/bookmarks_helper.cc View 1 chunk +1 line, -2 lines 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 +2 lines, -4 lines 0 comments Download
M chrome/browser/ui/libgtk2ui/select_file_dialog_impl_gtk2.cc View 1 chunk +1 line, -2 lines 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 +7 lines, -13 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 +1 line, -2 lines 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 +1 line, -2 lines 0 comments Download
M chrome/common/service_process_util_unittest.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/installer/gcapi/gcapi.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/installer/setup/setup_util_unittest.cc View 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/installer/util/shell_util.cc View 1 chunk +1 line, -2 lines 0 comments Download
M chrome/renderer/chrome_content_renderer_client.cc View 2 chunks +5 lines, -10 lines 0 comments Download
M cloud_print/service/win/service_utils.cc View 1 chunk +1 line, -2 lines 0 comments Download
M components/devtools_http_handler/devtools_http_handler.cc View 1 chunk +6 lines, -11 lines 0 comments Download
M components/google/core/browser/google_util.cc View 1 chunk +1 line, -2 lines 0 comments Download
M components/omnibox/browser/keyword_provider.cc View 1 chunk +1 line, -2 lines 0 comments Download
M components/omnibox/browser/search_suggestion_parser.cc View 1 chunk +1 line, -2 lines 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 +1 line, -2 lines 0 comments Download
M components/plugins/renderer/mobile_youtube_plugin.cc View 1 chunk +2 lines, -4 lines 0 comments Download
M components/plugins/renderer/plugin_placeholder.cc View 1 chunk +2 lines, -3 lines 0 comments Download
M components/search/search.cc View 1 chunk +1 line, -2 lines 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 +1 line, -2 lines 0 comments Download
M components/wifi/wifi_service_win.cc View 1 chunk +1 line, -2 lines 0 comments Download
M content/browser/accessibility/android_granularity_movement_browsertest.cc View 2 chunks +2 lines, -4 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 +1 line, -2 lines 0 comments Download
M content/browser/web_contents/web_contents_impl.cc View 1 chunk +1 line, -2 lines 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 +6 lines, -4 lines 0 comments Download
M extensions/browser/api/declarative_webrequest/webrequest_condition_attribute.cc View 3 chunks +5 lines, -6 lines 0 comments Download
M extensions/browser/api/web_request/web_request_permissions.cc View 2 chunks +4 lines, -6 lines 0 comments Download
M extensions/common/user_script.cc View 1 chunk +1 line, -2 lines 0 comments Download
M extensions/utility/unpacker_unittest.cc View 2 chunks +2 lines, -4 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 +1 line, -2 lines 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 +7 lines, -9 lines 0 comments Download
M media/video/capture/mac/video_capture_device_factory_mac.mm View 2 chunks +3 lines, -5 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 +1 line, -2 lines 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 +1 line, -2 lines 0 comments Download
M net/http/http_server_properties_impl.cc View 4 chunks +4 lines, -8 lines 0 comments Download
M net/quic/crypto/quic_crypto_client_config.cc View 1 chunk +1 line, -2 lines 0 comments Download
M net/spdy/spdy_session.cc View 1 chunk +1 line, -2 lines 0 comments Download
M net/spdy/spdy_session_test_util.cc View 1 chunk +2 lines, -3 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 +1 line, -2 lines 0 comments Download
M pdf/document_loader.cc View 1 chunk +6 lines, -9 lines 0 comments Download
M remoting/host/it2me/it2me_host.cc View 1 chunk +1 line, -2 lines 0 comments Download
M remoting/host/remoting_me2me_host.cc View 1 chunk +1 line, -2 lines 0 comments Download
M third_party/zlib/google/zip_reader.cc View 1 chunk +1 line, -2 lines 0 comments Download
M tools/gn/ninja_binary_target_writer.cc View 1 chunk +1 line, -2 lines 0 comments Download
M tools/gn/ninja_target_writer.cc View 1 chunk +1 line, -2 lines 0 comments Download
M ui/base/l10n/l10n_util.cc View 1 chunk +2 lines, -3 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 +1 line, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
yosin_UTC9
Created Revert of Remove some legacy versions of StartsWith and EndsWith.
5 years, 5 months ago (2015-07-16 07:09:21 UTC) #1
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1233453011/1
5 years, 5 months ago (2015-07-16 07:09:40 UTC) #2
commit-bot: I haz the power
Committed patchset #1 (id:1)
5 years, 5 months ago (2015-07-16 07:11:12 UTC) #3
commit-bot: I haz the power
5 years, 5 months ago (2015-07-16 07:12:17 UTC) #4
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/ddbde8ff8310abbd529ad5cc43e3991e79106a52
Cr-Commit-Position: refs/heads/master@{#338998}

Powered by Google App Engine
This is Rietveld 408576698