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

Side by Side Diff: Source/core/html/HTMLOptGroupElement.cpp

Issue 365673002: Pass a struct to ContainerNode::childrenChanged() instead of separate arguments (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Remove dead code Created 6 years, 5 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 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010 Apple Inc. All rights reserv ed. 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2010 Apple Inc. All rights reserv ed.
6 * (C) 2006 Alexey Proskuryakov (ap@nypop.com) 6 * (C) 2006 Alexey Proskuryakov (ap@nypop.com)
7 * 7 *
8 * This library is free software; you can redistribute it and/or 8 * This library is free software; you can redistribute it and/or
9 * modify it under the terms of the GNU Library General Public 9 * modify it under the terms of the GNU Library General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 { 47 {
48 return fastHasAttribute(disabledAttr); 48 return fastHasAttribute(disabledAttr);
49 } 49 }
50 50
51 bool HTMLOptGroupElement::rendererIsFocusable() const 51 bool HTMLOptGroupElement::rendererIsFocusable() const
52 { 52 {
53 // Optgroup elements do not have a renderer so we check the renderStyle inst ead. 53 // Optgroup elements do not have a renderer so we check the renderStyle inst ead.
54 return renderStyle() && renderStyle()->display() != NONE; 54 return renderStyle() && renderStyle()->display() != NONE;
55 } 55 }
56 56
57 void HTMLOptGroupElement::childrenChanged(bool changedByParser, Node* beforeChan ge, Node* afterChange, int childCountDelta) 57 void HTMLOptGroupElement::childrenChanged(const ChildrenChange& change)
58 { 58 {
59 recalcSelectOptions(); 59 recalcSelectOptions();
60 HTMLElement::childrenChanged(changedByParser, beforeChange, afterChange, chi ldCountDelta); 60 HTMLElement::childrenChanged(change);
61 } 61 }
62 62
63 void HTMLOptGroupElement::parseAttribute(const QualifiedName& name, const Atomic String& value) 63 void HTMLOptGroupElement::parseAttribute(const QualifiedName& name, const Atomic String& value)
64 { 64 {
65 HTMLElement::parseAttribute(name, value); 65 HTMLElement::parseAttribute(name, value);
66 recalcSelectOptions(); 66 recalcSelectOptions();
67 67
68 if (name == disabledAttr) 68 if (name == disabledAttr)
69 didAffectSelector(AffectedSelectorDisabled | AffectedSelectorEnabled); 69 didAffectSelector(AffectedSelectorDisabled | AffectedSelectorEnabled);
70 } 70 }
(...skipping 65 matching lines...) Expand 10 before | Expand all | Expand 10 after
136 select->accessKeyAction(false); 136 select->accessKeyAction(false);
137 } 137 }
138 138
139 bool HTMLOptGroupElement::isDisplayNone() const 139 bool HTMLOptGroupElement::isDisplayNone() const
140 { 140 {
141 RenderStyle* style = nonRendererStyle(); 141 RenderStyle* style = nonRendererStyle();
142 return style && style->display() == NONE; 142 return style && style->display() == NONE;
143 } 143 }
144 144
145 } // namespace 145 } // namespace
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698