New issue
Advanced search Search tips

Issue 703190 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Mar 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug-Regression

Blocking:
issue 166704



Sign in to add a comment

[Regression] navigator.hardwareConcurrency returns 0

Reported by a...@scirra.com, Mar 20 2017

Issue description

Steps to reproduce the problem:
Visit https://www.scirra.com/labs/bugs/hwcbug.html and observe the value of hardwareConcurrency.

What is the expected behavior?
It should return 4, which is what it does in Chrome 56.0.2924.87.

What went wrong?
navigator.hardwareConcurrency returns 0 on Chrome 58.0.3029.21 on Android 7 (on a HTC 10).

Did this work before? Yes 56.0.2924.87

Chrome version: 58.0.3029.21  Channel: beta
OS Version: 7.0
Flash Version: n/a

It still appears to work in Windows in that version.

hardwareConcurrency is important to know how many workers to spin up, and some apps may be broken since they will spin up zero workers and not get any work done.
 
Components: -Blink Blink>Internals
Labels: -Pri-2 Pri-1
Owner: tedc...@chromium.org
Status: Assigned (was: Unconfirmed)
Confirmed on a Nexus5X as well. Appears to work fine on Linux I wonder if this is something to do with the Android sandbox?
Cc: tedc...@chromium.org
Labels: M-58 ReleaseBlock-Stable
Owner: rsesek@chromium.org
Confirmed disabling the sandbox fixes the issue.

Over to Robert.
Blocking: 166704

Comment 4 by rsesek@chromium.org, Mar 20 2017

Status: Started (was: Assigned)
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 20 2017

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

commit e5c08912ddec1fe44b990a2416c68c8d77f9b5e5
Author: rsesek <rsesek@chromium.org>
Date: Mon Mar 20 22:30:32 2017

[Android] __NR_getdents64 is available on all architectures.

BUG= 703190 
R=jorgelo@chromium.org

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

[modify] https://crrev.com/e5c08912ddec1fe44b990a2416c68c8d77f9b5e5/content/common/sandbox_linux/android/sandbox_bpf_base_policy_android.cc

Comment 6 by rsesek@chromium.org, Mar 20 2017

Labels: Merge-Request-58
Project Member

Comment 7 by sheriffbot@chromium.org, Mar 20 2017

Labels: -Merge-Request-58 Hotlist-Merge-Approved Merge-Approved-58
Your change meets the bar and is auto-approved for M58. Please go ahead and merge the CL to branch 3029 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 8 by bugdroid1@chromium.org, Mar 21 2017

Labels: -merge-approved-58 merge-merged-3029
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1f5117d10e496c8bc1869120539cfa2dfc425b14

commit 1f5117d10e496c8bc1869120539cfa2dfc425b14
Author: Robert Sesek <rsesek@chromium.org>
Date: Tue Mar 21 15:14:15 2017

[Android] __NR_getdents64 is available on all architectures.

BUG= 703190 
R=jorgelo@chromium.org

Review-Url: https://codereview.chromium.org/2762583004
Cr-Commit-Position: refs/heads/master@{#458205}
(cherry picked from commit e5c08912ddec1fe44b990a2416c68c8d77f9b5e5)

Review-Url: https://codereview.chromium.org/2762233002 .
Cr-Commit-Position: refs/branch-heads/3029@{#329}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}

[modify] https://crrev.com/1f5117d10e496c8bc1869120539cfa2dfc425b14/content/common/sandbox_linux/android/sandbox_bpf_base_policy_android.cc

Comment 9 by rsesek@chromium.org, Mar 21 2017

Status: Fixed (was: Started)
Status: Verified (was: Fixed)
Verified fix 'return 4' on HTC 10/NRD90M with '58.0.3029.33'

Sign in to add a comment