Improve Paint Timing to include time up to swap time (merge into beta channel) |
||||||||
Issue descriptionCurrent paint timing (in beta currently) only captures time up to "paint" within blink. This misses the time consumed by the bulk of the rendering machinery including rasterization (CPU & GPU raster), image decode etc. We have since updated the timestamp to include time up to GPU swap (already in canary): https://codereview.chromium.org/2932593002/ This bug is to request merging this change into beta, so it will be available in M60 when the API ships. The risk of merging this CL is very low - it only affects one web perf API: https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/timing/PerformancePaintTiming.idl The value for this change is quite high for web developers and can eliminate confusion from shipping it in later milestone.
,
Jun 19 2017
,
Jun 19 2017
,
Jun 19 2017
The bug is marked as P3 or Feature. It should not be merged as M60 is in beta. Please contact the approriate milestone owner if you have questions. Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 19 2017
(oops, update bug priority. used default bug template to create) +bustamante for advise.
,
Jun 19 2017
This LGTM for merging into M60, it meets all the requirements (been in canary >24 hours, lots of tests, impact of the bug). Flipping the bit.
,
Jun 23 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 26 2017
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible! If all merges have been completed, please remove any remaining Merge-Approved labels from this issue. Thanks for your time! To disable nags, add the Disable-Nags label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jun 30 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by panicker@chromium.org
, Jun 19 2017