New issue
Advanced search Search tips

Issue 709019 link

Starred by 2 users
Status: Fixed
Owner:
foolip@chromium.org
Closed: Apr 2017
Cc:
domenic@chromium.org
panicker@chromium.org
yoavweiss@chromium.org
npm@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 674593


Show other hotlists

Hotlists containing this issue:
Non-Standard-IDL


Sign in to add a comment

Standardize navigator.hardwareConcurrency

Project Member Reported by foolip@chromium.org, Apr 6 2017 
https://wiki.whatwg.org/wiki/Navigator_HW_Concurrency

This is shipping in Chrome, Edge, Firefox and Safari.

We should graduate this into something that looks more like a real spec.

(This is a tracking bug for the fact that Blink has a bit of IDL that isn't found in any of the specs we link to.)

Comment 1 by foolip@chromium.org, Apr 6 2017 

domenic@, opinions on where we put this?

Comment 2 by domenic@chromium.org, Apr 10 2017 

This is already in HTML: https://html.spec.whatwg.org/multipage/workers.html#navigator.hardwareconcurrency

I'll update the wiki page, and leave this bug open in case you want to track updating Blink's IDL files.
Project Member

Comment 3 by bugdroid1@chromium.org, Apr 17 2017 

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

commit 4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1
Author: foolip <foolip@chromium.org>
Date: Mon Apr 17 09:57:35 2017

Rename NavigatorCPU mixin to NavigatorConcurrentHardware to match spec

The name of a mixin is not observable, so this is refactoring only.
This also changes the type to unsigned long long to match the spec,
but this will not be observable in any way.

BUG= 709019 
R=domenic@chromium.org

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

[modify] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/core_idl_files.gni
[modify] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/frame/BUILD.gn
[modify] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/frame/Navigator.h
[modify] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/frame/Navigator.idl
[delete] https://crrev.com/c105f1574d48e229385fbef9293aca66fc087f9b/third_party/WebKit/Source/core/frame/NavigatorCPU.idl
[rename] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/frame/NavigatorConcurrentHardware.cpp
[rename] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/frame/NavigatorConcurrentHardware.h
[add] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/frame/NavigatorConcurrentHardware.idl
[modify] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/workers/WorkerNavigator.h
[modify] https://crrev.com/4f4d0e4f9122db7ee36d5bbd3b086d998aa9e9c1/third_party/WebKit/Source/core/workers/WorkerNavigator.idl

Comment 4 by foolip@chromium.org, Apr 17 2017

Owner: foolip@chromium.org
Status: Fixed (was: Untriaged)
►

Sign in to add a comment