Always Reconstruct when stylesheets change.

sky/engine/core/dom/DocumentStyleSheetCollection.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, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmobile.com/)
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
@@ skipping 49 matching lines @@
     }
 }
 
 void DocumentStyleSheetCollection::collectStyleSheets(StyleEngine* engine, DocumentStyleSheetCollector& collector)
 {
     ASSERT(document().styleEngine() == engine);
     collector.appendActiveStyleSheets(engine->documentAuthorStyleSheets());
     collectStyleSheetsFromCandidates(engine, collector);
 }
 
-void DocumentStyleSheetCollection::updateActiveStyleSheets(StyleEngine* engine, StyleResolverUpdateMode updateMode)
+void DocumentStyleSheetCollection::updateActiveStyleSheets(StyleEngine* engine)
 {
     StyleSheetCollection collection;
     ActiveDocumentStyleSheetCollector collector(collection);
     collectStyleSheets(engine, collector);
 
-    StyleSheetChange change;
-    analyzeStyleSheetChange(updateMode, collection, change);
+    engine->clearMasterResolver();
+    // FIMXE: The following depends on whether StyleRuleFontFace was modified or not.
+    // No need to always-clear font cache.
+    engine->clearFontCache();
 
-    if (change.styleResolverUpdateType == Reconstruct) {
-        engine->clearMasterResolver();
-        // FIMXE: The following depends on whether StyleRuleFontFace was modified or not.
-        // No need to always-clear font cache.
-        engine->clearFontCache();
-    } else if (StyleResolver* styleResolver = engine->resolver()) {
-        if (change.styleResolverUpdateType != Additive) {
-            ASSERT(change.styleResolverUpdateType == Reset);
-            styleResolver->resetAuthorStyle(treeScope());
-            engine->removeFontFaceRules(change.fontFaceRulesToRemove);
-            styleResolver->removePendingAuthorStyleSheets(m_activeAuthorStyleSheets);
-            styleResolver->lazyAppendAuthorStyleSheets(0, collection.activeAuthorStyleSheets());
-        } else {
-            styleResolver->lazyAppendAuthorStyleSheets(m_activeAuthorStyleSheets.size(), collection.activeAuthorStyleSheets());
-        }
-    }
-    if (change.requiresFullStyleRecalc)
-        document().setNeedsStyleRecalc(SubtreeStyleChange);
+    // TODO(esprehn): This is terrible and not needed in Sky, we should mark
+    // specific tree scopes dirty instead.
+    document().setNeedsStyleRecalc(SubtreeStyleChange);
 
     collection.swap(*this);
 
     updateUsesRemUnits();
 }
 
 }