Reviewed by Darin Adler.
authorap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Dec 2005 07:55:26 +0000 (07:55 +0000)
committerap <ap@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 23 Dec 2005 07:55:26 +0000 (07:55 +0000)
        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6118
        Investigate not using the frameset charset as a default for frames

        * khtml/khtml_part.cpp (KHTMLPart::write):
        Do not use the parent frame encoding as a default for sub-frames (revert
        rdar://3100151).

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

WebCore/ChangeLog
WebCore/khtml/khtml_part.cpp

index 944ad3ec41c30b19fff1957bb5954a54e699c585..2edb0bf4be96fb0b4bfd5464419caba4bf47598c 100644 (file)
@@ -1,3 +1,14 @@
+2005-12-22  Alexey Proskuryakov  <ap@nypop.com>
+
+        Reviewed by Darin Adler.
+
+        - fix http://bugzilla.opendarwin.org/show_bug.cgi?id=6118
+        Investigate not using the frameset charset as a default for frames
+
+        * khtml/khtml_part.cpp (KHTMLPart::write):
+        Do not use the parent frame encoding as a default for sub-frames (revert
+        rdar://3100151). 
+
 2005-12-22  Maciej Stachowiak  <mjs@apple.com>
 
         Reviewed by Eric.
index f64787300313960ec619f3922c91b48c7670e11d..160a0869a4bddd2af5d5ff26d85183758127924f 100644 (file)
@@ -7,6 +7,7 @@
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  *                     2001 George Staikos <staikos@kde.org>
  * Copyright (C) 2004 Apple Computer, Inc.
+ * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ -944,12 +945,9 @@ void KHTMLPart::write( const char *str, int len )
         if (!d->m_encoding.isNull())
             d->m_decoder->setEncoding(d->m_encoding.latin1(),
                 d->m_haveEncoding ? Decoder::UserChosenEncoding : Decoder::EncodingFromHTTPHeader);
-        else {
-            // Inherit the default encoding from the parent frame if there is one.
-            const char *defaultEncoding = (parentPart() && parentPart()->d->m_decoder)
-                ? parentPart()->d->m_decoder->encoding() : settings()->encoding().latin1();
-            d->m_decoder->setEncoding(defaultEncoding, Decoder::DefaultEncoding);
-        }
+        else
+            d->m_decoder->setEncoding(settings()->encoding().latin1(), Decoder::DefaultEncoding);
+
         if (d->m_doc)
             d->m_doc->setDecoder(d->m_decoder.get());
     }