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

Issue 10389164: Cleanup: Remove unneeded scoped_ptr.h includes from content. (Closed)

Created:
8 years, 7 months ago by Lei Zhang
Modified:
8 years, 7 months ago
Reviewers:
jam
CC:
chromium-reviews, jochen+watch-content_chromium.org, jam, apatrick_chromium, joi+watch-content_chromium.org, feature-media-reviews_chromium.org, Satish, darin-cc_chromium.org, rdsmith+dwatch_chromium.org, kinuko+watch
Visibility:
Public.

Description

Cleanup: Remove unneeded scoped_ptr.h includes from content. BUG=none TEST=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=137637

Patch Set 1 #

Patch Set 2 : fix lint errors #

Patch Set 3 : #

Patch Set 4 : #

Patch Set 5 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+54 lines, -56 lines) Patch
M content/browser/debugger/worker_devtools_manager.h View 1 2 3 2 chunks +1 line, -2 lines 0 comments Download
M content/browser/download/download_buffer.h View 1 2 3 3 chunks +3 lines, -4 lines 0 comments Download
M content/browser/fileapi/browser_file_system_helper.h View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
M content/browser/geolocation/location_api_adapter_android.h View 1 2 3 2 chunks +6 lines, -6 lines 0 comments Download
M content/browser/geolocation/location_provider_android.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M content/browser/geolocation/mock_location_provider.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M content/browser/gpu/gpu_data_manager_impl.h View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/renderer_host/java/java_bridge_dispatcher_host_manager.h View 1 2 3 2 chunks +2 lines, -2 lines 0 comments Download
M content/browser/renderer_host/x509_user_cert_resource_handler.h View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M content/browser/speech/input_tag_speech_dispatcher_host.h View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M content/common/gpu/client/command_buffer_proxy_impl.h View 1 2 3 2 chunks +6 lines, -6 lines 0 comments Download
M content/common/gpu/media/gpu_video_decode_accelerator.h View 1 2 3 2 chunks +2 lines, -2 lines 0 comments Download
M content/common/gpu/media/h264_parser.h View 1 2 3 4 2 chunks +4 lines, -4 lines 0 comments Download
M content/common/gpu/media/h264_parser.cc View 1 2 3 10 chunks +11 lines, -11 lines 0 comments Download
M content/common/mac/attributed_string_coder.h View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
M content/gpu/gpu_watchdog_thread.h View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/media/audio_device_thread.h View 1 2 3 1 chunk +0 lines, -1 line 0 comments Download
M content/renderer/media/media_stream_dependency_factory.h View 1 2 3 2 chunks +1 line, -2 lines 0 comments Download
M content/renderer/media/mock_peer_connection_impl.h View 1 2 3 2 chunks +2 lines, -1 line 0 comments Download
M content/renderer/media/peer_connection_handler_base.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/media/pepper_platform_video_decoder_impl.h View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/pepper/pepper_platform_audio_input_impl.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/pepper/pepper_platform_context_3d_impl.h View 1 2 3 1 chunk +2 lines, -1 line 0 comments Download
M content/renderer/renderer_webcolorchooser_impl.h View 1 2 3 1 chunk +1 line, -1 line 0 comments Download
M content/renderer/websharedworker_proxy.h View 1 2 3 2 chunks +1 line, -2 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Lei Zhang
Mostly rubberstampable.
8 years, 7 months ago (2012-05-16 08:43:39 UTC) #1
jam
lgtm
8 years, 7 months ago (2012-05-16 22:24:21 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/10389164/12003
8 years, 7 months ago (2012-05-16 22:26:36 UTC) #3
commit-bot: I haz the power
Try job failure for 10389164-12003 (retry) on linux_chromeos for step "compile" (clobber build). It's a ...
8 years, 7 months ago (2012-05-16 22:44:32 UTC) #4
Lei Zhang
rebased and fixed includes in content/common/gpu/media/h264_parser.*
8 years, 7 months ago (2012-05-17 01:18:32 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/10389164/2011
8 years, 7 months ago (2012-05-17 01:19:31 UTC) #6
commit-bot: I haz the power
Try job failure for 10389164-2011 (retry) on win for step "update" (clobber build). It's a ...
8 years, 7 months ago (2012-05-17 01:54:19 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thestig@chromium.org/10389164/2011
8 years, 7 months ago (2012-05-17 05:18:23 UTC) #8
commit-bot: I haz the power
8 years, 7 months ago (2012-05-17 06:53:39 UTC) #9
Change committed as 137637

Powered by Google App Engine
This is Rietveld 408576698