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

Side by Side Diff: Source/core/dom/StyleEngine.cpp

Issue 180003002: Consistently use on-heap collections for StyleRuleBase descendants. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: addressed reviewers comments Created 6 years, 10 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 | Annotate | Revision Log
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, 2011, 2012 Apple Inc. All r ights reserved. 6 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011, 2012 Apple Inc. All r ights reserved.
7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/) 7 * Copyright (C) 2008, 2009 Torch Mobile Inc. All rights reserved. (http://www.t orchmobile.com/)
8 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved. 8 * Copyright (C) 2008, 2009, 2011, 2012 Google Inc. All rights reserved.
9 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 9 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
10 * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved. 10 * Copyright (C) Research In Motion Limited 2010-2011. All rights reserved.
(...skipping 521 matching lines...) Expand 10 before | Expand all | Expand 10 after
532 void StyleEngine::updateGenericFontFamilySettings() 532 void StyleEngine::updateGenericFontFamilySettings()
533 { 533 {
534 if (!m_fontSelector) 534 if (!m_fontSelector)
535 return; 535 return;
536 536
537 m_fontSelector->updateGenericFontFamilySettings(m_document); 537 m_fontSelector->updateGenericFontFamilySettings(m_document);
538 if (m_resolver) 538 if (m_resolver)
539 m_resolver->invalidateMatchedPropertiesCache(); 539 m_resolver->invalidateMatchedPropertiesCache();
540 } 540 }
541 541
542 void StyleEngine::removeFontFaceRules(const Vector<const StyleRuleFontFace*>& fo ntFaceRules) 542 void StyleEngine::removeFontFaceRules(const WillBeHeapVector<RawPtrWillBeMember< const StyleRuleFontFace> >& fontFaceRules)
543 { 543 {
544 if (!m_fontSelector) 544 if (!m_fontSelector)
545 return; 545 return;
546 546
547 FontFaceCache* cache = m_fontSelector->fontFaceCache(); 547 FontFaceCache* cache = m_fontSelector->fontFaceCache();
548 for (unsigned i = 0; i < fontFaceRules.size(); ++i) 548 for (unsigned i = 0; i < fontFaceRules.size(); ++i)
549 cache->remove(fontFaceRules[i]); 549 cache->remove(fontFaceRules[i]);
550 if (m_resolver) 550 if (m_resolver)
551 m_resolver->invalidateMatchedPropertiesCache(); 551 m_resolver->invalidateMatchedPropertiesCache();
552 } 552 }
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
610 { 610 {
611 HashMap<StyleSheetContents*, AtomicString>::iterator it = sheetToTextCache() .find(contents); 611 HashMap<StyleSheetContents*, AtomicString>::iterator it = sheetToTextCache() .find(contents);
612 if (it == sheetToTextCache().end()) 612 if (it == sheetToTextCache().end())
613 return; 613 return;
614 614
615 textToSheetCache().remove(it->value); 615 textToSheetCache().remove(it->value);
616 sheetToTextCache().remove(contents); 616 sheetToTextCache().remove(contents);
617 } 617 }
618 618
619 } 619 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698