Set encoding to UTF-8 for HTML Imports if no encoding is specified in the HTTP response

Source/core/html/imports/HTMLImportLoader.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 76 matching lines @@
 
     setState(finishWriting());
 }
 
 HTMLImportLoader::State HTMLImportLoader::startWritingAndParsing(const ResourceResponse& response)
 {
     DocumentInit init = DocumentInit(response.url(), 0, m_import->master()->contextDocument(), m_import)
         .withRegistrationContext(m_import->master()->registrationContext());
     m_importedDocument = HTMLDocument::create(init);
     m_importedDocument->initContentSecurityPolicy(ContentSecurityPolicyResponseHeaders(response));
-    m_writer = DocumentWriter::create(m_importedDocument.get(), response.mimeType(), response.textEncodingName());
+    AtomicString textEncoding = response.textEncodingName();
+    // FIXME: Should we test for more than emptiness here? For example, we could require that the encoding be some valid encoding.
+    if (textEncoding.isEmpty())
+        textEncoding = "UTF-8";
+    m_writer = DocumentWriter::create(m_importedDocument.get(), response.mimeType(), textEncoding);

[abarth-chromium, 2014/02/24 22:30:27]

e.g., just pass "UTF-8" unconditionally here

 
     return StateLoading;
 }
 
 HTMLImportLoader::State HTMLImportLoader::finishWriting()
 {
     return StateWritten;
 }
 
 HTMLImportLoader::State HTMLImportLoader::finishParsing()
@@ skipping 48 matching lines @@
         client->didFinishLoading();
 }
 
 void HTMLImportLoader::removeClient(HTMLImportLoaderClient* client)
 {
     ASSERT(kNotFound != m_clients.find(client));
     m_clients.remove(m_clients.find(client));
 }
 
 } // namespace WebCore