https://bugs.webkit.org/show_bug.cgi?id=149131
Reviewed by Chris Dumez.
Use more self-explaining function name "characterSetWithUTF8Fallback" instead of
"characterSetForBindings". Optimize the way to get text encoding.
* dom/Document.cpp:
(WebCore::Document::characterSetWithUTF8Fallback):
(WebCore::Document::encoding): Deleted.
(WebCore::Document::characterSetForBindings): Deleted.
* dom/Document.h:
(WebCore::Document::encoding):
(WebCore::Document::textEncoding):
* dom/Document.idl:
* loader/DocumentWriter.cpp:
(WebCore::DocumentWriter::createDecoderIfNeeded):
* loader/FormSubmission.cpp:
(WebCore::encodingFromAcceptCharset):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@189771
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2015-09-14 Dewei Zhu <dewei_zhu@apple.com>
+
+ Polish code for r189579.
+ https://bugs.webkit.org/show_bug.cgi?id=149131
+
+ Reviewed by Chris Dumez.
+
+ Use more self-explaining function name "characterSetWithUTF8Fallback" instead of
+ "characterSetForBindings". Optimize the way to get text encoding.
+
+ * dom/Document.cpp:
+ (WebCore::Document::characterSetWithUTF8Fallback):
+ (WebCore::Document::encoding): Deleted.
+ (WebCore::Document::characterSetForBindings): Deleted.
+ * dom/Document.h:
+ (WebCore::Document::encoding):
+ (WebCore::Document::textEncoding):
+ * dom/Document.idl:
+ * loader/DocumentWriter.cpp:
+ (WebCore::DocumentWriter::createDecoderIfNeeded):
+ * loader/FormSubmission.cpp:
+ (WebCore::encodingFromAcceptCharset):
+
2015-09-14 Chris Dumez <cdumez@apple.com>
Drop non-standard [IsIndex] WebKit IDL extended attribute
#include "StyleSheetList.h"
#include "SubresourceLoader.h"
#include "TextNodeTraversal.h"
-#include "TextResourceDecoder.h"
#include "TransformSource.h"
#include "TreeWalker.h"
#include "VisitedLinkState.h"
setVisualUpdatesAllowed(true);
}
-AtomicString Document::encoding() const
-{
- if (TextResourceDecoder* d = decoder())
- return d->encoding().domName();
- return nullAtom;
-}
-
-String Document::characterSetForBindings() const
+String Document::characterSetWithUTF8Fallback() const
{
AtomicString name = encoding();
if (!name.isNull())
#include "ScriptExecutionContext.h"
#include "StringWithDirection.h"
#include "StyleResolveTree.h"
+#include "TextResourceDecoder.h"
#include "Timer.h"
#include "TreeScope.h"
#include "UserActionElementSet.h"
String defaultCharset() const;
String charset() const { return Document::encoding(); }
- String characterSetForBindings() const;
+ String characterSetWithUTF8Fallback() const;
+ TextEncoding textEncoding() const;
- AtomicString encoding() const;
+ AtomicString encoding() const { return textEncoding().domName(); }
void setCharset(const String&);
didRemoveAllPendingStylesheet();
}
+inline TextEncoding Document::textEncoding() const
+{
+ if (auto* decoder = this->decoder())
+ return decoder->encoding();
+ return TextEncoding();
+}
+
#if ENABLE(TEMPLATE_ELEMENT)
inline const Document* Document::templateDocument() const
{
// DOM Level 3 Core
- [ImplementedAs=characterSetForBindings] readonly attribute DOMString inputEncoding;
+ [ImplementedAs=characterSetWithUTF8Fallback] readonly attribute DOMString inputEncoding;
[TreatReturnedNullStringAs=Null] readonly attribute DOMString xmlEncoding;
[TreatReturnedNullStringAs=Null, TreatNullAs=NullString, SetterRaisesException] attribute DOMString xmlVersion;
#if defined(LANGUAGE_JAVASCRIPT) && LANGUAGE_JAVASCRIPT
DOMSelection getSelection();
#endif
- [ImplementedAs=characterSetForBindings] readonly attribute DOMString characterSet;
+ [ImplementedAs=characterSetWithUTF8Fallback] readonly attribute DOMString characterSet;
// WebKit extensions
m_decoder->setHintEncoding(parentFrame->document()->decoder());
if (m_encoding.isEmpty()) {
if (canReferToParentFrameEncoding(m_frame, parentFrame))
- m_decoder->setEncoding(TextEncoding(parentFrame->document()->encoding()), TextResourceDecoder::EncodingFromParentFrame);
+ m_decoder->setEncoding(parentFrame->document()->textEncoding(), TextResourceDecoder::EncodingFromParentFrame);
} else {
m_decoder->setEncoding(m_encoding,
m_encodingWasChosenByUser ? TextResourceDecoder::UserChosenEncoding : TextResourceDecoder::EncodingFromHTTPHeader);
return encoding;
}
- return TextEncoding(document.encoding());
+ return document.textEncoding();
}
Ref<FormSubmission> FormSubmission::create(HTMLFormElement* form, const Attributes& attributes, PassRefPtr<Event> event, LockHistory lockHistory, FormSubmissionTrigger trigger)