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

Issue 10578026: Remove message_loop_helpers.h and update the includes to point to the new location. (Closed)

Created:
8 years, 6 months ago by tfarina
Modified:
8 years, 6 months ago
Reviewers:
akalin
CC:
chromium-reviews, jam, sadrul, nkostylev+watch_chromium.org, erikwright (departed), kinuko+watch, stevenjb+watch_chromium.org, ncarter (slow), kkania, joi+watch-content_chromium.org, darin-cc_chromium.org, dhollowa+watch_chromium.org, tim (not reviewing), jochen+watch-content_chromium.org, Raghu Simha, brettw-cc_chromium.org, cbentzel+watch_chromium.org, feature-media-reviews_chromium.org, oshima+watch_chromium.org, michaeln, mihaip-chromium-reviews_chromium.org, akalin, Aaron Boodman, eroman, robertshield, davemoore+watch_chromium.org, rdsmith+dwatch_chromium.org, mmenke, willchan no longer on Chromium, Ben Goodger (Google), tony
Visibility:
Public.

Description

Remove message_loop_helpers.h and update the includes to point to the new location. Note: This was a TODO for akalin@. This patch was generated by: $ find . -name '*.h' -o -name '*.cc' -o -name '*.mm' | xargs sed -i -e "s/base\/message_loop_helpers.h/base\/sequenced_task_runner_helpers.h/g" R=akalin@chromium.org TBR=willchan@chromium.org,ben@chromium.org,jam@chromium.org,tony@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=143174

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+99 lines, -113 lines) Patch
M base/base.gypi View 1 chunk +0 lines, -1 line 0 comments Download
D base/message_loop_helpers.h View 1 chunk +0 lines, -13 lines 0 comments Download
M chrome/browser/automation/automation_provider.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/automation/automation_provider_observers.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browsing_data_quota_helper.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/browsing_data_remover.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/login/screen_locker.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/chromeos/web_socket_proxy.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/extension_data_deleter.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/extensions/extension_function.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/printing/print_dialog_gtk.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/printing/print_system_task_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/process_singleton_linux.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/protector/protector_service.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/renderer_host/chrome_render_message_filter.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/safe_browsing/browser_feature_extractor.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/safe_browsing/client_side_detection_host.cc View 3 chunks +2 lines, -2 lines 0 comments Download
M chrome/browser/safe_browsing/download_protection_service.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/safe_browsing/malware_details_history.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/safe_browsing/safe_browsing_service.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/search_engines/search_provider_install_data.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/spellchecker/spellcheck_host_impl.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/sync/glue/data_type_controller.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/themes/browser_theme_pack.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/gtk/create_application_shortcuts_dialog_gtk.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/views/extensions/browser_action_overflow_menu_controller.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/chrome_url_data_manager.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/media/media_internals_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/ui/webui/net_internals/net_internals_ui.cc View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/user_style_sheet_watcher.h View 1 chunk +1 line, -1 line 0 comments Download
M chrome/browser/webdata/web_data_service.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/appcache/chrome_appcache_service.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/download/download_manager_impl.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/fileapi/chrome_blob_storage_context.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/host_zoom_map_impl.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/gpu_message_filter.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/media/audio_input_renderer_host.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/media/audio_renderer_host.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/media/video_capture_host.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/pepper_file_message_filter.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/render_message_filter.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/renderer_host/resource_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M content/browser/ssl/ssl_client_auth_handler.h View 1 chunk +1 line, -1 line 0 comments Download
M content/public/browser/download_manager.h View 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/webplugin_delegate_proxy.h View 1 chunk +1 line, -1 line 0 comments Download
M net/url_request/url_request_context_getter.h View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/npapi/webplugin_delegate_impl.h View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/plugins/ppapi/gfx_conversion.h View 1 chunk +48 lines, -48 lines 0 comments Download
M webkit/plugins/ppapi/ppapi_webplugin_impl.h View 1 chunk +1 line, -1 line 0 comments Download
M webkit/plugins/webview_plugin.h View 1 chunk +3 lines, -3 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
tfarina
Please, take a look.
8 years, 6 months ago (2012-06-19 22:43:38 UTC) #1
akalin
LGTM! You probably wanna TBR all the owners.
8 years, 6 months ago (2012-06-20 08:51:52 UTC) #2
tfarina
8 years, 6 months ago (2012-06-20 15:09:42 UTC) #3
On 2012/06/20 08:51:52, akalin wrote:
> LGTM!
> 
> You probably wanna TBR all the owners.

Yup, TBRing... ;)

Powered by Google App Engine
This is Rietveld 408576698