Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(53)

Issue 1524433002: Revert "Added a call to ActivityManager.isLowRamDevice() from SysUtils.detectLowEndDevice()." (Closed)

Created:
5 years ago by aelias_OOO_until_Jul13
Modified:
5 years ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2564
Target Ref:
refs/pending/branch-heads/2564
Project:
chromium
Visibility:
Public.

Description

Revert "Added a call to ActivityManager.isLowRamDevice() from SysUtils.detectLowEndDevice()." This reverts commit 80dc8d87cfd1ef70541c101a355965f3a7a74415. Some devices in the wild with 1GB of RAM are returning true for ActivityManager.isLowRamDevice(), causing us to inappropriately enable aggressive memory optimizations designed for 512MB devices. Change back to only considering the true memory capacity of the device. BUG=568192, 459781 Review URL: https://codereview.chromium.org/1514713002 Cr-Commit-Position: refs/heads/master@{#364619} (cherry picked from commit 1dc262725270493bf333d53888319acacc5e8baa) Committed: https://chromium.googlesource.com/chromium/src/+/0d1cb19aeb0237c314f58d902e862d51bb9aad1a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -29 lines) Patch
M base/android/java/src/org/chromium/base/SysUtils.java View 6 chunks +5 lines, -29 lines 0 comments Download

Messages

Total messages: 2 (1 generated)
aelias_OOO_until_Jul13
5 years ago (2015-12-11 20:36:46 UTC) #2
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
0d1cb19aeb0237c314f58d902e862d51bb9aad1a (tree was closed).

Powered by Google App Engine
This is Rietveld 408576698