Eagerly dispose of ScheduledActions (reland.)

third_party/WebKit/Source/bindings/core/v8/ScheduledAction.cpp

 /*
  * Copyright (C) 2007-2009 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 43 matching lines @@
 
 ScheduledAction* ScheduledAction::create(ScriptState* scriptState,
                                          const String& handler) {
   return new ScheduledAction(scriptState, handler);
 }
 
 DEFINE_TRACE(ScheduledAction) {
   visitor->trace(m_code);
 }
 
 ScheduledAction::~ScheduledAction() {}

[haraken, 2016/12/05 10:53:27]

Ditto.

[sof, 2016/12/05 14:07:09]

Added.

 
+void ScheduledAction::dispose() {
+  m_code.dispose();
+  m_info.Clear();
+  m_function.clear();
+  m_scriptState.clear();
+}
+
 void ScheduledAction::execute(ExecutionContext* context) {
   if (context->isDocument()) {
     LocalFrame* frame = toDocument(context)->frame();
     if (!frame) {
       DVLOG(1) << "ScheduledAction::execute " << this << ": no frame";
       return;
     }
     if (!frame->script().canExecuteScripts(AboutToExecuteScript)) {
       DVLOG(1) << "ScheduledAction::execute " << this
                << ": frame can not execute scripts";
@@ skipping 90 matching lines @@
 }
 
 void ScheduledAction::createLocalHandlesForArgs(
     Vector<v8::Local<v8::Value>>* handles) {
   handles->reserveCapacity(m_info.Size());
   for (size_t i = 0; i < m_info.Size(); ++i)
     handles->append(m_info.Get(i));
 }
 
 }  // namespace blink