[webkit-reviews] review requested: [Bug 56810] [EFL] Add seek forward / backword buttons to MediaControl UI : [Attachment 88769] Modified Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Thu Apr 7 22:57:41 PDT 2011
Gyuyoung Kim <gyuyoung.kim at samsung.com> has asked for review:
Bug 56810: [EFL] Add seek forward / backword buttons to MediaControl UI
https://bugs.webkit.org/show_bug.cgi?id=56810
Attachment 88769: Modified Patch
https://bugs.webkit.org/attachment.cgi?id=88769&action=review
------- Additional Comments from Gyuyoung Kim <gyuyoung.kim at samsung.com>
I add type checking condition as below,
86 + Node* node = object->node();
87 + if (!node || !node->isMediaControlElement())
88 + return 0;
Eric, how do you think this ?
More information about the webkit-reviews
mailing list