2009-03-24 Adele Peterson <adele@apple.com>
authoradele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Mar 2009 18:25:43 +0000 (18:25 +0000)
committeradele@apple.com <adele@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Mar 2009 18:25:43 +0000 (18:25 +0000)
        RS by Mark Rowe.

        Only build these Mail quirks checks in on the Mac.

        * dom/Document.cpp: (WebCore::disableRangeMutation):
        * html/HTMLElement.cpp: (WebCore::HTMLElement::inEitherTagList):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@41944 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebCore/ChangeLog
WebCore/dom/Document.cpp
WebCore/html/HTMLElement.cpp

index 037e92d..2e8a37b 100644 (file)
@@ -1,3 +1,12 @@
+2009-03-24  Adele Peterson  <adele@apple.com>
+
+        RS by Mark Rowe.
+
+        Only build these Mail quirks checks in on the Mac.
+
+        * dom/Document.cpp: (WebCore::disableRangeMutation):
+        * html/HTMLElement.cpp: (WebCore::HTMLElement::inEitherTagList):
+
 2009-03-24  Brent Fulgham  <bfulgham@webkit.org>
 
         Build fix, no review.
index 2d87c91..a783917 100644 (file)
@@ -277,9 +277,13 @@ static bool acceptsEditingFocus(Node *node)
 
 static bool disableRangeMutation(Page* page)
 {
+#if PLATFORM(MAC)
     // Disable Range mutation on document modifications in Tiger and Leopard Mail
     // See <rdar://problem/5865171>
     return page && (page->settings()->needsLeopardMailQuirks() || page->settings()->needsTigerMailQuirks());
+#else
+    return false;
+#endif
 }
 
 static HashSet<Document*>* changedDocuments = 0;
index 7901d45..51f5e2d 100644 (file)
@@ -921,6 +921,7 @@ bool HTMLElement::inEitherTagList(const Node* newChild)
     if (newChild->isHTMLElement()) {
         const HTMLElement* child = static_cast<const HTMLElement*>(newChild);
         if (inlineTagList()->contains(child->tagQName().localName().impl())) {
+#if PLATFORM(MAC)
             if (child->tagQName().localName() == styleTag) {
                 // Leopard Mail doesn't expect <style> to be in the body of the document, so don't allow it in that case.
                 // See <rdar://problem/6621310>
@@ -928,6 +929,7 @@ bool HTMLElement::inEitherTagList(const Node* newChild)
                 if (settings && settings->needsLeopardMailQuirks())
                     return false;
             }
+#endif
             return true;
         }
         if (blockTagList()->contains(child->tagQName().localName().impl()))