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

Side by Side Diff: Source/core/html/HTMLFormControlsCollection.h

Issue 159503003: Do not cause unnecessary node lists invalidation on id/name attribute change (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase 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
« no previous file with comments | « Source/core/html/HTMLCollection.cpp ('k') | Source/core/html/HTMLFormControlsCollection.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserv ed. 4 * Copyright (C) 2003, 2004, 2005, 2006, 2007, 2008 Apple Inc. All rights reserv ed.
5 * Copyright (C) 2014 Samsung Electronics. All rights reserved. 5 * Copyright (C) 2014 Samsung Electronics. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 static PassRefPtr<HTMLFormControlsCollection> create(ContainerNode*, Collect ionType); 42 static PassRefPtr<HTMLFormControlsCollection> create(ContainerNode*, Collect ionType);
43 43
44 virtual ~HTMLFormControlsCollection(); 44 virtual ~HTMLFormControlsCollection();
45 45
46 virtual Element* namedItem(const AtomicString& name) const OVERRIDE; 46 virtual Element* namedItem(const AtomicString& name) const OVERRIDE;
47 void namedGetter(const AtomicString& name, bool& radioNodeListEnabled, RefPt r<RadioNodeList>&, bool& elementEnabled, RefPtr<Element>&); 47 void namedGetter(const AtomicString& name, bool& radioNodeListEnabled, RefPt r<RadioNodeList>&, bool& elementEnabled, RefPtr<Element>&);
48 48
49 private: 49 private:
50 explicit HTMLFormControlsCollection(ContainerNode*); 50 explicit HTMLFormControlsCollection(ContainerNode*);
51 51
52 virtual void updateNameCache() const OVERRIDE; 52 virtual void updateIdNameCache() const OVERRIDE;
53 virtual void supportedPropertyNames(Vector<String>& names) OVERRIDE; 53 virtual void supportedPropertyNames(Vector<String>& names) OVERRIDE;
54 54
55 const Vector<FormAssociatedElement*>& formControlElements() const; 55 const Vector<FormAssociatedElement*>& formControlElements() const;
56 const Vector<HTMLImageElement*>& formImageElements() const; 56 const Vector<HTMLImageElement*>& formImageElements() const;
57 virtual Element* virtualItemAfter(Element*) const OVERRIDE; 57 virtual Element* virtualItemAfter(Element*) const OVERRIDE;
58 virtual void invalidateCache() const OVERRIDE; 58 virtual void invalidateCache(Document* oldDocument = 0) const OVERRIDE;
59 59
60 mutable Element* m_cachedElement; 60 mutable Element* m_cachedElement;
61 mutable unsigned m_cachedElementOffsetInArray; 61 mutable unsigned m_cachedElementOffsetInArray;
62 }; 62 };
63 63
64 } // namespace 64 } // namespace
65 65
66 #endif 66 #endif
OLDNEW
« no previous file with comments | « Source/core/html/HTMLCollection.cpp ('k') | Source/core/html/HTMLFormControlsCollection.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698