[Webkit-unassigned] [Bug 63062] Add capture attribute for HTML Media Capture

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri Feb 17 06:14:01 PST 2012


--- Comment #31 from Jongseok Yang <js45.yang at samsung.com>  2012-02-17 06:13:59 PST ---
(In reply to comment #30)

Thank you for your review. It was a good help in my patch. 

I have some question about your comment. Could you please give me an answer?

> (From update of attachment 127553 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=127553&action=review
> Thanks for the latest patch Jongseok! This looks closer to me to what the original reviewers were looking for. I am not a reviewer so cannot formally review the patch, but I have a few comments and will defer back to the original reviewers for their thoughts. Sam, would you mind taking another look at this?
> Cheers, Ben
> > Source/WebCore/platform/FileChooser.cpp:64
> > +    return static_cast<MediaCaptureChooser*>(m_chooser.get());
> Don't think the static_cast is needed?
I'll fix that.

> > Source/WebCore/platform/FileChooser.cpp:73
> > +        for (Vector<String>::const_iterator iterator = input->acceptMIMETypes().begin(); iterator != input->acceptMIMETypes().end(); ++iterator)
> Could this be replaced with something like:
> if (MIMETypeRegistry::isSupportCaptureType("image/*") && input->acceptMIMETypes().contains("image/*)) ?
Could you please check the URL? 
I'd like to implement a good solution to meet the editor's idea.
Could you please give me your idea?

> > Source/WebCore/platform/FileChooser.h:44
> > +#endif
> Maybe merge these two #ifdef blocks?
> > Source/WebCore/platform/FileChooser.h:51
> > +    Microphone,
> These should probably be in alphabetical order.
Do you mean that named types in enum should be sorted? I observed that there are many enum types that they don't meet the coding style. 
Could you please clarify the coding style?

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list