Stop re-signaling a slotchange event

third_party/WebKit/Source/core/dom/Node.h

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  * Copyright (C) 2004-2011, 2014 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
  * (http://www.torchmobile.com/)
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 94 matching lines @@
 enum class CustomElementState {
   // https://dom.spec.whatwg.org/#concept-element-custom-element-state
   Uncustomized = 0,
   Custom = 1 << nodeCustomElementShift,
   Undefined = 2 << nodeCustomElementShift,
   Failed = 3 << nodeCustomElementShift,
 
   NotDefinedFlag = 2 << nodeCustomElementShift,
 };
 
+enum class SlotChangeType {
+  Initial,
+  Chained,
+};

[kochi, 2016/10/31 08:39:13]

nit: clang-format seems to prefer fitting in one line,
enum class SlotChangeType { Initial, Chained };
but if presubmit check didn't complain this, I think this is okay.
(also for the trailing comma for |Chained|).

+
 class NodeRareDataBase {
  public:
   LayoutObject* layoutObject() const { return m_layoutObject; }
   void setLayoutObject(LayoutObject* layoutObject) {
     m_layoutObject = layoutObject;
   }
 
  protected:
   NodeRareDataBase(LayoutObject* layoutObject) : m_layoutObject(layoutObject) {}
 
@@ skipping 642 matching lines @@
 
   void setAlreadySpellChecked(bool flag) {
     setFlag(flag, AlreadySpellCheckedFlag);
   }
   bool isAlreadySpellChecked() { return getFlag(AlreadySpellCheckedFlag); }
 
   bool isFinishedParsingChildren() const {
     return getFlag(IsFinishedParsingChildrenFlag);
   }
 
-  void checkSlotChange();
-  void checkSlotChangeAfterInserted() { checkSlotChange(); }
-  void checkSlotChangeBeforeRemoved() { checkSlotChange(); }
+  void checkSlotChange(SlotChangeType);
+  void checkSlotChangeAfterInserted() {
+    checkSlotChange(SlotChangeType::Initial);
+  }
+  void checkSlotChangeBeforeRemoved() {
+    checkSlotChange(SlotChangeType::Initial);
+  }
 
   DECLARE_VIRTUAL_TRACE();
 
   DECLARE_VIRTUAL_TRACE_WRAPPERS();
 
   unsigned lengthOfContents() const;
 
   v8::Local<v8::Object> wrap(v8::Isolate*,
                              v8::Local<v8::Object> creationContext) override;
   v8::Local<v8::Object> associateWithWrapper(
@@ skipping 232 matching lines @@
 }  // namespace blink
 
 #ifndef NDEBUG
 // Outside the WebCore namespace for ease of invocation from gdb.
 void showNode(const blink::Node*);
 void showTree(const blink::Node*);
 void showNodePath(const blink::Node*);
 #endif
 
 #endif  // Node_h