[Webkit-unassigned] [Bug 16814] Give plugin a chance to handle ActiveX objects

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=16814


eric at webkit.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22688|review?(andersca at apple.com) |review-
               Flag|                            |




------- Comment #10 from eric at webkit.org  2008-08-27 18:07 PDT -------
(From update of attachment 22688)
Another way to do this would be to  have this code ask the platform via
PluginInfoStore.h or a similar abstraction.  That would allow platforms which
do or don't have these plugins to override these mappings at runtime, which is
probably more useful (As then you can test for the presence of the plugin)
instead of using a compile-time check.

Also, "ShouldUseEmbedForObject" is not following WebKit-style naming
conventions.  WebKit style asks that we use camelCase such as:
"shouldUseEmbedForObject()"

Also, we generally try to use early-returns instead of long ifs, which would
mean this function would become:

if (!attributes)
    return true;
for (... ) {
   ...

Again, I think that this is probably better suited as a platform-specific
question, which means we should add a function to PluginInfoStore.h which
answers this question on a per-platform basis.

Looking forward to Anders' thoughts.


-- 
Configure bugmail: https://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.