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

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

Issue 195813003: Use new is*Element() helper functions further more in HTML code (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix bad assertion Created 6 years, 9 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) 2011 Nokia Corporation and/or its subsidiary(-ies) 2 * Copyright (C) 2011 Nokia Corporation and/or its subsidiary(-ies)
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 54
55 void HTMLSummaryElement::didAddUserAgentShadowRoot(ShadowRoot& root) 55 void HTMLSummaryElement::didAddUserAgentShadowRoot(ShadowRoot& root)
56 { 56 {
57 root.appendChild(DetailsMarkerControl::create(document())); 57 root.appendChild(DetailsMarkerControl::create(document()));
58 root.appendChild(HTMLContentElement::create(document())); 58 root.appendChild(HTMLContentElement::create(document()));
59 } 59 }
60 60
61 HTMLDetailsElement* HTMLSummaryElement::detailsElement() const 61 HTMLDetailsElement* HTMLSummaryElement::detailsElement() const
62 { 62 {
63 Node* parent = NodeRenderingTraversal::parent(this); 63 Node* parent = NodeRenderingTraversal::parent(this);
64 if (parent && parent->hasTagName(detailsTag)) 64 if (isHTMLDetailsElement(parent))
65 return toHTMLDetailsElement(parent); 65 return toHTMLDetailsElement(parent);
66 return 0; 66 return 0;
67 } 67 }
68 68
69 bool HTMLSummaryElement::isMainSummary() const 69 bool HTMLSummaryElement::isMainSummary() const
70 { 70 {
71 if (HTMLDetailsElement* details = detailsElement()) 71 if (HTMLDetailsElement* details = detailsElement())
72 return details->findMainSummary() == this; 72 return details->findMainSummary() == this;
73 73
74 return false; 74 return false;
(...skipping 57 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 132
133 bool HTMLSummaryElement::willRespondToMouseClickEvents() 133 bool HTMLSummaryElement::willRespondToMouseClickEvents()
134 { 134 {
135 if (isMainSummary() && renderer()) 135 if (isMainSummary() && renderer())
136 return true; 136 return true;
137 137
138 return HTMLElement::willRespondToMouseClickEvents(); 138 return HTMLElement::willRespondToMouseClickEvents();
139 } 139 }
140 140
141 } 141 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698