[oilpan] The Node hierarchy should have correct accept method chains

Source/core/html/shadow/MediaControlElements.cpp

 /*
  * Copyright (C) 2008, 2009, 2010, 2011 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.
@@ skipping 309 matching lines @@
     ASSERT(controls);
 
     Handle<MediaControlPanelMuteButtonElement> button = adoptTreeShared(new MediaControlPanelMuteButtonElement(document, controls));
     button->ensureUserAgentShadowRoot();
     button->setType("button");
     return button;
 }
 
 void MediaControlPanelMuteButtonElement::acceptHeapVisitor(Visitor* visitor) const
 {
+    visitor->visit(m_controls);
     MediaControlMuteButtonElement::acceptHeapVisitor(visitor);
 }
 
 void MediaControlPanelMuteButtonElement::defaultEventHandler(Event* event)
 {
     if (event->type() == eventNames().mouseoverEvent)
         m_controls->showVolumeSlider();
 
     MediaControlMuteButtonElement::defaultEventHandler(event);
 }
@@ skipping 208 matching lines @@
 
 Result<MediaControlClosedCaptionsTrackListElement> MediaControlClosedCaptionsTrackListElement::create(Handle<Document> document, MediaControls* controls)
 {
     ASSERT(controls);
     Handle<MediaControlClosedCaptionsTrackListElement> element = adoptTreeShared(new MediaControlClosedCaptionsTrackListElement(document, controls));
     return element;
 }
 
 void MediaControlClosedCaptionsTrackListElement::acceptHeapVisitor(Visitor* visitor) const
 {
+    visitor->visit(m_menuItems);
+    visitor->visit(m_controls);
     MediaControlDivElement::acceptHeapVisitor(visitor);
 }
 
 void MediaControlClosedCaptionsTrackListElement::defaultEventHandler(Event* event)
 {
     if (event->type() == eventNames().clickEvent) {
         Node* target = event->target()->toNode();
         if (!target || !target->isElementNode())
             return;
 
@@ skipping 48 matching lines @@
     TextTrackList* trackList = mediaElement->textTracks();
 
     if (!trackList || !trackList->length())
         return;
 
     if (m_trackListHasChanged)
         rebuildTrackListMenu();
 
     bool captionsVisible = mediaElement->closedCaptionsVisible();
     for (unsigned i = 0, length = m_menuItems.size(); i < length; ++i) {
-        RefPtr<Element> trackItem = m_menuItems[i];
+        Handle<Element> trackItem = m_menuItems[i];
 
-        int trackIndex = trackListIndexForElement(trackItem.get());
+        int trackIndex = trackListIndexForElement(trackItem.raw());
         if (trackIndex == HTMLMediaElement::textTracksIndexNotFound())
             continue;
 
         if (trackIndex == HTMLMediaElement::textTracksOffIndex()) {
             if (captionsVisible)
                 trackItem->classList()->remove(selectedClassValue, ASSERT_NO_EXCEPTION);
             else
                 trackItem->classList()->add(selectedClassValue, ASSERT_NO_EXCEPTION);
             continue;
         }
 
         RefPtr<TextTrack> textTrack;
-        MenuItemToTrackMap::iterator iter = m_menuToTrackMap.find(trackItem.get());
+        MenuItemToTrackMap::iterator iter = m_menuToTrackMap.find(trackItem.raw());
         if (iter == m_menuToTrackMap.end())
             continue;
         textTrack = iter->value;
         if (!textTrack)
             continue;
         if (textTrack->mode() == TextTrack::showingKeyword())
             trackItem->classList()->add(selectedClassValue, ASSERT_NO_EXCEPTION);
         else
             trackItem->classList()->remove(selectedClassValue, ASSERT_NO_EXCEPTION);
     }
@@ skipping 26 matching lines @@
     RefPtr<Element> captionsHeader = doc->createElement(h3Tag, ASSERT_NO_EXCEPTION);
     captionsHeader->appendChild(doc->createTextNode(textTrackSubtitlesText()));
     appendChild(captionsHeader);
     RefPtr<Element> captionsMenuList = doc->createElement(ulTag, ASSERT_NO_EXCEPTION);
 
     RefPtr<Element> menuItem;
     menuItem = doc->createElement(liTag, ASSERT_NO_EXCEPTION);
     menuItem->appendChild(doc->createTextNode(textTrackOffText()));
     menuItem->setAttribute(trackIndexAttributeName(), textTracksOffAttrValue, ASSERT_NO_EXCEPTION);
     captionsMenuList->appendChild(menuItem);
-    m_menuItems.append(menuItem);
+    m_menuItems.append(Handle<Element>(menuItem.get()));
 
     for (unsigned i = 0, length = tracksForMenu.size(); i < length; ++i) {
         RefPtr<TextTrack> textTrack = tracksForMenu[i];
         menuItem = doc->createElement(liTag, ASSERT_NO_EXCEPTION);
 
         // Add a custom attribute to the <li> element which will allow
         // us to easily associate the user tapping here with the
         // track. Since this list is rebuilt if the tracks change, we
         // should always be in sync.
         menuItem->setAttribute(trackIndexAttributeName(), String::number(i), ASSERT_NO_EXCEPTION);
 
         menuItem->appendChild(doc->createTextNode(captionsUserPreferences->displayNameForTrack(textTrack.get())));
 
         captionsMenuList->appendChild(menuItem);
-        m_menuItems.append(menuItem);
+        m_menuItems.append(Handle<Element>(menuItem.get()));
         m_menuToTrackMap.add(menuItem, textTrack);
     }
 
     appendChild(captionsMenuList);
 
     updateDisplay();
 }
 
 // ----------------------------
 
 MediaControlTimelineElement::MediaControlTimelineElement(Handle<Document> document, MediaControls* controls)
     : MediaControlInputElement(document, MediaSlider)
     , m_controls(controls)
 {
 }
 
 Result<MediaControlTimelineElement> MediaControlTimelineElement::create(Handle<Document> document, MediaControls* controls)
 {
     ASSERT(controls);
 
     Handle<MediaControlTimelineElement> timeline = adoptTreeShared(new MediaControlTimelineElement(document, controls));
     timeline->ensureUserAgentShadowRoot();
     timeline->setType("range");
     timeline->setAttribute(precisionAttr, "float");
     return timeline;
 }
 
 void MediaControlTimelineElement::acceptHeapVisitor(Visitor* visitor) const
 {
+    visitor->visit(m_controls);
     MediaControlInputElement::acceptHeapVisitor(visitor);
 }
 
 void MediaControlTimelineElement::defaultEventHandler(Event* event)
 {
     // Left button is 0. Rejects mouse events not from left button.
     if (event->isMouseEvent() && static_cast<MouseEvent*>(event)->button())
         return;
 
     if (!attached())
@@ skipping 380 matching lines @@
 }
 
 void MediaControlTextTrackContainerElement::textTrackRepresentationBoundsChanged(const IntRect&)
 {
     updateSizes();
 }
 
 // ----------------------------
 
 } // namespace WebCore