HTMLParser should use current value of scriptEnabled flag

Source/core/dom/ExecutionContext.cpp

 /*
  * Copyright (C) 2008 Apple Inc. All Rights Reserved.
  * Copyright (C) 2012 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 46 matching lines @@
     int m_columnNumber;
     String m_sourceURL;
     RefPtr<ScriptCallStack> m_callStack;
 };
 
 ExecutionContext::ExecutionContext()
     : m_client(0)
     , m_sandboxFlags(SandboxNone)
     , m_circularSequentialID(0)
     , m_inDispatchErrorEvent(false)
+    , m_scriptEnabled(false)

[abarth-chromium, 2014/06/26 19:53:37]

What about other subclasses of ExecutionContext?  For example, doesn't
WorkerGlobalScope want to enable scripts?

[mmal, 2014/06/27 19:39:16]

WorkerGlobalScope seems to have its own WorkerScriptController. Only LocalFrame
instantiates ScriptController. Maybe bool m_scriptEnabled belongs more to
Document than to ExecutionContext?

     , m_activeDOMObjectsAreSuspended(false)
     , m_activeDOMObjectsAreStopped(false)
 {
 }
 
 ExecutionContext::~ExecutionContext()
 {
 }
 
 bool ExecutionContext::hasPendingActivity()
@@ skipping 249 matching lines @@
 
     RELEASE_ASSERT(m_client);
     // The SandboxOrigin is stored redundantly in the security origin.
     if (isSandboxed(SandboxOrigin) && m_client->securityContext().securityOrigin() && !m_client->securityContext().securityOrigin()->isUnique()) {
         m_client->securityContext().setSecurityOrigin(SecurityOrigin::createUnique());
         m_client->didUpdateSecurityOrigin();
     }
 }
 
 } // namespace WebCore