Block HTML Imports from loading when inserted via innerHTML.

third_party/WebKit/Source/core/html/HTMLLinkElement.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2008, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
@@ skipping 115 matching lines @@
     bool m_firedLoad;
     bool m_loadedSheet;
     bool m_fetchFollowingCORS;
 };
 
 
 class CORE_EXPORT HTMLLinkElement final : public HTMLElement, public LinkLoaderClient, private DOMSettableTokenListObserver {
     DEFINE_WRAPPERTYPEINFO();
     WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(HTMLLinkElement);
 public:
-    static PassRefPtrWillBeRawPtr<HTMLLinkElement> create(Document&, bool createdByParser);
+    static PassRefPtrWillBeRawPtr<HTMLLinkElement> create(Document&, bool createdByParser, bool alreadyStarted = false);

[dglazkov, 2016/02/05 18:25:53]

Can we use an enum bere?

https://www.chromium.org/blink/coding-style#TOC-Names

"Prefer enums to bools on function parameters if callers are likely to be
passing constants, since named constants are easier to read at the call site. An
exception to this rule is a setter function, where the name of the function
already makes clear what the boolean is."

[kouhei (in TOK), 2016/02/08 01:30:24]

+1
I prefer extending "bool createdByParser" to enum, and appending
CreatedByFragmentParser state there.

     ~HTMLLinkElement() override;
 
     KURL href() const;
     const AtomicString& rel() const;
     String media() const { return m_media; }
     String typeValue() const { return m_type; }
     String asValue() const { return m_as; }
     const LinkRelAttribute& relAttribute() const { return m_relAttribute; }
     DOMTokenList& relList() const { return static_cast<DOMTokenList&>(*m_relList); }
 
@@ skipping 23 matching lines @@
     static void dispatchPendingLoadEvents();
 
     // From LinkLoaderClient
     bool shouldLoadLink() override;
 
     // For LinkStyle
     bool loadLink(const String& type, const String& as, const KURL&);
     bool isAlternate() const { return linkStyle()->isUnset() && m_relAttribute.isAlternate(); }
     bool shouldProcessStyle() { return linkResourceToProcess() && linkStyle(); }
     bool isCreatedByParser() const { return m_createdByParser; }
+    bool isAlreadyStarted() const { return m_alreadyStarted; }
 
     // Parse the icon size attribute into |iconSizes|, make this method public
     // visible for testing purpose.
     static void parseSizesAttribute(const AtomicString& value, Vector<IntSize>& iconSizes);
 
     DECLARE_VIRTUAL_TRACE();
 
 private:
-    HTMLLinkElement(Document&, bool createdByParser);
+    HTMLLinkElement(Document&, bool createdByParser, bool alreadyStarted);

[dglazkov, 2016/02/05 18:25:53]

alreadyStarted is not clear in this context. What had started?

 
     LinkStyle* linkStyle() const;
     LinkImport* linkImport() const;
     LinkResource* linkResourceToProcess();
 
     void process();
     static void processCallback(Node*);
 
     // From Node and subclassses
     void parseAttribute(const QualifiedName&, const AtomicString&, const AtomicString&) override;
@@ skipping 23 matching lines @@
 
     String m_type;
     String m_as;
     String m_media;
     RefPtrWillBeMember<DOMSettableTokenList> m_sizes;
     Vector<IntSize> m_iconSizes;
     OwnPtrWillBeMember<RelList> m_relList;
     LinkRelAttribute m_relAttribute;
 
     bool m_createdByParser;
+    bool m_alreadyStarted;
     bool m_isInShadowTree;
 };
 
 } // namespace blink
 
 #endif // HTMLLinkElement_h