Oilpan: Make the Node hierarchy RefCountedGarbageCollected instead of TreeShared.

Source/core/html/forms/ValidationMessage.cpp

Index: Source/core/html/forms/ValidationMessage.cpp
diff --git a/Source/core/html/forms/ValidationMessage.cpp b/Source/core/html/forms/ValidationMessage.cpp
index 78046464722525708059d86d852328735944ea5b..ab4ebc1b113fa65b8547331806606202647fb30c 100644
--- a/Source/core/html/forms/ValidationMessage.cpp
+++ b/Source/core/html/forms/ValidationMessage.cpp
@@ -46,8 +46,10 @@ ALWAYS_INLINE ValidationMessage::ValidationMessage(HTMLFormControlElement* eleme
 
 ValidationMessage::~ValidationMessage()
 {
+#if !ENABLE(OILPAN)
     if (ValidationMessageClient* client = validationMessageClient())
         client->hideValidationMessage(*m_element);

[haraken, 2014/05/06 04:20:16]

Now you're calling hideValidationMessage() in
HTMLFormControllerElement::removedFrom(). So, for consistency, I'd remove
hideValidationMessage() from here and instead add:

~HTMLFormControllerElement() {
#if !ENABLE(OILPAN)
  hideValidationMessage();
#endif
}

[Mads Ager (chromium), 2014/05/06 08:26:00]

Yeah, let me do that. There is a comment in the ValidationMessage.h file that
ValidationMessage should be removed in any case. Doing this will move a bit in
that direction.

+#endif
 }
 
 PassOwnPtr<ValidationMessage> ValidationMessage::create(HTMLFormControlElement* element)