Changeset 70748 in webkit

Timestamp:

Oct 27, 2010 11:15:09 PM (14 years ago)

Author:

aestes@apple.com

Message:

2010-10-27 Andy Estes <​aestes@apple.com>

Reviewed by Adam Barth.

r66156 broke AtlasCT library, formerly affected ​http://map.d.co.il/
​https://bugs.webkit.org/show_bug.cgi?id=45679

This patch removes WebKit's map of classid values to MIME types. It also
changes the behavior of object elements to render fallback content when
a non-empty classid attribute is specified, which is the behavior HTML5
specifies when a UA encounters a classid it doesn't understand.

Test: fast/replaced/object-with-non-empty-classid-triggers-fallback.html

• html/HTMLObjectElement.cpp: Remove serviceTypeForClassId(), createClassIdToTypeMap(), and the ClassIdToTypeMap typedef. (WebCore::HTMLObjectElement::updateWidget): Do not call serviceTypeForClassId() when there is no type attribute, and render fallback content if the classid attribute is non-empty.

2010-10-27 Andy Estes <​aestes@apple.com>

Reviewed by Adam Barth.

r66156 broke AtlasCT library, formerly affected ​http://map.d.co.il/
​https://bugs.webkit.org/show_bug.cgi?id=45679

• fast/dom/object-embed-plugin-scripting.html: Changed classid attributes to type attributes.
• fast/dom/object-plugin-hides-properties.html: Ditto.
• fast/replaced/object-with-non-empty-classid-triggers-fallback-expected.txt: Added.
• fast/replaced/object-with-non-empty-classid-triggers-fallback.html: Added.

Location:

trunk

Files:

• LayoutTests/ChangeLog (modified) (1 diff)
• LayoutTests/fast/dom/object-embed-plugin-scripting.html (modified) (2 diffs)
• LayoutTests/fast/dom/object-plugin-hides-properties.html (modified) (1 diff)
• LayoutTests/fast/replaced/object-with-non-empty-classid-triggers-fallback-expected.txt (added)
• LayoutTests/fast/replaced/object-with-non-empty-classid-triggers-fallback.html (added)
• WebCore/ChangeLog (modified) (1 diff)
• WebCore/html/HTMLObjectElement.cpp (modified) (3 diffs)

trunk/LayoutTests/ChangeLog

@@ +1 @@
+2010-10-27  Andy Estes  <aestes@apple.com>
+
+        Reviewed by Adam Barth.
+
+        r66156 broke AtlasCT library, formerly affected http://map.d.co.il/
+        https://bugs.webkit.org/show_bug.cgi?id=45679
+
+        * fast/dom/object-embed-plugin-scripting.html: Changed classid
+        attributes to type attributes.
+        * fast/dom/object-plugin-hides-properties.html: Ditto.
+        * fast/replaced/object-with-non-empty-classid-triggers-fallback-expected.txt: Added.
+        * fast/replaced/object-with-non-empty-classid-triggers-fallback.html: Added.
+
 2010-10-27  Kinuko Yasuda  <kinuko@chromium.org>
 

trunk/LayoutTests/fast/dom/object-embed-plugin-scripting.html

@@ -35 +35 @@
 <OBJECT 
     id="myO"
-    classid="clsid:02BF25D5-8C17-4B23-BC80-D3488ABDDC6B"
+    type="audio/mp4"
     width = 0 height = 0
     >
@@ -101 +101 @@
 <object
     name="Plugin"
-    classid="clsid:02BF25D5-8C17-4B23-BC80-D3488ABDDC6B"
+    type="audio/mp4"
     width = 0 height = 0
     >

trunk/LayoutTests/fast/dom/object-plugin-hides-properties.html

@@ -20 +20 @@
 </p>
 <p id="result"></p>
