Oilpan: Remove guardRef and guardDeref from TreeScope.

Source/core/html/HTMLStyleElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2003, 2010 Apple Inc. All rights reserved.
  *           (C) 2007 Rob Buis (buis@kde.org)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
@@ skipping 40 matching lines @@
     , m_loadedSheet(false)
     , m_scopedStyleRegistrationState(NotRegistered)
 {
     ScriptWrappable::init(this);
 }
 
 HTMLStyleElement::~HTMLStyleElement()
 {
     // During tear-down, willRemove isn't called, so m_scopedStyleRegistrationState may still be RegisteredAsScoped or RegisteredInShadowRoot here.
     // Therefore we can't ASSERT(m_scopedStyleRegistrationState == NotRegistered).
+#if !ENABLE(OILPAN)
     StyleElement::clearDocumentData(document(), this);

[haraken, 2014/04/25 14:30:32]

clearDocumentData() is calling m_sheet->clearOwnerNode(). I think we still need
to call that in oilpan because the StyleSheet can outlive the HTMLStyleElement.

[Mads Ager (chromium), 2014/04/28 09:45:21]

Good point. The StyleSheet and its owner should live and die together with
Oilpan. However, there are a couple of owners that we need to move to the heap
in order to make that happen. Added the clearing of m_sheet's owner for Oilpan
with a comment that it should be removed when the sheet has a strong pointer to
its owner node.

+#endif
 
     styleLoadEventSender().cancelEvent(this);
 }
 
 PassRefPtr<HTMLStyleElement> HTMLStyleElement::create(Document& document, bool createdByParser)
 {
     return adoptRef(new HTMLStyleElement(document, createdByParser));
 }
 
 void HTMLStyleElement::parseAttribute(const QualifiedName& name, const AtomicString& value)
@@ skipping 209 matching lines @@
     return m_sheet->disabled();
 }
 
 void HTMLStyleElement::setDisabled(bool setDisabled)
 {
     if (CSSStyleSheet* styleSheet = sheet())
         styleSheet->setDisabled(setDisabled);
 }
 
 }