[webkit-reviews] review granted: [Bug 53713] Get rid of timestamps in UpdateInfo : [Attachment 81111] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 3 14:13:13 PST 2011


mitz at webkit.org has granted Anders Carlsson <andersca at apple.com>'s request for
review:
Bug 53713: Get rid of timestamps in UpdateInfo
https://bugs.webkit.org/show_bug.cgi?id=53713

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

------- Additional Comments from mitz at webkit.org
View in context: https://bugs.webkit.org/attachment.cgi?id=81111&action=review

> Source/WebKit2/Shared/UpdateInfo.h:44
> -    UpdateInfo() : timestamp(0) { }
> +    UpdateInfo() { }

No need for this empty constructor.

> Source/WebKit2/UIProcess/DrawingAreaProxyImpl.cpp:54
> +    , m_lastDidSetSizeSequenceNumber(false)

This should be (0)


More information about the webkit-reviews mailing list