-<object id="obj" classid="clsid:02BF25D5-8C17-4B23-BC80-D3488ABDDC6B"><!-- quicktime -->
+<object id="obj" type="video/quicktime"><!-- quicktime -->
 </object>
 </body>

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-10-27  Andy Estes  <aestes@apple.com>
+
+        Reviewed by Adam Barth.
+
+        r66156 broke AtlasCT library, formerly affected http://map.d.co.il/
+        https://bugs.webkit.org/show_bug.cgi?id=45679
+
+        This patch removes WebKit's map of classid values to MIME types. It also
+        changes the behavior of object elements to render fallback content when
+        a non-empty classid attribute is specified, which is the behavior HTML5
+        specifies when a UA encounters a classid it doesn't understand.
+
+        Test: fast/replaced/object-with-non-empty-classid-triggers-fallback.html
+
+        * html/HTMLObjectElement.cpp: Remove serviceTypeForClassId(),
+        createClassIdToTypeMap(), and the ClassIdToTypeMap typedef.
+        (WebCore::HTMLObjectElement::updateWidget): Do not call
+        serviceTypeForClassId() when there is no type attribute, and render
+        fallback content if the classid attribute is non-empty.
+
 2010-10-27  Eric Uhrhane  <ericu@chromium.org>
 

trunk/WebCore/html/HTMLObjectElement.cpp

@@ -117 +117 @@
 }
 
-typedef HashMap<String, String, CaseFoldingHash> ClassIdToTypeMap;
-
-static ClassIdToTypeMap* createClassIdToTypeMap()
-{
-    ClassIdToTypeMap* map = new ClassIdToTypeMap;
-    map->add("clsid:D27CDB6E-AE6D-11CF-96B8-444553540000", "application/x-shockwave-flash");
-    map->add("clsid:CFCDAA03-8BE4-11CF-B84B-0020AFBBCCFA", "audio/x-pn-realaudio-plugin");
-    map->add("clsid:02BF25D5-8C17-4B23-BC80-D3488ABDDC6B", "video/quicktime");
-    map->add("clsid:166B1BCA-3F9C-11CF-8075-444553540000", "application/x-director");
-    map->add("clsid:6BF52A52-394A-11D3-B153-00C04F79FAA6", "application/x-mplayer2");
-    map->add("clsid:22D6F312-B0F6-11D0-94AB-0080C74C7E95", "application/x-mplayer2");
-    return map;
-}
-
-static String serviceTypeForClassId(const String& classId)
-{
-    // Return early if classId is empty (since we won't do anything below).
-    // Furthermore, if classId is null, calling get() below will crash.
-    if (classId.isEmpty())
-        return String();
-    
-    static ClassIdToTypeMap* map = createClassIdToTypeMap();
-    return map->get(classId);
-}
-
 static void mapDataParamToSrc(Vector<String>* paramNames, Vector<String>* paramValues)
 {
@@ -256 +231 @@
     if (!isFinishedParsingChildren())
         return;
-
+    
     String url = this->url();
-    
-    // If the object does not specify a MIME type via a type attribute, but does
-    // contain a classid attribute, try to map the classid to a MIME type.
     String serviceType = this->serviceType();
-    if (serviceType.isEmpty())
-        serviceType = serviceTypeForClassId(classId());
 
     // FIXME: These should be joined into a PluginParameters class.
@@ -292 +262 @@
         return;
 
+    // HTML5 says that fallback content should be rendered if a non-empty
+    // classid is specified for which the UA can't find a suitable plug-in.
+    bool hasEmptyClassId = classId().isEmpty();
+
     SubframeLoader* loader = document()->frame()->loader()->subframeLoader();
-    bool success = beforeLoadAllowedLoad && loader->requestObject(this, url, getAttribute(nameAttr), serviceType, paramNames, paramValues);
+    bool success = beforeLoadAllowedLoad && hasEmptyClassId && loader->requestObject(this, url, getAttribute(nameAttr), serviceType, paramNames, paramValues);
 
     if (!success && fallbackContent)