Prepare SMILTimeContainer for oilpan

Source/core/svg/animation/SVGSMILElement.cpp

 /*
  * Copyright (C) 2008 Apple 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 189 matching lines @@
 #if !ENABLE(OILPAN)
     clearResourceAndEventBaseReferences();
 #endif
     smilEndEventSender().cancelEvent(this);
     smilBeginEventSender().cancelEvent(this);
     smilRepeatEventSender().cancelEvent(this);
     smilRepeatNEventSender().cancelEvent(this);
 #if !ENABLE(OILPAN)
     clearConditions();
     if (m_timeContainer && m_targetElement && hasValidAttributeName())
         m_timeContainer->unschedule(this, m_targetElement, m_attributeName);

[haraken, 2014/05/27 08:25:56]

Just to confirm: This code is already removed in oilpan builds, but it is wrong
to remove it before landing this CL. It becomes correct after landing this CL.
Am I understanding correctly?

[kouhei (in TOK), 2014/05/28 01:00:50]

For 209-210, yes. 208 is still wrong (added comment)

[haraken, 2014/05/28 01:43:55]

Thanks. Recently we're doing a lot of complicated things, and it's hard to keep
track of them without having a FIXME. It's a nice idea to have a FIXME even if
you're planning to fix it in a follow-up.

 #endif
 }
 
 void SVGSMILElement::clearResourceAndEventBaseReferences()
 {
     document().accessSVGExtensions().removeAllTargetReferencesForElement(this);
 }
 
 void SVGSMILElement::clearConditions()
 {
@@ skipping 1121 matching lines @@
 }
 
 void SVGSMILElement::Condition::trace(Visitor* visitor)
 {
     visitor->trace(m_syncBase);
 }
 
 void SVGSMILElement::trace(Visitor* visitor)
 {
     visitor->trace(m_targetElement);
+    visitor->trace(m_timeContainer);
     visitor->trace(m_conditions);
     visitor->trace(m_syncBaseDependents);
     SVGElement::trace(visitor);
 }
 
 }