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

Issue 2158923004: Convert media constants to constexpr. (Closed)

Created:
4 years, 5 months ago by DaleCurtis
Modified:
4 years, 5 months ago
Reviewers:
mmoroz, alokp
CC:
alokp+watch_chromium.org, avayvod+watch_chromium.org, chromium-reviews, darin-cc_chromium.org, eme-reviews_chromium.org, feature-media-reviews_chromium.org, halliwell+watch_chromium.org, jam, lcwu+watch_chromium.org, mcasas+watch+vc_chromium.org, miu+watch_chromium.org, mlamouri+watch-content_chromium.org, mlamouri+watch-media_chromium.org, piman+watch_chromium.org, posciak+watch_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Convert media constants to constexpr. Replaces silly MEDIA_EXPORT inline functions with constexpr equivalents. Fingers crossed it passes the bots! BUG=627984 TEST=compiles Committed: https://crrev.com/39a7f93d67f79d6afadb0f74254eef19b5ff9318 Cr-Commit-Position: refs/heads/master@{#406326}

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+523 lines, -588 lines) Patch
M chrome/browser/media/android/remote/remote_media_player_bridge.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M chromecast/media/cma/base/balanced_media_task_runner_factory.cc View 7 chunks +7 lines, -10 lines 0 comments Download
M chromecast/media/cma/base/buffering_controller.h View 3 chunks +4 lines, -4 lines 0 comments Download
M chromecast/media/cma/base/buffering_controller.cc View 1 chunk +4 lines, -4 lines 0 comments Download
M chromecast/media/cma/base/buffering_state.h View 2 chunks +2 lines, -2 lines 0 comments Download
M chromecast/media/cma/base/buffering_state.cc View 5 chunks +13 lines, -16 lines 0 comments Download
M chromecast/media/cma/base/demuxer_stream_adapter.cc View 2 chunks +3 lines, -4 lines 0 comments Download
M chromecast/media/cma/base/media_task_runner.h View 1 chunk +1 line, -1 line 0 comments Download
M chromecast/media/cma/decoder/cast_audio_decoder_linux.cc View 1 chunk +1 line, -1 line 0 comments Download
M chromecast/media/cma/pipeline/av_pipeline_impl.cc View 5 chunks +7 lines, -7 lines 0 comments Download
M chromecast/media/cma/pipeline/media_pipeline_impl.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M content/renderer/media/android/media_source_delegate.cc View 3 chunks +4 lines, -4 lines 0 comments Download
M content/renderer/media/android/webmediaplayer_android.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M content/renderer/media/webrtc/media_stream_remote_video_source.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M media/base/android/audio_decoder_job.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/base/android/demuxer_android.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/android/media_codec_decoder_unittest.cc View 2 chunks +4 lines, -5 lines 0 comments Download
M media/base/android/media_codec_player.cc View 7 chunks +10 lines, -10 lines 0 comments Download
M media/base/android/media_codec_player_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/base/android/media_decoder_job.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/android/media_decoder_job.cc View 9 chunks +13 lines, -12 lines 0 comments Download
M media/base/android/media_player_bridge.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/base/android/media_source_player.cc View 5 chunks +5 lines, -5 lines 0 comments Download
M media/base/android/media_source_player_unittest.cc View 13 chunks +13 lines, -13 lines 0 comments Download
M media/base/android/media_statistics.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/android/media_statistics.cc View 3 chunks +6 lines, -6 lines 0 comments Download
M media/base/android/video_media_codec_decoder.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/base/audio_buffer.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/audio_buffer.cc View 2 chunks +5 lines, -15 lines 0 comments Download
M media/base/audio_buffer_converter.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M media/base/audio_buffer_queue_unittest.cc View 1 chunk +2 lines, -7 lines 0 comments Download
M media/base/audio_discard_helper.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/audio_discard_helper.cc View 5 chunks +6 lines, -6 lines 0 comments Download
M media/base/audio_discard_helper_unittest.cc View 4 chunks +5 lines, -11 lines 0 comments Download
M media/base/audio_splicer.h View 2 chunks +2 lines, -2 lines 0 comments Download
M media/base/audio_splicer.cc View 8 chunks +12 lines, -13 lines 0 comments Download
M media/base/audio_splicer_unittest.cc View 7 chunks +7 lines, -7 lines 0 comments Download
M media/base/audio_timestamp_helper.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/audio_timestamp_helper.cc View 4 chunks +4 lines, -5 lines 0 comments Download
M media/base/audio_timestamp_helper_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/base/bitstream_buffer.h View 2 chunks +3 lines, -3 lines 0 comments Download
M media/base/decoder_buffer.h View 3 chunks +5 lines, -5 lines 0 comments Download
M media/base/decoder_buffer.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/base/decoder_buffer_queue.cc View 3 chunks +4 lines, -6 lines 0 comments Download
M media/base/decoder_buffer_queue_unittest.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/base/demuxer.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/demuxer_perftest.cc View 5 chunks +5 lines, -5 lines 0 comments Download
M media/base/fake_demuxer_stream.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/base/pipeline_impl.cc View 4 chunks +5 lines, -5 lines 0 comments Download
M media/base/seekable_buffer.h View 1 chunk +1 line, -1 line 0 comments Download
M media/base/seekable_buffer.cc View 4 chunks +4 lines, -5 lines 0 comments Download
M media/base/seekable_buffer_unittest.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M media/base/stream_parser_buffer.h View 3 chunks +3 lines, -3 lines 0 comments Download
M media/base/stream_parser_buffer.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M media/base/time_delta_interpolator.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M media/base/timestamp_constants.h View 1 chunk +4 lines, -7 lines 0 comments Download
M media/base/video_frame.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/blink/buffered_data_source_host_impl.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/blink/webmediaplayer_impl.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/blink/websourcebuffer_impl.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M media/capture/video/mac/video_capture_device_avfoundation_mac.mm View 1 chunk +1 line, -1 line 0 comments Download
M media/capture/video/win/sink_input_pin_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/capture/video/win/video_capture_device_win.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/cdm/ppapi/external_clear_key/ffmpeg_cdm_audio_decoder.cc View 5 chunks +8 lines, -9 lines 0 comments Download
M media/filters/android/media_codec_audio_decoder.cc View 3 chunks +3 lines, -4 lines 0 comments Download
M media/filters/audio_decoder_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/audio_renderer_algorithm_unittest.cc View 1 chunk +3 lines, -3 lines 0 comments Download
M media/filters/audio_timestamp_validator.cc View 3 chunks +5 lines, -5 lines 0 comments Download
M media/filters/chunk_demuxer.cc View 7 chunks +10 lines, -10 lines 0 comments Download
M media/filters/chunk_demuxer_unittest.cc View 14 chunks +16 lines, -21 lines 0 comments Download
M media/filters/decoder_stream.h View 2 chunks +3 lines, -3 lines 0 comments Download
M media/filters/decoder_stream.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/decrypting_audio_decoder.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/decrypting_audio_decoder_unittest.cc View 2 chunks +7 lines, -13 lines 0 comments Download
M media/filters/ffmpeg_audio_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/ffmpeg_demuxer.cc View 22 chunks +37 lines, -38 lines 0 comments Download
M media/filters/ffmpeg_demuxer_unittest.cc View 1 chunk +3 lines, -5 lines 0 comments Download
M media/filters/ffmpeg_video_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/frame_processor.h View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/frame_processor.cc View 11 chunks +16 lines, -16 lines 0 comments Download
M media/filters/frame_processor_unittest.cc View 4 chunks +5 lines, -5 lines 0 comments Download
M media/filters/opus_audio_decoder.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/source_buffer_range.h View 3 chunks +3 lines, -3 lines 0 comments Download
M media/filters/source_buffer_range.cc View 3 chunks +3 lines, -3 lines 0 comments Download
M media/filters/source_buffer_stream.h View 3 chunks +4 lines, -4 lines 0 comments Download
M media/filters/source_buffer_stream.cc View 10 chunks +13 lines, -13 lines 0 comments Download
M media/filters/source_buffer_stream_unittest.cc View 7 chunks +12 lines, -12 lines 0 comments Download
M media/filters/video_frame_stream_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/filters/video_renderer_algorithm_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/filters/vpx_video_decoder.cc View 2 chunks +5 lines, -5 lines 0 comments Download
M media/formats/mp2t/es_adapter_video.h View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mp2t/es_adapter_video.cc View 5 chunks +6 lines, -6 lines 0 comments Download
M media/formats/mp2t/es_adapter_video_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mp2t/es_parser.cc View 2 chunks +2 lines, -4 lines 1 comment Download
M media/formats/mp2t/es_parser_adts.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M media/formats/mp2t/es_parser_h264.cc View 2 chunks +2 lines, -3 lines 1 comment Download
M media/formats/mp2t/es_parser_h264_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mp2t/es_parser_mpeg1audio.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M media/formats/mp2t/es_parser_test_base.cc View 4 chunks +4 lines, -8 lines 0 comments Download
M media/formats/mp2t/mp2t_stream_parser.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mp2t/ts_section_pes.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mp4/mp4_stream_parser.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mp4/mp4_stream_parser_unittest.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/mpeg/mpeg_audio_stream_parser_base.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/webm/webm_cluster_parser.h View 7 chunks +9 lines, -9 lines 0 comments Download
M media/formats/webm/webm_cluster_parser.cc View 20 chunks +24 lines, -24 lines 0 comments Download
M media/formats/webm/webm_cluster_parser_unittest.cc View 9 chunks +11 lines, -11 lines 0 comments Download
M media/formats/webm/webm_stream_parser.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/webm/webm_stream_parser_unittest.cc View 4 chunks +4 lines, -4 lines 0 comments Download
M media/formats/webm/webm_tracks_parser.h View 1 chunk +1 line, -1 line 0 comments Download
M media/formats/webm/webm_tracks_parser.cc View 1 chunk +2 lines, -2 lines 0 comments Download
M media/formats/webm/webm_tracks_parser_unittest.cc View 2 chunks +3 lines, -3 lines 0 comments Download
M media/gpu/android_video_decode_accelerator.cc View 8 chunks +23 lines, -29 lines 0 comments Download
M media/renderers/audio_renderer_impl.cc View 5 chunks +6 lines, -6 lines 0 comments Download
M media/renderers/skcanvas_video_renderer.h View 1 chunk +1 line, -1 line 0 comments Download
M media/renderers/skcanvas_video_renderer.cc View 1 chunk +1 line, -1 line 0 comments Download
M media/renderers/skcanvas_video_renderer_unittest.cc View 2 chunks +2 lines, -2 lines 0 comments Download
M media/test/pipeline_integration_test.cc View 6 chunks +6 lines, -6 lines 0 comments Download
M testing/libfuzzer/fuzzers/es_parser_adts_fuzzer.cc View 1 chunk +1 line, -1 line 1 comment Download
M testing/libfuzzer/fuzzers/es_parser_h264_fuzzer.cc View 1 chunk +1 line, -1 line 0 comments Download
M testing/libfuzzer/fuzzers/es_parser_mpeg1audio_fuzzer.cc View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 18 (9 generated)
DaleCurtis
+alokp, mmoroz@ for review. trybots are green enough.
4 years, 5 months ago (2016-07-19 00:00:49 UTC) #4
mmoroz
LGTM with a small question about |kNoDecodeTimestamp()|. https://codereview.chromium.org/2158923004/diff/1/media/formats/mp2t/es_parser.cc File media/formats/mp2t/es_parser.cc (right): https://codereview.chromium.org/2158923004/diff/1/media/formats/mp2t/es_parser.cc#newcode14 media/formats/mp2t/es_parser.cc:14: : dts(kNoDecodeTimestamp()), ...
4 years, 5 months ago (2016-07-19 08:50:33 UTC) #7
alokp
lgtm
4 years, 5 months ago (2016-07-19 17:09:13 UTC) #8
DaleCurtis
I'll send a followup CL for kNoDecodeTimestamp() didn't see that one! Thanks
4 years, 5 months ago (2016-07-19 18:21:42 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2158923004/1
4 years, 5 months ago (2016-07-19 18:22:34 UTC) #13
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 5 months ago (2016-07-19 18:35:35 UTC) #14
commit-bot: I haz the power
CQ bit was unchecked.
4 years, 5 months ago (2016-07-19 18:35:50 UTC) #15
commit-bot: I haz the power
Patchset 1 (id:??) landed as https://crrev.com/39a7f93d67f79d6afadb0f74254eef19b5ff9318 Cr-Commit-Position: refs/heads/master@{#406326}
4 years, 5 months ago (2016-07-19 18:37:27 UTC) #17
DaleCurtis
4 years, 5 months ago (2016-07-19 20:31:24 UTC) #18
Message was sent while issue was closed.
Actually converting kDecodeTimestamp is pretty involved, we need to switch
DecodeTimestamp away from using an inner TimeDelta to only having a primitive
type in order to use constexpr. It ends up using too many abstractions in its
current implementation to be switched over.

Powered by Google App Engine
This is Rietveld 408576698