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

Source/core/html/HTMLSelectElement.h

 /*
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies).
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2000 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2009, 2010, 2011 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 18 matching lines @@
 #include "core/dom/Event.h"
 #include "core/html/HTMLFormControlElementWithState.h"
 #include "core/html/HTMLOptionsCollection.h"
 #include "core/html/TypeAhead.h"
 #include <wtf/Vector.h>
 
 namespace WebCore {
 
 class HTMLOptionElement;
 
 class HTMLSelectElement : public HTMLFormControlElementWithState, public TypeAheadDataSource {

[Mads Ager (chromium), 2013/07/25 13:35:12]

We will probably have TypeAheadDataSource in members somewhere (in TypeAhead?).
It has no members so we don't have to visit it, however, we do need to have a
trait for it to adjust the pointers when we visit them through members because
it will not point to the start of the object.

Nothing needs to be done for this change.

 public:
     static Result<HTMLSelectElement> create(const QualifiedName&, Handle<Document>, HTMLFormElement*);
 
     int selectedIndex() const;
     void setSelectedIndex(int);
 
     void optionSelectedByUser(int index, bool dispatchChangeEvent, bool allowMultipleSelection = false);
 
     // For ValidityState
     virtual String validationMessage() const OVERRIDE;
@@ skipping 48 matching lines @@
     int optionToListIndex(int optionIndex) const;
     int activeSelectionStartListIndex() const;
     int activeSelectionEndListIndex() const;
     void setActiveSelectionAnchorIndex(int);
     void setActiveSelectionEndIndex(int);
     void updateListBoxSelection(bool deselectOtherOptions);
     
     // For use in the implementation of HTMLOptionElement.
     void optionSelectionStateChanged(HTMLOptionElement*, bool optionIsSelected);
 
-    virtual void acceptHeapVisitor(Visitor*) const OVERRIDE;
-
 protected:
     HTMLSelectElement(const QualifiedName&, Handle<Document>, HTMLFormElement*);
 
+    virtual void acceptHeapVisitor(Visitor*) const OVERRIDE;
+
 private:
     virtual const AtomicString& formControlType() const;
     
     virtual bool isKeyboardFocusable(KeyboardEvent*) const;
     virtual bool isMouseFocusable() const;
 
     virtual void dispatchFocusEvent(PassRefPtr<Node> oldFocusedNode, FocusDirection) OVERRIDE;
     virtual void dispatchBlurEvent(PassRefPtr<Node> newFocusedNode);
     
     virtual bool canStartSelection() const { return false; }
@@ skipping 96 matching lines @@
 {
     ASSERT_WITH_SECURITY_IMPLICATION(!node || isHTMLSelectElement(node));
     return adoptRawResult(static_cast<const HTMLSelectElement*>(node));
 }
 
 void toHTMLSelectElement(const HTMLSelectElement*); // This overload will catch anyone doing an unnecessary cast.
 
 } // namespace
 
 #endif