Commit 8cc64f92 authored by Mikel Astiz's avatar Mikel Astiz Committed by Commit Bot

Revert "Crash if more than 1024 media events are generated in a second."

This reverts commit 8e85f058.

Reason for revert: causes blink_web_tests failures on Mac for
media/video-replaces-poster.html

Original change's description:
> Crash if more than 1024 media events are generated in a second.
>
> These should be low frequency, otherwise there's a bug.
>
> R=​tmathmeyer
>
> Bug: 1141985
> Change-Id: Ibfc915e7f2a4d524da33100dd43fff10b463376e
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2496023
> Auto-Submit: Dale Curtis <dalecurtis@chromium.org>
> Commit-Queue: Ted Meyer <tmathmeyer@chromium.org>
> Reviewed-by: Ted Meyer <tmathmeyer@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#822364}

TBR=dalecurtis@chromium.org,tmathmeyer@chromium.org

Change-Id: I2b37e8fe1ad5429f05d47cca125d0a081749c585
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 1141985,1144096
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2509469Reviewed-by: default avatarMikel Astiz <mastiz@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#822577}
parent 7e03a934
...@@ -131,10 +131,6 @@ void BatchingMediaLog::AddLogRecordLocked( ...@@ -131,10 +131,6 @@ void BatchingMediaLog::AddLogRecordLocked(
queued_media_events_.push_back(*event); queued_media_events_.push_back(*event);
} }
// TODO(crbug.com/1141985): This shouldn't exceed 1024 events within a
// single second. Crash to help us track down what's going wrong.
CHECK_LE(queued_media_events_.size(), 1024u);
if (ipc_send_pending_) if (ipc_send_pending_)
return; return;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment