Bug 157177

Summary: Limit ResourceTiming to WEB_TIMING enabled builds
Product: WebKit Reporter: Yoav Weiss <yoav>
Component: New BugsAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: achristensen, cdumez, commit-queue, esprehn+autocc, kondapallykalyan, ossy
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch achristensen: review+

Yoav Weiss
Reported 2016-04-29 05:44:31 PDT
Limit ResourceTiming to WEB_TIMING enabled builds
Attachments
Patch (3.04 KB, patch)
2016-04-29 05:47 PDT, Yoav Weiss 		achristensen: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Yoav Weiss
Comment 1 2016-04-29 05:47:50 PDT
Created attachment 277694 [details] Patch
Yoav Weiss
Comment 2 2016-04-29 05:50:44 PDT
When looking into why https://bugs.webkit.org/show_bug.cgi?id=157133 broke the Mac cmake build, I saw that the build breaks when WEB_TIMING is not enabled (which is the case for the Mac cmake build). This corrects such builds. As a next step, I also intend to turn on WEB_TIMING on the Mac cmake build by default.
Alex Christensen
Comment 3 2016-04-29 10:04:06 PDT
Comment on attachment 277694 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=277694&action=review > Source/WebCore/page/PerformanceResourceTiming.h:88 > +#endif // !ENABLE(WEB_TIMING) no !
Yoav Weiss
Comment 4 2016-04-29 12:21:48 PDT
Since https://bugs.webkit.org/show_bug.cgi?id=157133 got reverted, this patch can no longer be applied. I'll merge it into the one in 157133 *** This bug has been marked as a duplicate of bug 157133 ***
Note You need to log in before you can comment on or make changes to this bug.