diff options
author | jk | 2020-09-06 11:44:29 -0500 |
---|---|---|
committer | jk | 2020-09-06 11:44:29 -0500 |
commit | 727a1ecd1d56ecbdc55d47af6b3cbc9bc04d64dc (patch) | |
tree | 720fc7bae34102a2ddf7b6eda2465738624d6630 /remove-NotifyError-calls-and-just-send-a-normal-message.patch | |
parent | 76d22723a239d8281dbe657cfe6e8b3b9e05a3d3 (diff) | |
download | aur-727a1ecd1d56ecbdc55d47af6b3cbc9bc04d64dc.tar.gz |
upgpkg: ungoogled-chromium 85.0.4183.83-1
upstream release
Diffstat (limited to 'remove-NotifyError-calls-and-just-send-a-normal-message.patch')
-rw-r--r-- | remove-NotifyError-calls-and-just-send-a-normal-message.patch | 42 |
1 files changed, 0 insertions, 42 deletions
diff --git a/remove-NotifyError-calls-and-just-send-a-normal-message.patch b/remove-NotifyError-calls-and-just-send-a-normal-message.patch deleted file mode 100644 index ab638bbef026..000000000000 --- a/remove-NotifyError-calls-and-just-send-a-normal-message.patch +++ /dev/null @@ -1,42 +0,0 @@ -From 3bfec15358b788a2b7e1beefaa177ba84a6da328 Mon Sep 17 00:00:00 2001 -From: Ted Meyer <tmathmeyer@chromium.org> -Date: Mon, 1 Jun 2020 19:31:18 +0000 -Subject: [PATCH] Remove NotifyError calls and just send a normal message - instead - -rewrote the TODO as well. - -Bug: 1088055 -Change-Id: If83ceeb3a786a1813e41bd3ac6c7309791e92400 -Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2223010 -Auto-Submit: Ted Meyer <tmathmeyer@chromium.org> -Commit-Queue: Dale Curtis <dalecurtis@chromium.org> -Reviewed-by: Dale Curtis <dalecurtis@chromium.org> -Cr-Commit-Position: refs/heads/master@{#773776} ---- - media/filters/decoder_selector.cc | 12 +++++------- - 1 file changed, 5 insertions(+), 7 deletions(-) - -diff --git a/media/filters/decoder_selector.cc b/media/filters/decoder_selector.cc -index bb06378b92d..9f510bb578c 100644 ---- a/media/filters/decoder_selector.cc -+++ b/media/filters/decoder_selector.cc -@@ -165,13 +165,11 @@ void DecoderSelector<StreamType>::OnDecoderInitializeDone(Status status) { - DCHECK(task_runner_->BelongsToCurrentThread()); - - if (!status.is_ok()) { -- // TODO(tmathmeyer) this might be noisy in media log. Consider batching -- // all failures as causes to a single Status object and only surfacing it if -- // decoder selection fails entirely. -- media_log_->NotifyError( -- Status(StatusCode::kDecoderFailedInitialization) -- .WithData("Decoder name", decoder_->GetDisplayName()) -- .AddCause(std::move(status))); -+ // TODO(tmathmeyer) this was too noisy in media log. Batch all the logs -+ // together and then send them as an informational notice instead of -+ // using NotifyError. -+ MEDIA_LOG(INFO, media_log_) -+ << "Failed to initialize " << decoder_->GetDisplayName(); - - // Try the next decoder on the list. - decoder_.reset(); |