[webkit-reviews] review denied: [Bug 120598] [EFL] Synchronize caret interval with that of EFL entry : [Attachment 210275] Patch

[bugzilla-daemon at webkit.org]

Christophe Dumez <dchris at gmail.com> has denied Jinwoo Song
<jinwoo7.song at samsung.com>'s request for review:
Bug 120598: [EFL] Synchronize caret interval with that of EFL entry
https://bugs.webkit.org/show_bug.cgi?id=120598

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

------- Additional Comments from Christophe Dumez <dchris at gmail.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=210275&action=review


> Source/WebCore/platform/efl/RenderThemeEfl.h:119
> +    virtual double caretBlinkInterval() const { return 0.6; }

Would you please move the implementation to the cpp file, add a global constant
for the magic 0.6 value with a comment explaining why we use this value?