New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 607311 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Left Chrome team
Closed: Nov 2016
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

512MB devices should be low end

Project Member Reported by mariakho...@chromium.org, Apr 27 2016

Issue description

Since https://codereview.chromium.org/1514713002, we have made a non-exclusive check for low end devices. Include 512MB devices into accounting.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0064e5ec6d8999d150a15ea657e26bc5b180bfa7

commit 0064e5ec6d8999d150a15ea657e26bc5b180bfa7
Author: mariakhomenko <mariakhomenko@chromium.org>
Date: Thu Apr 28 19:25:33 2016

Make 512MB devices low end.

Devices reporting exactly 512MB are currently not categorized as low-end
thanks to a < instead of <=.  Because of reserved amounts, physical 512MB
devices are reported as lower, so there isn't a known case where this makes
any difference, but this line of code has caused confusion a few times, so change
it for clarity's sake.

BUG= 607311 

Review-Url: https://codereview.chromium.org/1925813002
Cr-Commit-Position: refs/heads/master@{#390446}

[modify] https://crrev.com/0064e5ec6d8999d150a15ea657e26bc5b180bfa7/base/android/java/src/org/chromium/base/SysUtils.java
[modify] https://crrev.com/0064e5ec6d8999d150a15ea657e26bc5b180bfa7/base/sys_info.cc

Status: Fixed (was: Assigned)

Sign in to add a comment