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

Issue 17151011: Add a method for catching low end devices (Closed)

Created:
7 years, 6 months ago by David Trainor- moved to gerrit
Modified:
7 years, 6 months ago
CC:
chromium-reviews, erikwright+watch_chromium.org
Visibility:
Public.

Description

Add a method for catching low end devices Add a Java and native method for detecting if a device should be considered low end. BUG=246134 Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207498

Patch Set 1 #

Patch Set 2 : Private constructor #

Patch Set 3 : Fixed comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+92 lines, -0 lines) Patch
M base/android/base_jni_registrar.cc View 2 chunks +2 lines, -0 lines 0 comments Download
A base/android/java/src/org/chromium/base/SysUtils.java View 1 2 1 chunk +30 lines, -0 lines 0 comments Download
A base/android/sys_utils.h View 1 1 chunk +26 lines, -0 lines 0 comments Download
A base/android/sys_utils.cc View 1 1 chunk +31 lines, -0 lines 0 comments Download
M base/base.gyp View 1 chunk +1 line, -0 lines 0 comments Download
M base/base.gypi View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
David Trainor- moved to gerrit
ptal
7 years, 6 months ago (2013-06-18 23:37:08 UTC) #1
David Trainor- moved to gerrit
ptal thanks! klobag@: just a general fyi review bulach@: base/android/* darin@: base/*
7 years, 6 months ago (2013-06-19 00:44:49 UTC) #2
darin (slow to review)
LGTM for base/*gyp*
7 years, 6 months ago (2013-06-19 04:32:49 UTC) #3
bulach
lgtm, but as you mentioned in the bug, this won't be accessible early in the ...
7 years, 6 months ago (2013-06-19 11:32:38 UTC) #4
David Trainor- moved to gerrit
On 2013/06/19 11:32:38, bulach wrote: > lgtm, but as you mentioned in the bug, this ...
7 years, 6 months ago (2013-06-19 21:22:29 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/dtrainor@chromium.org/17151011/6001
7 years, 6 months ago (2013-06-20 01:50:51 UTC) #6
commit-bot: I haz the power
7 years, 6 months ago (2013-06-20 18:36:10 UTC) #7
Message was sent while issue was closed.
Change committed as 207498

Powered by Google App Engine
This is Rietveld 408576698