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

Issue 801442 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[console] format specifiers on Symbols

Project Member Reported by domfarolino@gmail.com, Jan 12 2018

Issue description

Chrome Version       : 63.0.3239.108 (Official Build) (64-bit)
URLs (if applicable) :
Other browsers tested: https://github.com/w3c/web-platform-tests/pull/9008
  Add OK or FAIL, along with the version, after other browsers where you
have tested this issue:
     Safari: Fail
    Firefox: Fail
       Edge: Fail

What steps will reproduce the problem?
(1) Run the JavaScript appearing in the <script> tag in the test in https://github.com/w3c/web-platform-tests/pull/9008


What is the expected result?

See "expected.png" image attached

What happens instead?

See "chrome.png" image attached

Please provide any additional information below. Attach a screenshot if
possible.
 
expected.png
46.7 KB View Download
chrome.png
29.1 KB View Download
I should note, with some guidance I'd like to help contribute some fixes for this stuff!!
P.S. The tests have been updated to

  1. Be more inclusive to all console methods using format specifiers
  2. To ensure we're not testing console.dir, since console.dir is not spec'd to take in format specifiers. This was a mistake in the original tests. I'm noting this here since there has been some confusion on this.
Cc: vamshi.k...@techmahindra.com
Labels: Needs-Feedback Triaged-ET Needs-Triage-M63
Thanks for filing the issue!

We tried checking the issue by navigating to https://github.com/w3c/web-platform-tests/pull/9008 on reported chrome version 63.0.3239.108 and on the latest canary 65.0.3323.0 Using Mac 10.13.1 with the below mentioned steps.
1. Downloaded the code/file and unzipped it.
2. We did not find any javascript appearing in the <script> tag to test

@Reporter: Could you please share a sample test file which helps us to triage the issue in a better way.
Hmm, I'm not entirely sure what the problem with the PR is...the PR adds two files to the repository; one is a manual test that can be seen from this jsbin: https://jsbin.com/tisorisopa/edit?html,console,output and the other which basically says:

console.METHOD(FORMAT, Symbol.for("description")) should throw for all possible combinations of the following:

METHOD = {log, dirxml, trace, group, groupCollapsed}
FORMAT = {"%i", "%d", "%f"}

Hopefully this makes sense, but let me know if I can help clarify further. Also, by running a local copy of the WPT repository checked out to my fork's PR branch, running `./wpt run chrome console` should give a pretty good idea of the failures.
Project Member

Comment 5 by sheriffbot@chromium.org, Jan 30 2018

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding requester "vamshi.kommuri@techmahindra.com" to the cc list and removing "Needs-Feedback" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Adding some more info and bumping this thread. The test PR not only enforces error-throwing behavior for the aforementioned specifiers and their application on Symbols, but also accompanies the Console spec change (https://github.com/whatwg/console/pull/123) that specifies a Symbol's descriptor should be used when applying the %s% format specifier.
Components: Platform>DevTools
Owner: l...@chromium.org
Status: Assigned (was: Unconfirmed)
Erik, mind taking a look?

Comment 9 by l...@chromium.org, Mar 12 2018

Labels: Needs-Feedback
Thanks for the report.  After reading the discussions on Github/WebKit tracker, I'd like to clarify what needs to be done.  Based on the last comment in this thread [1],
"it seems we've decided not to throw when the %s specifier is applied to Symbols"

So it sounds like "%s" string formatter has been decided, but numeric formatters (%i/d/f) have not.  The spec thread includes 2 proposed paths forward for Symbols: they should either
- always throw error whether DevTools are open/closed (revise this crbug or file a V8 bug)
OR
- print 'NaN' (what Chrome does today, we could close this crbug)

Since it hasn't settled yet, it looks like we need to wait.  dom@, please let me know if I'm wrong here, or if I'm not be up to date.


[1] https://github.com/whatwg/console/issues/113
You're up to date as far as I can tell! I've posted another comment in that thread to see if we can get some more info from Gecko and Edge, but it looks like we'll probably move down the path of showing `NaN` for the %d/f/i format specifiers, so likely nothing will change here.

Comment 11 by l...@chromium.org, Mar 22 2018

Labels: -Needs-Feedback
Status: WontFix (was: Assigned)
Gotcha, I'll close this crbug for now then.  If things change and we know what to change, let us know and we can reopen or file a new one!

Sign in to add a comment