chromium/src
0
Fork 0
Code Activity

Apparently some 32-bit platforms can allocate 2gb+

Browse Source 
So remove the test checking otherwise.

Fixed: 375201227
Change-Id: I5d3661c6b0c3fc650d6d4396e84bed830c282f57
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5957127
Reviewed-by: Lijin Shen <lazzzis@google.com>
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Auto-Submit: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1372875}
This commit is contained in:
Dale Curtis
2024-10-23 19:43:33 +00:00
committed by Chromium LUCI CQ
parent bc0b89485e
commit a865b4e33c
1 changed files with 0 additions and 7 deletions

7
media/base/video_frame_unittest.cc

@ -873,13 +873,6 @@ TEST(VideoFrame, NoFrameSizeExceedsUint32) {
base::CheckedNumeric<uint32_t> allocation_size =
VideoFrame::AllocationSize(format, max_size);
ASSERT_TRUE(allocation_size.IsValid());
// Allocations above 2gb on 32-bit platforms should fail gracefully.
#ifdef ARCH_CPU_32_BITS
if (allocation_size.ValueOrDie() >= std::numeric_limits<int32_t>::max()) {
ASSERT_FALSE(VideoFrame::CreateBlackFrame(max_size));
}
#endif
}
}