Commit 596bcfc4 authored by Bo Liu's avatar Bo Liu Committed by Chromium LUCI CQ

Remove legacy mailbox fallback on all platforms

Hopefully no longer needed on all platforms after r832898

Bug: 1153376
Change-Id: I7bc173cd4f404d6ababbe3e549f495b04bb028d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2571151Reviewed-by: default avatarVasiliy Telezhnikov <vasilyt@chromium.org>
Commit-Queue: Bo <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833282}
parent 3944fd9c
...@@ -6,7 +6,6 @@ ...@@ -6,7 +6,6 @@
#include <utility> #include <utility>
#include "build/build_config.h"
#include "components/viz/common/resources/resource_format_utils.h" #include "components/viz/common/resources/resource_format_utils.h"
#include "gpu/command_buffer/common/shared_image_usage.h" #include "gpu/command_buffer/common/shared_image_usage.h"
#include "gpu/command_buffer/service/mailbox_manager.h" #include "gpu/command_buffer/service/mailbox_manager.h"
...@@ -104,19 +103,12 @@ void ImageContextImpl::BeginAccessIfNecessary( ...@@ -104,19 +103,12 @@ void ImageContextImpl::BeginAccessIfNecessary(
std::vector<GrBackendSemaphore>* end_semaphores) { std::vector<GrBackendSemaphore>* end_semaphores) {
// Prepare for accessing shared image. // Prepare for accessing shared image.
if (mailbox_holder().mailbox.IsSharedImage()) { if (mailbox_holder().mailbox.IsSharedImage()) {
constexpr bool allow_legacy_mailbox_fallback = if (!BeginAccessIfNecessaryForSharedImage(
#if defined(OS_ANDROID) context_state, representation_factory, begin_semaphores,
false; end_semaphores)) {
#else
true;
#endif
bool result = BeginAccessIfNecessaryForSharedImage(
context_state, representation_factory, begin_semaphores,
end_semaphores);
if (!result && !allow_legacy_mailbox_fallback) {
CreateFallbackImage(context_state); CreateFallbackImage(context_state);
return;
} }
return;
} }
// Prepare for accessing legacy mailbox. // Prepare for accessing legacy mailbox.
......
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