[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