New issue
Advanced search Search tips

Issue 914441 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

[UserTimingL2] Return undefined for mark, measure API

Project Member Reported by maxlg@chromium.org, Dec 12

Issue description

Reproduction steps:
1.performance.mark(1)
2.performance.mark(2)
3.performance.measure(3, 1, 2), check the return

Expectation:
The result should be undefined

Actual:
The result is null
 
Description: Show this description
Bug filed as found by discussion at https://github.com/w3c/user-timing/pull/46
The test mark-measure-return-null.html must be changed.
Status: Started (was: Assigned)
Draw back to assigned. Discussed with Nicolas, we should change the tests to check undefined instead of null. The change will cause Chrome to fail, but the failure will go away when we ship L3. Also, I am not sure whether we can define the API to return undefined (instead of null) and an object.
Status: Assigned (was: Started)
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 14

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

commit fb87be9f5c8801ba7ebb5478ad7f3dd447a33f3c
Author: Nicolas Pena <npm@chromium.org>
Date: Fri Dec 14 16:59:56 2018

[UserTiming] Fix mark-measure-return-null test

The test was incorrect per L2 spec: a void method 'returns' an undefined
value, not a null object. This CL changes the test to show how to
perform feature detection and compares L2 and L3 behavior.

Bug: 914441
Change-Id: Id7173b2693d5bd42013dcdd25266d64fa8956e6a
Reviewed-on: https://chromium-review.googlesource.com/c/1377129
Commit-Queue: Nicolás Peña Moreno <npm@chromium.org>
Reviewed-by: Yoav Weiss <yoav@yoav.ws>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Cr-Commit-Position: refs/heads/master@{#616712}
[modify] https://crrev.com/fb87be9f5c8801ba7ebb5478ad7f3dd447a33f3c/third_party/blink/renderer/core/timing/performance.idl
[modify] https://crrev.com/fb87be9f5c8801ba7ebb5478ad7f3dd447a33f3c/third_party/blink/web_tests/TestExpectations
[add] https://crrev.com/fb87be9f5c8801ba7ebb5478ad7f3dd447a33f3c/third_party/blink/web_tests/external/wpt/user-timing/mark-measure-feature-detection.html
[delete] https://crrev.com/3810f7e7bd1e65e6508e3cda6578cfa66b14e72d/third_party/blink/web_tests/external/wpt/user-timing/mark-measure-return-null.html

Sign in to add a comment