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

Side by Side Diff: Source/core/css/CSSSelector.cpp

Issue 322803004: Make all CSSSelector data members private (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix win dbg build error Created 6 years, 6 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-2003 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999-2003 Lars Knoll (knoll@kde.org)
3 * 1999 Waldo Bastian (bastian@kde.org) 3 * 1999 Waldo Bastian (bastian@kde.org)
4 * 2001 Andreas Schlapbach (schlpbch@iam.unibe.ch) 4 * 2001 Andreas Schlapbach (schlpbch@iam.unibe.ch)
5 * 2001-2003 Dirk Mueller (mueller@kde.org) 5 * 2001-2003 Dirk Mueller (mueller@kde.org)
6 * Copyright (C) 2002, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserv ed. 6 * Copyright (C) 2002, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserv ed.
7 * Copyright (C) 2008 David Smith (catfish.man@gmail.com) 7 * Copyright (C) 2008 David Smith (catfish.man@gmail.com)
8 * Copyright (C) 2010 Google Inc. All rights reserved. 8 * Copyright (C) 2010 Google Inc. All rights reserved.
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 718 matching lines...) Expand 10 before | Expand all | Expand 10 after
729 } 729 }
730 730
731 void CSSSelector::setSelectorList(PassOwnPtr<CSSSelectorList> selectorList) 731 void CSSSelector::setSelectorList(PassOwnPtr<CSSSelectorList> selectorList)
732 { 732 {
733 createRareData(); 733 createRareData();
734 m_data.m_rareData->m_selectorList = selectorList; 734 m_data.m_rareData->m_selectorList = selectorList;
735 } 735 }
736 736
737 static bool validateSubSelector(const CSSSelector* selector) 737 static bool validateSubSelector(const CSSSelector* selector)
738 { 738 {
739 switch (selector->m_match) { 739 switch (selector->match()) {
740 case CSSSelector::Tag: 740 case CSSSelector::Tag:
741 case CSSSelector::Id: 741 case CSSSelector::Id:
742 case CSSSelector::Class: 742 case CSSSelector::Class:
743 case CSSSelector::Exact: 743 case CSSSelector::Exact:
744 case CSSSelector::Set: 744 case CSSSelector::Set:
745 case CSSSelector::List: 745 case CSSSelector::List:
746 case CSSSelector::Hyphen: 746 case CSSSelector::Hyphen:
747 case CSSSelector::Contain: 747 case CSSSelector::Contain:
748 case CSSSelector::Begin: 748 case CSSSelector::Begin:
749 case CSSSelector::End: 749 case CSSSelector::End:
750 return true; 750 return true;
751 case CSSSelector::PseudoElement: 751 case CSSSelector::PseudoElement:
752 case CSSSelector::Unknown:
752 return false; 753 return false;
753 case CSSSelector::PagePseudoClass: 754 case CSSSelector::PagePseudoClass:
754 case CSSSelector::PseudoClass: 755 case CSSSelector::PseudoClass:
755 break; 756 break;
756 } 757 }
757 758
758 switch (selector->pseudoType()) { 759 switch (selector->pseudoType()) {
759 case CSSSelector::PseudoEmpty: 760 case CSSSelector::PseudoEmpty:
760 case CSSSelector::PseudoLink: 761 case CSSSelector::PseudoLink:
761 case CSSSelector::PseudoVisited: 762 case CSSSelector::PseudoVisited:
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
885 return false; 886 return false;
886 return (count - m_b) % m_a == 0; 887 return (count - m_b) % m_a == 0;
887 } else { 888 } else {
888 if (count > m_b) 889 if (count > m_b)
889 return false; 890 return false;
890 return (m_b - count) % (-m_a) == 0; 891 return (m_b - count) % (-m_a) == 0;
891 } 892 }
892 } 893 }
893 894
894 } // namespace WebCore 895 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698