Commit fb083b7c authored by Jonathan Backer's avatar Jonathan Backer Committed by Commit Bot

Fix order of low memory check.

Certain builds of Fuchsia set low-end device flag, which forces
SysInfo::AmountOfPhysicalMemory to return 512 MB.

This corrects an ordering issue introduced in crrev.com/c/2419555

Bug: 1126490, 1130036
Change-Id: I8d0520cc0393864265b2ad8c4646b146b7bcf547
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2421300Reviewed-by: default avatarPeng Huang <penghuang@chromium.org>
Commit-Queue: Jonathan Backer <backer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#808996}
parent add3c057
......@@ -105,16 +105,20 @@ bool IsUsingSkiaRenderer() {
if (base::android::BuildInfo::GetInstance()->sdk_int() <=
base::android::SDK_VERSION_KITKAT)
return false;
// https://crbug.com/1126490 Mali-400 with <= 512 MB is currently broken.
if (base::SysInfo::AmountOfPhysicalMemoryMB() <= 512)
return false;
#endif
// Viz for webview requires SkiaRenderer.
if (IsUsingVizForWebView())
return true;
#if defined(OS_ANDROID)
// https://crbug.com/1126490 Mali-400 with <= 512 MB is currently broken.
// Must be checked after IsUsingVizForWebView because it requires
// SkiaRenderer.
if (base::SysInfo::AmountOfPhysicalMemoryMB() <= 512)
return false;
#endif
return base::FeatureList::IsEnabled(kUseSkiaRenderer) ||
base::FeatureList::IsEnabled(kVulkan);
}
......
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