[Webkit-unassigned] [Bug 16815] Crash with navigator.plugins and navigator.mimeTypes after plugins.refresh

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Sun Feb 24 19:51:47 PST 2008


http://bugs.webkit.org/show_bug.cgi?id=16815





------- Comment #16 from darin at apple.com  2008-02-24 19:51 PDT -------
(From update of attachment 19188)
One thing I'm not happy about is all the classes that have the name XXXInfo and
XXXData. I find them confusing. Like -- what's the difference between
PluginData and PluginInfo? Anything we can do to rationalize these would be a
step in the right direction.

+        PluginData(const Page* page);

We'd normally omit the parameter name for something like this.

+        inline const Vector<PluginInfo*>& plugins() const
+        { return m_plugins; }
+        inline const Vector<MimeClassInfo*>& mimes() const
+        { return m_mimes; }

No reason for the inline keyword on these lines; it has no effect. Also,
normally the function body would be to the right rather than on a separate line
below.

Those are all my comments. I finished reviewing now.

I'd love to help you finish the patch; if I have time soon I might take it and
fix some of the issues I mentioned and do some testing on Mac OS X.


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



More information about the webkit-unassigned mailing list