[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