Changeset 116746 in webkit

Timestamp:

May 11, 2012 3:36:31 AM (12 years ago)

Author:

shinyak@chromium.org

Message:

[Refactoring] Move Selection from DOMWindow to TreeScope.
​https://bugs.webkit.org/show_bug.cgi?id=82699

Reviewed by Ryosuke Niwa.

Since ShadowRoot will also manage its own version of DOMSelection, we would like to
share the code among Document and DOMSelection. This patch moves DOMSelection from DOMWindow to TreeScope
so that ShadowRoot can also use it.

No new tests, should covered by existing tests.

• dom/Document.cpp:

(WebCore::Document::updateFocusAppearanceTimerFired):

• dom/Document.h:

(Document):

• dom/ShadowRoot.cpp:

(WebCore::ShadowRoot::selection):

• dom/TreeScope.cpp:

(WebCore::TreeScope::~TreeScope):
(WebCore::TreeScope::getSelection):
(WebCore):

• dom/TreeScope.h:

(WebCore):
(TreeScope):

• page/DOMSelection.cpp:

(WebCore::DOMSelection::DOMSelection):
(WebCore::DOMSelection::clearTreeScope):
(WebCore):

• page/DOMSelection.h:

(WebCore):
(WebCore::DOMSelection::create):
(DOMSelection):

• page/DOMWindow.cpp:

(WebCore::DOMWindow::~DOMWindow):
(WebCore::DOMWindow::clearDOMWindowProperties):
(WebCore::DOMWindow::getSelection):

• page/DOMWindow.h:

(DOMWindow):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• dom/Document.cpp (modified) (2 diffs)
• dom/Document.h (modified) (1 diff)
• dom/ShadowRoot.cpp (modified) (1 diff)
• dom/TreeScope.cpp (modified) (4 diffs)
• dom/TreeScope.h (modified) (3 diffs)
• page/DOMSelection.cpp (modified) (1 diff)
• page/DOMSelection.h (modified) (4 diffs)
• page/DOMWindow.cpp (modified) (3 diffs)
• page/DOMWindow.h (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2012-05-11  Shinya Kawanaka  <shinyak@chromium.org>
+
+        [Refactoring] Move Selection from DOMWindow to TreeScope.
+        https://bugs.webkit.org/show_bug.cgi?id=82699
+
+        Reviewed by Ryosuke Niwa.
+
+        Since ShadowRoot will also manage its own version of DOMSelection, we would like to
+        share the code among Document and DOMSelection. This patch moves DOMSelection from DOMWindow to TreeScope
+        so that ShadowRoot can also use it.
+
+        No new tests, should covered by existing tests.
+
+        * dom/Document.cpp:
+        (WebCore::Document::updateFocusAppearanceTimerFired):
+        * dom/Document.h:
+        (Document):
+        * dom/ShadowRoot.cpp:
+        (WebCore::ShadowRoot::selection):
+        * dom/TreeScope.cpp:
+        (WebCore::TreeScope::~TreeScope):
+        (WebCore::TreeScope::getSelection):
+        (WebCore):
+        * dom/TreeScope.h:
+        (WebCore):
+        (TreeScope):
+        * page/DOMSelection.cpp:
+        (WebCore::DOMSelection::DOMSelection):
+        (WebCore::DOMSelection::clearTreeScope):
+        (WebCore):
+        * page/DOMSelection.h:
+        (WebCore):
+        (WebCore::DOMSelection::create):
+        (DOMSelection):
+        * page/DOMWindow.cpp:
+        (WebCore::DOMWindow::~DOMWindow):
+        (WebCore::DOMWindow::clearDOMWindowProperties):
+        (WebCore::DOMWindow::getSelection):
+        * page/DOMWindow.h:
+        (DOMWindow):
+
 2012-05-04  Yury Semikhatsky  <yurys@chromium.org>
 

trunk/Source/WebCore/dom/Document.cpp

@@ -47 +47 @@
 #include "CookieJar.h"
 #include "DOMImplementation.h"
+#include "DOMSelection.h"
 #include "DOMWindow.h"
 #include "DateComponents.h"
@@ -5093 +5094 @@
 }
 
