Remove scoped styles.

Source/core/html/HTMLStyleElement.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2010 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.
  *
@@ skipping 24 matching lines @@
 typedef EventSender<HTMLStyleElement> StyleEventSender;
 
 class HTMLStyleElement FINAL : public HTMLElement, private StyleElement {
     WILL_BE_USING_GARBAGE_COLLECTED_MIXIN(HTMLStyleElement);
 public:
     static PassRefPtrWillBeRawPtr<HTMLStyleElement> create(Document&, bool createdByParser);
     virtual ~HTMLStyleElement();
 
     void setType(const AtomicString&);
 
-    bool scoped() const;
-    void setScoped(bool);
     ContainerNode* scopingNode();
-    bool isRegisteredAsScoped() const
-    {
-        // Note: We cannot rely on the 'scoped' attribute still being present when this method is invoked.
-        // Therefore we cannot rely on scoped()!
-        if (m_scopedStyleRegistrationState == NotRegistered)
-            return false;
-        return true;
-    }
-
-    bool isRegisteredInShadowRoot() const
-    {
-        return m_scopedStyleRegistrationState == RegisteredInShadowRoot;
-    }
+    bool isScoped() const { return m_scoped; }

[tasak, 2014/06/03 05:57:48]

I think, it would be beter to do:
(a) return isInShadowTree() or,
(b) remove this method and replace other code (using isScope()) with
isInShadowTree().

[kochi, 2014/06/03 07:47:05]

I'll take the (b) approach.
Less "scope" terminology usage will make the code more readable.

 
     using StyleElement::sheet;
 
     bool disabled() const;
     void setDisabled(bool);
 
     void dispatchPendingEvent(StyleEventSender*);
     static void dispatchPendingLoadEvents();
 
     virtual void trace(Visitor*) OVERRIDE;
@@ skipping 10 matching lines @@
 
     virtual void finishParsingChildren() OVERRIDE;
 
     virtual bool sheetLoaded() OVERRIDE { return StyleElement::sheetLoaded(document()); }
     virtual void notifyLoadedSheetAndAllCriticalSubresources(bool errorOccurred) OVERRIDE;
     virtual void startLoadingDynamicSheet() OVERRIDE { StyleElement::startLoadingDynamicSheet(document()); }
 
     virtual const AtomicString& media() const OVERRIDE;
     virtual const AtomicString& type() const OVERRIDE;
 
-    void scopedAttributeChanged(bool);
-    void registerWithScopingNode(bool);
-    void unregisterWithScopingNode(ContainerNode*);
-
     bool m_firedLoad;
     bool m_loadedSheet;
-
-    enum ScopedStyleRegistrationState {
-        NotRegistered,
-        RegisteredAsScoped,
-        RegisteredInShadowRoot
-    };
-    ScopedStyleRegistrationState m_scopedStyleRegistrationState;
+    bool m_scoped;

[tasak, 2014/06/03 05:57:48]

We still need m_scoped? I think, we could replace this with isInShadowTree().

[kochi, 2014/06/03 07:47:05]

Removed.

 };
 
 } //namespace
 
 #endif