On Chromium port, there is a compilation error when PROGRESS_TAG enabled, because RenderProgress.h is missing CurrentTime.h.
Created attachment 52524 [details] v0; should be trivial
Qt port may include it somewhere in port specific file.
Comment on attachment 52524 [details] v0; should be trivial CurrentTime.h not used within the header file. It should only be included where it’s used.
Created attachment 52525 [details] move inclusion to RenderProgress.cpp
bdash, thank you for reviewing. And you are definitely right. > CurrentTime.h not used within the header file. It should only be included > where it’s used. Fixed to move inclusion to RenderProgress.cpp I also updated the bug summary to reflect that.
Comment on attachment 52525 [details] move inclusion to RenderProgress.cpp Doesn't the file also need to include <math.h> or <wtf/MathExtras.h>?
(In reply to comment #6) > (From update of attachment 52525 [details]) > Doesn't the file also need to include <math.h> or <wtf/MathExtras.h>? I'm not sure. Chromium is successfully built only with CurrentTime.h. math.h might be included somewhere else.
Comment on attachment 52525 [details] move inclusion to RenderProgress.cpp Clearing flags on attachment: 52525 Committed r57130: <http://trac.webkit.org/changeset/57130>
All reviewed patches have been landed. Closing bug.