Oilpan: have PendingScripts trace their script elements.

Source/core/html/parser/HTMLDocumentParser.cpp

 /*
  * Copyright (C) 2010 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 143 matching lines @@
     ASSERT(!m_haveBackgroundParser);
     // FIXME: We should be able to ASSERT(m_speculations.isEmpty()),
     // but there are cases where that's not true currently. For example,
     // we we're told to stop parsing before we've consumed all the input.
 #endif
 }
 
 void HTMLDocumentParser::trace(Visitor* visitor)
 {
     visitor->trace(m_treeBuilder);
+    // FIXME: Oilpan: m_scriptRunner is a traced OwnPtr. This is done
+    // so as to allow its destructor to refer to its 'host' (this
+    // HTMLDocumentParser object) when tidying up. Determine if there
+    // is a way to put HTMLScriptRunner on the heap while taking care
+    // of its finalization. It cannot be a part object, as
+    // m_scriptRunner isn't set for DocumentFragment parsing.
+    //
+    // See PendingScript::trace() comment for more background.
+    visitor->trace(m_scriptRunner);
     ScriptableDocumentParser::trace(visitor);
 }
 
 void HTMLDocumentParser::pinToMainThread()
 {
     ASSERT(!m_haveBackgroundParser);
     ASSERT(!m_isPinnedToMainThread);
     m_isPinnedToMainThread = true;
     if (!m_tokenizer) {
         ASSERT(!m_token);
@@ skipping 894 matching lines @@
 void HTMLDocumentParser::setDecoder(PassOwnPtr<TextResourceDecoder> decoder)
 {
     ASSERT(decoder);
     DecodedDataDocumentParser::setDecoder(decoder);
 
     if (m_haveBackgroundParser)
         HTMLParserThread::shared()->postTask(bind(&BackgroundHTMLParser::setDecoder, m_backgroundParser, takeDecoder()));
 }
 
 }