[webkit-reviews] review requested: [Bug 100370] [EFL][WK2] Remove some C'ism from EwkView : [Attachment 170657] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Oct 25 08:32:17 PDT 2012
Christophe Dumez <christophe.dumez at intel.com> has asked for review:
Bug 100370: [EFL][WK2] Remove some C'ism from EwkView
https://bugs.webkit.org/show_bug.cgi?id=100370
Attachment 170657: Patch
https://bugs.webkit.org/attachment.cgi?id=170657&action=review
------- Additional Comments from Christophe Dumez <christophe.dumez at intel.com>
Take feedback from Kenneth and Antonio into consideration:
- Renamed visible() / focused() to isVisible() / isFocused()
- Moved memory leak fix to another patch (Bug 100380)
- renamed "imf" to "inputMethod"
- Rebase on master since some AC fixes landed
More information about the webkit-reviews
mailing list