Enhance TextStream for logging, remove subclasses, log more things
Created attachment 263359 [details] Patch
Created attachment 263361 [details] Patch
Comment on attachment 263361 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=263361&action=review LGTM, but i'll wait for thorton to r+ it. > Source/WebCore/ChangeLog:58 > + * platform/graphics/ca/PlatformCAAnimation.cpp: Added. Why did you make a new file for this? > Source/WebCore/CMakeLists.txt:2142 > + platform/animation/TimingFunction.cpp Shouldn't there be additions for the new header files? > Source/WebCore/platform/animation/TimingFunction.cpp:1 > +/* Why did you make a new file for this?
(In reply to comment #3) > Comment on attachment 263361 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=263361&action=review > > LGTM, but i'll wait for thorton to r+ it. > > > Source/WebCore/ChangeLog:58 > > + * platform/graphics/ca/PlatformCAAnimation.cpp: Added. > > Why did you make a new file for this? Because there was only the .h file, and I don't want PlatformCAAnimation.h pulling in TextStream.h. > > Source/WebCore/CMakeLists.txt:2142 > > + platform/animation/TimingFunction.cpp > > Shouldn't there be additions for the new header files? Header files are existing. > > Source/WebCore/platform/animation/TimingFunction.cpp:1 > > +/* > > Why did you make a new file for this? Ditto.
https://trac.webkit.org/r191243