Remove Document::shouldScheduleLayout.

sky/engine/core/dom/Document.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved.
@@ skipping 923 matching lines @@
 
 bool Document::shouldScheduleRenderTreeUpdate() const
 {
     if (!isActive())
         return false;
     if (inStyleRecalc())
         return false;
     // InPreLayout will recalc style itself. There's no reason to schedule another recalc.
     if (m_lifecycle.state() == DocumentLifecycle::InPreLayout)
         return false;
-    if (!shouldScheduleLayout())
+    if (!isActive())
         return false;
     return true;

[abarth-chromium, 2015/01/13 07:17:41]

return isActive()

[esprehn, 2015/01/13 07:31:29]

Heh, there's already a check for isActive() in this same function, I'll just
remove this line.

 }
 
 void Document::scheduleRenderTreeUpdate()
 {
     ASSERT(!hasPendingStyleRecalc());
     ASSERT(shouldScheduleRenderTreeUpdate());
     ASSERT(needsRenderTreeUpdate());
 
     page()->animator().scheduleVisualUpdate();
     m_lifecycle.ensureStateAtMost(DocumentLifecycle::VisualUpdatePending);
@@ skipping 477 matching lines @@
 }
 
 void Document::setParsing(bool b)
 {
     m_isParsing = b;
 
     if (m_isParsing && !m_elementDataCache)
         m_elementDataCache = ElementDataCache::create();
 }
 
-bool Document::shouldScheduleLayout() const
-{
-   return isActive();
-}
-
 int Document::elapsedTime() const
 {
     return static_cast<int>((currentTime() - m_startTime) * 1000);
 }
 
 const KURL& Document::virtualURL() const
 {
     return m_url;
 }
 
@@ skipping 1193 matching lines @@
 using namespace blink;
 void showLiveDocumentInstances()
 {
     WeakDocumentSet& set = liveDocumentSet();
     fprintf(stderr, "There are %u documents currently alive:\n", set.size());
     for (WeakDocumentSet::const_iterator it = set.begin(); it != set.end(); ++it) {
         fprintf(stderr, "- Document %p URL: %s\n", *it, (*it)->url().string().utf8().data());
     }
 }
 #endif