Bug 1260651 part.41 Rename mozilla::dom::AutoReplaceContainerSelNotify to mozilla::AutoReplaceContainerSelNotify r=mccr8 draft
authorMasayuki Nakano <masayuki@d-toybox.com>
Fri, 24 Jun 2016 15:14:16 +0900
changeset 385873 7d56f47a088dabd34c8febae6e2a9adf1fc5e8e8
parent 385872 3a1abc1ee84024c4d447d5d3a6f7a923c7a7e274
child 385874 8453c4b0359e6e595e8c32cc165ce8d7340afb60
push id22587
push usermasayuki@d-toybox.com
push dateSat, 09 Jul 2016 06:59:31 +0000
reviewersmccr8
bugs1260651
milestone50.0a1
Bug 1260651 part.41 Rename mozilla::dom::AutoReplaceContainerSelNotify to mozilla::AutoReplaceContainerSelNotify r=mccr8 MozReview-Commit-ID: 2EB2yQOpjO9
editor/libeditor/SelectionState.h
--- a/editor/libeditor/SelectionState.h
+++ b/editor/libeditor/SelectionState.h
@@ -223,47 +223,45 @@ public:
       *mNode = mRangeItem->startNode;
     } else {
       *mDOMNode = GetAsDOMNode(mRangeItem->startNode);
     }
     *mOffset = mRangeItem->startOffset;
   }
 };
 
-/******************************************************************************
- * another helper class for SelectionState.  stack based class for doing
+/**
+ * Another helper class for SelectionState.  Stack based class for doing
  * Will/DidReplaceContainer()
  */
 
-namespace dom {
-class MOZ_STACK_CLASS AutoReplaceContainerSelNotify
+class MOZ_STACK_CLASS AutoReplaceContainerSelNotify final
 {
-  private:
-    RangeUpdater& mRU;
-    Element* mOriginalElement;
-    Element* mNewElement;
+private:
+  RangeUpdater& mRangeUpdater;
+  dom::Element* mOriginalElement;
+  dom::Element* mNewElement;
 
-  public:
-    AutoReplaceContainerSelNotify(RangeUpdater& aRangeUpdater,
-                                  Element* aOriginalElement,
-                                  Element* aNewElement)
-      : mRU(aRangeUpdater)
-      , mOriginalElement(aOriginalElement)
-      , mNewElement(aNewElement)
-    {
-      mRU.WillReplaceContainer();
-    }
+public:
+  AutoReplaceContainerSelNotify(RangeUpdater& aRangeUpdater,
+                                dom::Element* aOriginalElement,
+                                dom::Element* aNewElement)
+    : mRangeUpdater(aRangeUpdater)
+    , mOriginalElement(aOriginalElement)
+    , mNewElement(aNewElement)
+  {
+    mRangeUpdater.WillReplaceContainer();
+  }
 
-    ~AutoReplaceContainerSelNotify()
-    {
-      mRU.DidReplaceContainer(mOriginalElement, mNewElement);
-    }
+  ~AutoReplaceContainerSelNotify()
+  {
+    mRangeUpdater.DidReplaceContainer(mOriginalElement, mNewElement);
+  }
 };
 
-} // namespace dom
 } // namespace mozilla
 
 
 /***************************************************************************
  * another helper class for SelectionState.  stack based class for doing
  * Will/DidRemoveContainer()
  */