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

Disable SkiaRenderer on low memory devices

UMA data suggests that we never draw frames on Mali-400 with <= 512 MB.
This is probably inability to use RGB565 framebuffer.

Disabling SkiaRenderer until this is fixed.

Bug: 1126490
Change-Id: Id9081cb178febc54d612498b8344286bd76643a9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2416722
Commit-Queue: Jonathan Backer <backer@chromium.org>
Auto-Submit: Jonathan Backer <backer@chromium.org>
Reviewed-by: default avatarVasiliy Telezhnikov <vasilyt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#808041}
parent d12004dc
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#include "components/viz/common/features.h" #include "components/viz/common/features.h"
#include "base/command_line.h" #include "base/command_line.h"
#include "base/system/sys_info.h"
#include "build/chromecast_buildflags.h" #include "build/chromecast_buildflags.h"
#include "components/viz/common/switches.h" #include "components/viz/common/switches.h"
#include "components/viz/common/viz_utils.h" #include "components/viz/common/viz_utils.h"
...@@ -114,6 +115,10 @@ bool IsUsingSkiaRenderer() { ...@@ -114,6 +115,10 @@ bool IsUsingSkiaRenderer() {
if (IsUsingVizForWebView()) if (IsUsingVizForWebView())
return true; return true;
// https://crbug.com/1126490 Mali-400 with <= 512 MB is currently broken.
if (base::SysInfo::AmountOfPhysicalMemoryMB() <= 512)
return false;
return base::FeatureList::IsEnabled(kUseSkiaRenderer) || return base::FeatureList::IsEnabled(kUseSkiaRenderer) ||
base::FeatureList::IsEnabled(kVulkan); 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