[Webkit-unassigned] [Bug 42433] [Qt] Enable Web Timing for Qt

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Nov 11 16:43:32 PST 2010


Tony Gentilcore <tonyg at chromium.org> changed:

           What    |Removed                     |Added
                 CC|                            |simonjam at chromium.org

--- Comment #2 from Tony Gentilcore <tonyg at chromium.org>  2010-11-11 16:43:32 PST ---
I'm excited to see this getting enabled, but backfilling the times from the network stack means that these attributes report incorrect values with no hint to the user. It also isn't compliant w/ the spec.

Assuming modifying the network stack isn't a reasonable option, I'd recommend one of two approaches:
1. Keep zeroes for these times and just keep the failing tests in the Skipped file.
2. Do what is proposed in https://bugs.webkit.org/show_bug.cgi?id=48923 and make the webcore times enabled by default and just guard the network times with the enable web timing flag. This would result in those properties not being defined at all rather than backfilling or exposing zeroes.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list