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

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

Issue 239353007: Set proper referrer for css resource fetching (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: add test Created 6 years, 8 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
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 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
45 return true; 45 return true;
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(url, document().completeURL(url)))); 56 RefPtrWillBeRawPtr<CSSImageValue> imageValue = CSSImageValue::create (url, document().completeURL(url));
57 imageValue->setReferrer(document().url().strippedForUseAsReferrer()) ;
abarth-chromium 2014/04/18 16:36:49 document().outgoingReferrer()
bashi 2014/04/22 01:22:55 Done.
58 style->setProperty(CSSProperty(CSSPropertyBackgroundImage, imageValu e.release()));
59 }
57 } else if (name == valignAttr) { 60 } else if (name == valignAttr) {
58 if (equalIgnoringCase(value, "top")) 61 if (equalIgnoringCase(value, "top"))
59 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueTop); 62 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueTop);
60 else if (equalIgnoringCase(value, "middle")) 63 else if (equalIgnoringCase(value, "middle"))
61 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueMiddle); 64 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueMiddle);
62 else if (equalIgnoringCase(value, "bottom")) 65 else if (equalIgnoringCase(value, "bottom"))
63 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBottom); 66 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBottom);
64 else if (equalIgnoringCase(value, "baseline")) 67 else if (equalIgnoringCase(value, "baseline"))
65 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBaseline); 68 addPropertyToPresentationAttributeStyle(style, CSSPropertyVerticalAl ign, CSSValueBaseline);
66 else 69 else
(...skipping 18 matching lines...) Expand all
85 88
86 HTMLTableElement* HTMLTablePartElement::findParentTable() const 89 HTMLTableElement* HTMLTablePartElement::findParentTable() const
87 { 90 {
88 ContainerNode* parent = NodeRenderingTraversal::parent(this); 91 ContainerNode* parent = NodeRenderingTraversal::parent(this);
89 while (parent && !isHTMLTableElement(*parent)) 92 while (parent && !isHTMLTableElement(*parent))
90 parent = NodeRenderingTraversal::parent(parent); 93 parent = NodeRenderingTraversal::parent(parent);
91 return toHTMLTableElement(parent); 94 return toHTMLTableElement(parent);
92 } 95 }
93 96
94 } 97 }
OLDNEW
« Source/core/html/HTMLTableElement.cpp ('K') | « Source/core/html/HTMLTableElement.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698