-// FF method for accessing the selection added for compatibility.
-DOMSelection* Document::getSelection() const
-{
-    return frame() ? frame()->domWindow()->getSelection() : 0;
-}
-
 void Document::attachRange(Range* range)
 {

trunk/Source/WebCore/dom/Document.h

@@ -970 +970 @@
     void cancelFocusAppearanceUpdate();
         
-    // FF method for accessing the selection added for compatibility.
-    DOMSelection* getSelection() const;
-    
     // Extension for manipulating canvas drawing contexts for use in CSS
     CanvasRenderingContext* getCSSCanvasContext(const String& type, const String& name, int width, int height);

trunk/Source/WebCore/dom/ShadowRoot.cpp

@@ -152 +152 @@
 DOMSelection* ShadowRoot::selection()
 {
-    if (document() && document()->domWindow())
-        return document()->domWindow()->getSelection();
+    if (document())
+        return document()->getSelection();
     return 0;
 }

trunk/Source/WebCore/dom/TreeScope.cpp

@@ -28 +28 @@
 
 #include "ContainerNode.h"
+#include "DOMSelection.h"
+#include "DOMWindow.h"
 #include "Document.h"
 #include "Element.h"
@@ -37 +39 @@
 #include "HTMLNames.h"
 #include "Page.h"
+#include "RuntimeEnabledFeatures.h"
 #include "TreeScopeAdopter.h"
 #include <wtf/text/AtomicString.h>
@@ -55 +58 @@
 TreeScope::~TreeScope()
 {
+    if (m_selection) {
+        m_selection->clearTreeScope();
+        m_selection = 0;
+    }
 }
 
@@ -115 +122 @@
         return static_cast<HTMLMapElement*>(m_imageMapsByName.getElementByLowercasedMapName(AtomicString(name.lower()).impl(), this));
     return static_cast<HTMLMapElement*>(m_imageMapsByName.getElementByMapName(AtomicString(name).impl(), this));
+}
+
+DOMSelection* TreeScope::getSelection() const
+{
+    if (!rootNode()->document()->frame())
+        return 0;
+
+    if (m_selection)
+        return m_selection.get();
+
+    m_selection = DOMSelection::create(rootNode()->document());
+    return m_selection.get();
 }
 

trunk/Source/WebCore/dom/TreeScope.h

@@ -34 +34 @@
 
 class ContainerNode;
+class DOMSelection;
 class Element;
 class HTMLMapElement;
@@ -63 +64 @@
     bool hasNodeListCaches() const { return m_numNodeListCaches; }
 
+    DOMSelection* getSelection() const;
+
     // Find first anchor with the given name.
     // First searches for an element with the given ID, but if that fails, then looks
@@ -91 +94 @@
 
     unsigned m_numNodeListCaches;
+
+    mutable RefPtr<DOMSelection> m_selection;
 };
 

trunk/Source/WebCore/page/DOMSelection.cpp

@@ -59 +59 @@
 }
 
-DOMSelection::DOMSelection(Frame* frame)
-    : DOMWindowProperty(frame)
-{
+DOMSelection::DOMSelection(const TreeScope* treeScope)
+    : DOMWindowProperty(treeScope->rootNode()->document()->frame())
+    , m_treeScope(treeScope)
+{
+}
+
+void DOMSelection::clearTreeScope()
+{
+    m_treeScope = 0;
 }
 

trunk/Source/WebCore/page/DOMSelection.h

@@ -1 +1 @@
 /*
  * Copyright (C) 2007 Apple Inc.  All rights reserved.
+ * Copyright (C) 2012 Google Inc.  All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -39 +40 @@
 
     class Frame;
+    class Node;
     class Range;
-    class Node;
+    class TreeScope;
     class VisibleSelection;
 
@@ -47 +49 @@
     class DOMSelection : public RefCounted<DOMSelection>, public DOMWindowProperty {
     public:
-        static PassRefPtr<DOMSelection> create(Frame* frame) { return adoptRef(new DOMSelection(frame)); }
+        static PassRefPtr<DOMSelection> create(const TreeScope* treeScope) { return adoptRef(new DOMSelection(treeScope)); }
+
+        void clearTreeScope();
 
         // Safari Selection Object API
@@ -89 +93 @@
 
     private:
-        explicit DOMSelection(Frame*);
+        const TreeScope* m_treeScope;
+
+        explicit DOMSelection(const TreeScope*);
 
         // Convenience method for accessors, does not NULL check m_frame.

trunk/Source/WebCore/page/DOMWindow.cpp

@@ -400 +400 @@
     if (!m_suspendedForPageCache) {
         ASSERT(!m_screen);
-        ASSERT(!m_selection);
         ASSERT(!m_history);
         ASSERT(!m_crypto);
@@ -573 +572 @@
 
     m_screen = 0;
-    m_selection = 0;
     m_history = 0;
     m_crypto = 0;
@@ -887 +885 @@
 DOMSelection* DOMWindow::getSelection()
 {
-    if (!isCurrentlyDisplayedInFrame())
-        return 0;
-    if (!m_selection)
-        m_selection = DOMSelection::create(m_frame);
-    return m_selection.get();
+    if (!isCurrentlyDisplayedInFrame() || !m_frame)
+        return 0;
+
+    return m_frame->document()->getSelection();
 }
 

trunk/Source/WebCore/page/DOMWindow.h

@@ -429 +429 @@
 
         mutable RefPtr<Screen> m_screen;
-        mutable RefPtr<DOMSelection> m_selection;
         mutable RefPtr<History> m_history;
         mutable RefPtr<Crypto>  m_crypto;