Summary: Fold USER_TIMING into WEB_TIMING and make it a RuntimeEnabledFeature Nobody enabled the right set of build flags to use them anyways so there should be no web exposed change. All of the Performance Timing specifications are highly coupled. So let make WEB_TIMING encompass them all, and turn on and off individual pieces: - High Resolution Time (window.performance) - Performance Timeline (PerformanceEntry, PerformanceObserver) - Navigation Timing ("navigation" entries) - Resource Timing ("resource" entries) - User Timing ("mark" / "measure" entries)
Created attachment 299645 [details] [PATCH] Proposed Fix
Created attachment 299646 [details] [PATCH] Proposed Fix
Comment on attachment 299646 [details] [PATCH] Proposed Fix > /Volumes/Data/EWS/WebKit/Source/WebCore/bindings/js/JSPerformanceEntryCustom.cpp:38:10: fatal error: 'JSPerformanceMark.h' file not found > #include "JSPerformanceMark.h" > ^ > 1 error generated. Oops, I'll need to add these.
Created attachment 299658 [details] [PATCH] Proposed Fix
Comment on attachment 299658 [details] [PATCH] Proposed Fix Looks good!
Comment on attachment 299658 [details] [PATCH] Proposed Fix Clearing flags on attachment: 299658 Committed r211133: <http://trac.webkit.org/changeset/211133>
All reviewed patches have been landed. Closing bug.