[webkit-changes] cvs commit: WebCore/khtml/html html_objectimpl.cpp

[Maciej]

mjs         05/10/25 16:46:24

  Modified:    .        ChangeLog
               khtml/html html_objectimpl.cpp
  Log:
          Re-land the following fix, I tested to verify there is no performance regression:
  
  	Makes <object> tags not use image mode for svg content:
          http://bugzilla.opendarwin.org/show_bug.cgi?id=5175
  
          * khtml/html/html_objectimpl.cpp:
          (DOM::HTMLObjectElementImpl::isImageType):
  
  Revision  Changes    Path
  1.289     +10 -0     WebCore/ChangeLog
  
  Index: ChangeLog
  ===================================================================
  RCS file: /cvs/root/WebCore/ChangeLog,v
  retrieving revision 1.288
  retrieving revision 1.289
  diff -u -r1.288 -r1.289
  --- ChangeLog	25 Oct 2005 22:33:52 -0000	1.288
  +++ ChangeLog	25 Oct 2005 23:46:21 -0000	1.289
  @@ -1,3 +1,13 @@
  +2005-10-25  Maciej Stachowiak  <mjs at apple.com>
  +
  +        Re-land the following fix, I tested to verify there is no performance regression:
  +
  +	Makes <object> tags not use image mode for svg content:
  +        http://bugzilla.opendarwin.org/show_bug.cgi?id=5175
  +
  +        * khtml/html/html_objectimpl.cpp:
  +        (DOM::HTMLObjectElementImpl::isImageType):
  +
   2005-10-25  Adele Peterson  <adele at apple.com>
   
           Reviewed by Hyatt.
  
  
  
  1.85      +6 -0      WebCore/khtml/html/html_objectimpl.cpp
  
  Index: html_objectimpl.cpp
  ===================================================================
  RCS file: /cvs/root/WebCore/khtml/html/html_objectimpl.cpp,v
  retrieving revision 1.84
  retrieving revision 1.85
  diff -u -r1.84 -r1.85
  --- html_objectimpl.cpp	24 Oct 2005 21:42:14 -0000	1.84
  +++ html_objectimpl.cpp	25 Oct 2005 23:46:24 -0000	1.85
  @@ -839,6 +839,12 @@
                   serviceType = "text/plain"; // Data URLs with no MIME type are considered text/plain.
           }
       }
  +    
  +#if SVG_SUPPORT
  +    // Even if the image system supports it, we don't want to render SVGs as images.
  +    if (serviceType == "image/svg+xml")
  +        return false;
  +#endif
   
       return canRenderImageType(serviceType);
   }