Extract HTMLImportLoader from LinkImport

Source/core/html/HTMLImportsController.h

 /*
  * 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 22 matching lines @@
 
 #include "core/html/LinkResource.h"
 #include "core/loader/cache/CachedResourceClient.h"
 #include "core/loader/cache/CachedResourceHandle.h"
 #include "wtf/FastAllocBase.h"
 #include "wtf/PassOwnPtr.h"
 #include "wtf/Vector.h"
 
 namespace WebCore {
 
-class DocumentFragment;
+class HTMLImportLoader;
 class HTMLImportsController;
 
 //
 // A LinkResource subclasss used for @rel=import.
 //
 class LinkImport : public LinkResource, CachedResourceClient {

[dglazkov, 2013/06/19 02:53:52]

Does this still need to be a CachedResourceClient?

     WTF_MAKE_FAST_ALLOCATED;
 public:
-    enum State {
-        StatePreparing,
-        StateStarted,
-        StateError,
-        StateReady
-    };
 
     static PassRefPtr<LinkImport> create(HTMLLinkElement* owner);
 
     explicit LinkImport(HTMLLinkElement* owner);
     virtual ~LinkImport();
 
     // LinkResource
     virtual void process() OVERRIDE;
     virtual Type type() const OVERRIDE { return Import; }
     virtual void ownerRemoved() OVERRIDE;
 
     Document* importedDocument() const;
+
+private:
+    RefPtr<HTMLImportLoader> m_loader;
+};
+
+
+class HTMLImportLoader : public RefCounted<HTMLImportLoader>, public CachedResourceClient {
+public:
+    enum State {
+        StateLoading,
+        StateError,
+        StateReady
+    };
+
+    static PassRefPtr<HTMLImportLoader> create(HTMLImportsController*, const KURL&, const CachedResourceHandle<CachedScript>&);
+    virtual ~HTMLImportLoader();
+
+    Document* importedDocument() const;
     const KURL& url() const { return m_url; }
+
     void importDestroyed();
     bool isDone() const { return m_state == StateReady || m_state == StateError; }
 
 private:
-    State startRequest();
-    State finish();
-    void setState(State);
+    HTMLImportLoader(HTMLImportsController*, const KURL&, const CachedResourceHandle<CachedScript>&);
 
     // CachedResourceClient
     virtual void notifyFinished(CachedResource*) OVERRIDE;
 
+    State finish();
+    void setState(State);
+
     HTMLImportsController* m_controller;
-    LinkImport* m_ofSameLocation;
+    State m_state;
     KURL m_url;
-    State m_state;
     CachedResourceHandle<CachedScript> m_resource;
     RefPtr<Document> m_importedDocument;
 };
 
 
 class HTMLImportsController {
     WTF_MAKE_FAST_ALLOCATED;
 public:
     static PassOwnPtr<HTMLImportsController> create(Document*);
 
     explicit HTMLImportsController(Document*);
     virtual ~HTMLImportsController();
 
-    void addImport(PassRefPtr<LinkImport>);
+    void addImport(PassRefPtr<HTMLImportLoader>);
     void showSecurityErrorMessage(const String&);
-    PassRefPtr<LinkImport> findLinkFor(const KURL&) const;
+    PassRefPtr<HTMLImportLoader> findLinkFor(const KURL&) const;
     SecurityOrigin* securityOrigin() const;
     bool haveLoaded() const;
     void didLoad();
 
 private:
-
     Document* m_master;
 
     // List of import which has been loaded or being loaded.
-    typedef Vector<RefPtr<LinkImport> > ImportList;
+    typedef Vector<RefPtr<HTMLImportLoader> > ImportList;
     ImportList m_imports;
 };
 
 } // namespace WebCore
 
 #endif // HTMLImportsController_h