Make the parser an implementation detail of the Document

sky/engine/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 85 matching lines @@
 HTMLImportLoader::State HTMLImportLoader::startWritingAndParsing(mojo::URLResponsePtr response)
 {
     ASSERT(!m_imports.isEmpty());
     WeakPtr<Document> contextDocument = m_controller->master()->contextDocument();
     ASSERT(contextDocument.get());
     KURL url(ParsedURLString, String::fromUTF8(response->url));
     DocumentInit init = DocumentInit(url, 0, contextDocument, m_controller)
         .withRegistrationContext(m_controller->master()->registrationContext());
     m_document = HTMLDocument::create(init);
     m_module = Module::create(contextDocument.get(), nullptr, m_document.get(), url.string());
-    m_document->startParsing();
-    m_document->parser()->parse(response->body.Pass());
+    m_document->startParsing()->parse(response->body.Pass());

[eseidel, 2014/11/12 19:02:14]

Should this just take the mojo pipe in the startParsing() call?

[abarth-chromium, 2014/11/12 21:13:57]

Yes, that would be sensible, but in a later CL in this series, I'm adding
another argument.  We can make startParsing pass these arguments through to the
parser, but it seemed easier to just return the parser...

     return StateLoading;
 }
 
 HTMLImportLoader::State HTMLImportLoader::finishWriting()
 {
     return StateWritten;
 }
 
 HTMLImportLoader::State HTMLImportLoader::finishParsing()
 {
@@ skipping 82 matching lines @@
 {
     visitor->trace(m_controller);
 #if ENABLE(OILPAN)
     visitor->trace(m_imports);
 #endif
     visitor->trace(m_document);
     visitor->trace(m_microtaskQueue);
 }
 
 } // namespace blink