Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(373)

Side by Side Diff: third_party/WebKit/Source/core/dom/StyleEngine.h

Issue 2884993002: Don't trigger full active style update on styleSheets access. (Closed)
Patch Set: Not clearing document_scope_dirty_ flag in import styleSheets anymore. Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org) 5 * (C) 2006 Alexey Proskuryakov (ap@webkit.org)
6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All
7 * rights reserved. 7 * rights reserved.
8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. 8 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved.
9 * (http://www.torchmobile.com/) 9 * (http://www.torchmobile.com/)
10 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 10 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
278 // CSSFontSelectorClient implementation. 278 // CSSFontSelectorClient implementation.
279 void FontsNeedUpdate(CSSFontSelector*) override; 279 void FontsNeedUpdate(CSSFontSelector*) override;
280 280
281 private: 281 private:
282 StyleEngine(Document&); 282 StyleEngine(Document&);
283 bool NeedsActiveStyleSheetUpdate() const { 283 bool NeedsActiveStyleSheetUpdate() const {
284 return all_tree_scopes_dirty_ || tree_scopes_removed_ || 284 return all_tree_scopes_dirty_ || tree_scopes_removed_ ||
285 document_scope_dirty_ || dirty_tree_scopes_.size(); 285 document_scope_dirty_ || dirty_tree_scopes_.size();
286 } 286 }
287 287
288 TreeScopeStyleSheetCollection* EnsureStyleSheetCollectionFor(TreeScope&); 288 TreeScopeStyleSheetCollection& EnsureStyleSheetCollectionFor(TreeScope&);
289 TreeScopeStyleSheetCollection* StyleSheetCollectionFor(TreeScope&); 289 TreeScopeStyleSheetCollection* StyleSheetCollectionFor(TreeScope&);
290 bool ShouldUpdateDocumentStyleSheetCollection() const; 290 bool ShouldUpdateDocumentStyleSheetCollection() const;
291 bool ShouldUpdateShadowTreeStyleSheetCollection() const; 291 bool ShouldUpdateShadowTreeStyleSheetCollection() const;
292 292
293 void MarkDocumentDirty(); 293 void MarkDocumentDirty();
294 void MarkTreeScopeDirty(TreeScope&); 294 void MarkTreeScopeDirty(TreeScope&);
295 295
296 bool IsMaster() const { return is_master_; } 296 bool IsMaster() const { return is_master_; }
297 Document* Master(); 297 Document* Master();
298 Document& GetDocument() const { return *document_; } 298 Document& GetDocument() const { return *document_; }
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
397 397
398 std::unique_ptr<StyleResolverStats> style_resolver_stats_; 398 std::unique_ptr<StyleResolverStats> style_resolver_stats_;
399 unsigned style_for_element_count_ = 0; 399 unsigned style_for_element_count_ = 0;
400 400
401 friend class StyleEngineTest; 401 friend class StyleEngineTest;
402 }; 402 };
403 403
404 } // namespace blink 404 } // namespace blink
405 405
406 #endif 406 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698