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

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=42433


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.