[webkit-reviews] review granted: [Bug 127031] [EFL][WK1] Implement ProgressTrackerClientEfl to fix the crash : [Attachment 221230] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jan 15 01:21:03 PST 2014


Gyuyoung Kim <gyuyoung.kim at samsung.com> has granted Ryuan Choi
<ryuan.choi at samsung.com>'s request for review:
Bug 127031: [EFL][WK1] Implement ProgressTrackerClientEfl to fix the crash
https://bugs.webkit.org/show_bug.cgi?id=127031

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

------- Additional Comments from Gyuyoung Kim <gyuyoung.kim at samsung.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=221230&action=review


> Source/WebKit/efl/WebCoreSupport/ProgressTrackerClientEfl.h:44
> +    virtual void progressTrackerDestroyed() OVERRIDE;

How about using "FINAL" as well ?


More information about the webkit-reviews mailing list