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

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

Issue 23819007: Have Node::document() return a reference instead of a pointer (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebase on master Created 7 years, 3 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/HTMLTableElement.cpp ('k') | Source/core/html/HTMLTableRowElement.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) 1997 Martin Jones (mjones@kde.org) 2 * Copyright (C) 1997 Martin Jones (mjones@kde.org)
3 * (C) 1997 Torben Weis (weis@kde.org) 3 * (C) 1997 Torben Weis (weis@kde.org)
4 * (C) 1998 Waldo Bastian (bastian@kde.org) 4 * (C) 1998 Waldo Bastian (bastian@kde.org)
5 * (C) 1999 Lars Knoll (knoll@kde.org) 5 * (C) 1999 Lars Knoll (knoll@kde.org)
6 * (C) 1999 Antti Koivisto (koivisto@kde.org) 6 * (C) 1999 Antti Koivisto (koivisto@kde.org)
7 * Copyright (C) 2003, 2004, 2005, 2006 Apple Computer, Inc. 7 * Copyright (C) 2003, 2004, 2005, 2006 Apple Computer, Inc.
8 * 8 *
9 * This library is free software; you can redistribute it and/or 9 * This library is free software; you can redistribute it and/or
10 * modify it under the terms of the GNU Library General Public 10 * modify it under the terms of the GNU Library General Public
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 return HTMLElement::isPresentationAttribute(name); 46 return HTMLElement::isPresentationAttribute(name);
47 } 47 }
48 48
49 void HTMLTablePartElement::collectStyleForPresentationAttribute(const QualifiedN ame& name, const AtomicString& value, MutableStylePropertySet* style) 49 void HTMLTablePartElement::collectStyleForPresentationAttribute(const QualifiedN ame& name, const AtomicString& value, MutableStylePropertySet* style)
50 { 50 {
51 if (name == bgcolorAttr) 51 if (name == bgcolorAttr)
52 addHTMLColorToStyle(style, CSSPropertyBackgroundColor, value); 52 addHTMLColorToStyle(style, CSSPropertyBackgroundColor, value);
53 else if (name == backgroundAttr) { 53 else if (name == backgroundAttr) {
54 String url = stripLeadingAndTrailingHTMLSpaces(value); 54 String url = stripLeadingAndTrailingHTMLSpaces(value);
55 if (!url.isEmpty()) 55 if (!url.isEmpty())
56 style->setProperty(CSSProperty(CSSPropertyBackgroundImage, CSSImageV alue::create(document()->completeURL(url).string()))); 56 style->setProperty(CSSProperty(CSSPropertyBackgroundImage, CSSImageV alue::create(document().completeURL(url).string())));
57 } else if (name == valignAttr) { 57 } else if (name == valignAttr) {
58 if (equalIgnoringCase(value, "top")) 58 if (equalIgnoringCase(value, "top"))
59 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueTop); 59 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueTop);
60 else if (equalIgnoringCase(value, "middle")) 60 else if (equalIgnoringCase(value, "middle"))
61 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueMiddle); 61 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueMiddle);
62 else if (equalIgnoringCase(value, "bottom")) 62 else if (equalIgnoringCase(value, "bottom"))
63 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBottom); 63 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBottom);
64 else if (equalIgnoringCase(value, "baseline")) 64 else if (equalIgnoringCase(value, "baseline"))
65 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBaseline); 65 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBaseline);
66 else 66 else
(...skipping 18 matching lines...) Expand all
85 85
86 HTMLTableElement* HTMLTablePartElement::findParentTable() const 86 HTMLTableElement* HTMLTablePartElement::findParentTable() const
87 { 87 {
88 ContainerNode* parent = NodeRenderingTraversal::parent(this); 88 ContainerNode* parent = NodeRenderingTraversal::parent(this);
89 while (parent && !isHTMLTableElement(parent)) 89 while (parent && !isHTMLTableElement(parent))
90 parent = NodeRenderingTraversal::parent(parent); 90 parent = NodeRenderingTraversal::parent(parent);
91 return toHTMLTableElement(parent); 91 return toHTMLTableElement(parent);
92 } 92 }
93 93
94 } 94 }
OLDNEW
« no previous file with comments | « Source/core/html/HTMLTableElement.cpp ('k') | Source/core/html/HTMLTableRowElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698