[webkit-reviews] review denied: [Bug 58346] Convert media controls hooks to a client interface : [Attachment 89900] Patch
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Fri Apr 15 19:48:29 PDT 2011
Dimitri Glazkov (Google) <dglazkov at chromium.org> has denied Steve Lacey
<sjl at chromium.org>'s request for review:
Bug 58346: Convert media controls hooks to a client interface
https://bugs.webkit.org/show_bug.cgi?id=58346
Attachment 89900: Patch
https://bugs.webkit.org/attachment.cgi?id=89900&action=review
------- Additional Comments from Dimitri Glazkov (Google)
<dglazkov at chromium.org>
View in context: https://bugs.webkit.org/attachment.cgi?id=89900&action=review
> Source/WebCore/html/shadow/MediaControlClient.h:38
> +class MediaControlClient : public HTMLDivElement {
This is interesting. If it's a subclass of an HTMLDivElement, it's probably not
a client. I was imagining MediaControlClient to not be a subclass of anything,
but maybe that would be too convoluted.
I think what you have is fine, but you need to get rid of the Client
designation. This is a base class, and the convention in WebKit is to use a
Base suffix. So this would be an HTMLMediaControlBase.h
More information about the webkit-reviews
mailing list