[PAUSED] Add a scope for disallowing style update when calling hasEditableStyle

third_party/WebKit/Source/core/dom/DocumentLifecycle.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 63 matching lines @@
     DocumentLifecycle& lifecycle) {
   m_savedCount = s_allowThrottlingCount;
   s_allowThrottlingCount = 0;
 }
 
 DocumentLifecycle::DisallowThrottlingScope::~DisallowThrottlingScope() {
   s_allowThrottlingCount = m_savedCount;
 }
 
 DocumentLifecycle::DocumentLifecycle()
-    : m_state(Uninitialized), m_detachCount(0), m_disallowTransitionCount(0) {}
+    : m_state(Uninitialized),

[yosin_UTC9, 2016/11/25 09:52:37]

How about moving these initialization into class declaration and make ctor just
|= default|?

[Xiaocheng, 2016/11/25 10:05:32]

No objection, but that's a separate issue, though.

+      m_detachCount(0),
+      m_disallowTransitionCount(0),
+      m_disallowStyleRecalcCount(0) {}
 
 DocumentLifecycle::~DocumentLifecycle() {}
 
 #if DCHECK_IS_ON()
 
 bool DocumentLifecycle::canAdvanceTo(LifecycleState nextState) const {
   if (stateTransitionDisallowed())
     return false;
 
   // We can stop from anywhere.
@@ skipping 250 matching lines @@
     DEBUG_STRING_CASE(Stopping);
     DEBUG_STRING_CASE(Stopped);
   }
 
   NOTREACHED();
   return "Unknown";
 }
 #endif
 
 }  // namespace blink