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

Issue 2965015: Attempt four at landing the "histogram.h removed from message_loop.h" patch. (Closed)

Created:
10 years, 5 months ago by Elliot Glaysher
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin
CC:
chromium-reviews, Raghu Simha, cbentzel+watch_chromium.org, Paul Godavari, Alpha Left Google, idana, ben+cc_chromium.org, John Grabowski, Erik does not do reviews, Paweł Hajdan Jr., jam, ncarter (slow), Aaron Boodman, fbarchard, pam+watch_chromium.org, tim (not reviewing), brettw-cc_chromium.org, kuchhal, darin-cc_chromium.org, scherkus (not reviewing), davemoore+watch_chromium.org, awong, amit
Base URL:
http://src.chromium.org/git/chromium.git
Visibility:
Public.

Description

Attempt four at landing the "histogram.h removed from message_loop.h" patch. Previously committed as r52349 and r52336. Related commits: r52367, r52364 and r52343. Rerunning trybots due to previous trybot breakage. TEST=none BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=52496

Patch Set 1 #

Patch Set 2 : Fixes based on tree failure log #

Patch Set 3 : Fix unrelated license check #

Patch Set 4 : win build fixes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+123 lines, -80 lines) Patch
M app/animation_container_unittest.cc View 1 2 2 chunks +2 lines, -1 line 0 comments Download
M app/clipboard/clipboard_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M app/slide_animation_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M base/message_loop.h View 3 chunks +3 lines, -5 lines 0 comments Download
M base/message_loop.cc View 4 chunks +40 lines, -39 lines 0 comments Download
M base/message_loop_proxy_impl_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M base/observer_list_threadsafe.h View 1 chunk +2 lines, -1 line 0 comments Download
M base/weak_ptr_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/autocomplete/autocomplete_edit.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/autocomplete/autocomplete_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/autocomplete/search_provider.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/browser_init.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/cancelable_request.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/chrome_thread_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/chromeos/boot_times_loader.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/cocoa/download_item_controller.mm View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/cocoa/external_protocol_dialog.mm View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/debugger/devtools_remote_listen_socket_unittest.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/download/download_util.h View 2 chunks +0 lines, -18 lines 0 comments Download
M chrome/browser/extensions/crx_installer.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/extensions/extension_prefs.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/extensions/extension_updater.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/gears_integration.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/geolocation/access_token_store.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/google_url_tracker.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/gtk/download_item_gtk.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/idle.h View 1 chunk +0 lines, -2 lines 0 comments Download
M chrome/browser/idle_win.cc View 1 2 3 1 chunk +3 lines, -0 lines 0 comments Download
M chrome/browser/io_thread.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/memory_details.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/net/chrome_net_log.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/omnibox_search_hint.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/plugin_process_host.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/plugin_service.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/printing/print_job.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/profile_import_process_host.h View 1 chunk +0 lines, -1 line 0 comments Download
M chrome/browser/safe_browsing/safe_browsing_database_bloom.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/sessions/session_service.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/spellcheck_host.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/sync/util/extensions_activity_monitor.h View 1 chunk +2 lines, -0 lines 0 comments Download
M chrome/browser/translate/translate_infobar_delegate.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/browser/views/download_item_view.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/desktop_notifications/active_notification_tracker.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/net/gaia/gaia_authenticator2.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/common/process_watcher_win.cc View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/renderer/gpu_channel_host.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/service/cloud_print/job_status_updater.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome/service/service_process.h View 1 chunk +1 line, -0 lines 0 comments Download
M chrome_frame/urlmon_bind_status_callback.h View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/ipc_channel_proxy.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M ipc/ipc_channel_win.h View 1 chunk +1 line, -0 lines 0 comments Download
M ipc/ipc_sync_channel_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M media/base/filters.h View 1 chunk +1 line, -0 lines 0 comments Download
M media/base/pipeline_impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M media/base/video_frame_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M media/filters/ffmpeg_glue_unittest.cc View 1 chunk +1 line, -0 lines 0 comments Download
M media/filters/video_renderer_base.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/base/capturing_net_log.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/base/file_stream_posix.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/base/file_stream_win.cc View 1 2 1 chunk +4 lines, -3 lines 0 comments Download
M net/base/host_resolver_impl.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/base/listen_socket_unittest.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/base/net_log.h View 1 chunk +4 lines, -2 lines 0 comments Download
M net/base/net_log.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/base/net_util.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/http/http_auth_handler.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/proxy/proxy_service.cc View 1 chunk +1 line, -0 lines 0 comments Download
M net/server/http_listen_socket.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M net/server/http_server_request_info.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/tools/fetch/http_listen_socket.cc View 1 chunk +4 lines, -1 line 0 comments Download
M net/tools/fetch/http_server_request_info.h View 1 chunk +1 line, -0 lines 0 comments Download
M net/url_request/url_request.h View 1 chunk +0 lines, -2 lines 0 comments Download
M net/url_request/url_request_job.cc View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/glue/webmediaplayer_impl.h View 1 chunk +1 line, -0 lines 0 comments Download
M webkit/glue/weburlloader_impl.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Elliot Glaysher
Same patch as before, except with a lot more files including scoped_ptr.h.
10 years, 5 months ago (2010-07-14 20:29:50 UTC) #1
Evan Martin
10 years, 5 months ago (2010-07-14 20:32:52 UTC) #2
rubber stamp

Powered by Google App Engine
This is Rietveld 408576698