Remove RawPtr from core/css

third_party/WebKit/Source/core/css/StyleSheetContents.cpp

 /*
  * (C) 1999-2003 Lars Knoll (knoll@kde.org)
  * Copyright (C) 2004, 2006, 2007, 2012 Apple 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
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
  * This library is distributed in the hope that it will be useful,
@@ skipping 124 matching lines @@
     // It is not the original sheet anymore.
     if (m_isMutable)
         return false;
     // If the header is valid we are not going to need to check the SecurityOrigin.
     // FIXME: Valid mime type avoids the check too.
     if (!m_hasSyntacticallyValidCSSHeader)
         return false;
     return true;
 }
 
-void StyleSheetContents::parserAppendRule(RawPtr<StyleRuleBase> rule)
+void StyleSheetContents::parserAppendRule(StyleRuleBase* rule)
 {
     if (rule->isImportRule()) {
         // Parser enforces that @import rules come before anything else
         ASSERT(m_childRules.isEmpty());
-        StyleRuleImport* importRule = toStyleRuleImport(rule.get());
+        StyleRuleImport* importRule = toStyleRuleImport(rule);
         if (importRule->mediaQueries())
             setHasMediaQueries();
         m_importRules.append(importRule);
         m_importRules.last()->setParentStyleSheet(this);
         m_importRules.last()->requestStyleSheet();
         return;
     }
 
     if (rule->isNamespaceRule()) {
         // Parser enforces that @namespace rules come before all rules other than
@@ skipping 44 matching lines @@
 {
     for (unsigned i = 0; i < m_importRules.size(); ++i) {
         ASSERT(m_importRules.at(i)->parentStyleSheet() == this);
         m_importRules[i]->clearParentStyleSheet();
     }
     m_importRules.clear();
     m_namespaceRules.clear();
     m_childRules.clear();
 }
 
-bool StyleSheetContents::wrapperInsertRule(RawPtr<StyleRuleBase> rule, unsigned index)
+bool StyleSheetContents::wrapperInsertRule(StyleRuleBase* rule, unsigned index)
 {
     ASSERT(m_isMutable);
     ASSERT_WITH_SECURITY_IMPLICATION(index <= ruleCount());
 
     if (index < m_importRules.size() || (index == m_importRules.size() && rule->isImportRule())) {
         // Inserting non-import rule before @import is not allowed.
         if (!rule->isImportRule())
             return false;
 
-        StyleRuleImport* importRule = toStyleRuleImport(rule.get());
+        StyleRuleImport* importRule = toStyleRuleImport(rule);
         if (importRule->mediaQueries())
             setHasMediaQueries();
 
         m_importRules.insert(index, importRule);
         m_importRules[index]->setParentStyleSheet(this);
         m_importRules[index]->requestStyleSheet();
         // FIXME: Stylesheet doesn't actually change meaningfully before the imported sheets are loaded.
         return true;
     }
     // Inserting @import rule after a non-import rule is not allowed.
     if (rule->isImportRule())
         return false;
 
     index -= m_importRules.size();
 
     if (index < m_namespaceRules.size() || (index == m_namespaceRules.size() && rule->isNamespaceRule())) {
         // Inserting non-namespace rules other than import rule before @namespace is not allowed.
         if (!rule->isNamespaceRule())
             return false;
         // Inserting @namespace rule when rules other than import/namespace/charset are present is not allowed.
         if (!m_childRules.isEmpty())
             return false;
 
-        StyleRuleNamespace* namespaceRule = toStyleRuleNamespace(rule.get());
+        StyleRuleNamespace* namespaceRule = toStyleRuleNamespace(rule);
         m_namespaceRules.insert(index, namespaceRule);
         // For now to be compatible with IE and Firefox if namespace rule with same prefix is added
         // irrespective of adding the rule at any index, last added rule's value is considered.
         // TODO (ramya.v@samsung.com): As per spec last valid rule should be considered,
         // which means if namespace rule is added in the middle of existing namespace rules,
         // rule which comes later in rule list with same prefix needs to be considered.
         parserAddNamespace(namespaceRule->prefix(), namespaceRule->uri());
         return true;
     }
 
@@ skipping 104 matching lines @@
 
     StyleSheetContents* root = rootStyleSheet();
     return root->m_loadingClients.isEmpty();
 }
 
 void StyleSheetContents::checkLoaded()
 {
     if (isLoading())
         return;
 
-    // Avoid |this| being deleted by scripts that run via
-    // ScriptableDocumentParser::executeScriptsWaitingForResources().
-    // See https://bugs.webkit.org/show_bug.cgi?id=95106
-    RawPtr<StyleSheetContents> protect(this);
-
     StyleSheetContents* parentSheet = parentStyleSheet();
     if (parentSheet) {
         parentSheet->checkLoaded();
         return;
     }
 
     ASSERT(this == rootStyleSheet());
     if (m_loadingClients.isEmpty())
         return;
 
     // Avoid |CSSSStyleSheet| and |ownerNode| being deleted by scripts that run via
     // ScriptableDocumentParser::executeScriptsWaitingForResources(). Also protect
     // the |CSSStyleSheet| from being deleted during iteration via the |sheetLoaded|
     // method.
     //
     // When a sheet is loaded it is moved from the set of loading clients
     // to the set of completed clients. We therefore need the copy in order to
     // not modify the set while iterating it.
     HeapVector<Member<CSSStyleSheet>> loadingClients;
     copyToVector(m_loadingClients, loadingClients);
 
     for (unsigned i = 0; i < loadingClients.size(); ++i) {
         if (loadingClients[i]->loadCompleted())
             continue;
 
         // sheetLoaded might be invoked after its owner node is removed from document.
-        if (RawPtr<Node> ownerNode = loadingClients[i]->ownerNode()) {
+        if (Node* ownerNode = loadingClients[i]->ownerNode()) {
             if (loadingClients[i]->sheetLoaded())
                 ownerNode->notifyLoadedSheetAndAllCriticalSubresources(m_didLoadErrorOccur ? Node::ErrorOccurredLoadingSubresource : Node::NoErrorLoadingSubresource);
         }
     }
 }
 
 void StyleSheetContents::notifyLoadedSheet(const CSSStyleSheetResource* sheet)
 {
     ASSERT(sheet);
     m_didLoadErrorOccur |= sheet->errorOccurred();
@@ skipping 251 matching lines @@
     visitor->trace(m_importRules);
     visitor->trace(m_namespaceRules);
     visitor->trace(m_childRules);
     visitor->trace(m_loadingClients);
     visitor->trace(m_completedClients);
     visitor->trace(m_ruleSet);
 #endif
 }
 
 } // namespace blink