[webkit-reviews] review requested: [Bug 84605] [chromium] Do not clobber synchronized start times. : [Attachment 138373] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Apr 23 10:13:25 PDT 2012


vollick at chromium.org has asked	for review:
Bug 84605: [chromium] Do not clobber synchronized start times.
https://bugs.webkit.org/show_bug.cgi?id=84605

Attachment 138373: Patch
https://bugs.webkit.org/attachment.cgi?id=138373&action=review

------- Additional Comments from vollick at chromium.org
(In reply to comment #2)
> (From update of attachment 138369 [details])
> View in context:
https://bugs.webkit.org/attachment.cgi?id=138369&action=review
>
> > Source/WebCore/ChangeLog:5
> > +
>
> The comment from your test would go well here as well.
Done.
>
> > Source/WebCore/platform/graphics/chromium/cc/CCActiveAnimation.h:84
> > +	 void setStartTime(double startTime) { m_startTime = startTime;
m_hasSetStartTime = true; }
>
> one statement per line
Moved to the cpp.


More information about the webkit-reviews mailing list