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

Side by Side Diff: Source/core/css/CSSComputedStyleDeclaration.cpp

Issue 22336008: Use the runtime flag and remove '-webkit-' prefix for CSS3 text-indent. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: patch Created 7 years, 4 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) 2004 Zack Rusin <zack@kde.org> 2 * Copyright (C) 2004 Zack Rusin <zack@kde.org>
3 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved. 3 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 Apple Inc. All rights reserved.
4 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org> 4 * Copyright (C) 2007 Alexey Proskuryakov <ap@webkit.org>
5 * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com> 5 * Copyright (C) 2007 Nicholas Shanks <webkit@nickshanks.com>
6 * Copyright (C) 2011 Sencha, Inc. All rights reserved. 6 * Copyright (C) 2011 Sencha, Inc. All rights reserved.
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 Lesser General Public 9 * modify it under the terms of the GNU Lesser General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 2235 matching lines...) Expand 10 before | Expand all | Expand 10 after
2246 case TextEmphasisMarkTriangle: 2246 case TextEmphasisMarkTriangle:
2247 case TextEmphasisMarkSesame: { 2247 case TextEmphasisMarkSesame: {
2248 RefPtr<CSSValueList> list = CSSValueList::createSpaceSeparated() ; 2248 RefPtr<CSSValueList> list = CSSValueList::createSpaceSeparated() ;
2249 list->append(cssValuePool().createValue(style->textEmphasisFill( ))); 2249 list->append(cssValuePool().createValue(style->textEmphasisFill( )));
2250 list->append(cssValuePool().createValue(style->textEmphasisMark( ))); 2250 list->append(cssValuePool().createValue(style->textEmphasisMark( )));
2251 return list.release(); 2251 return list.release();
2252 } 2252 }
2253 } 2253 }
2254 case CSSPropertyTextIndent: { 2254 case CSSPropertyTextIndent: {
2255 RefPtr<CSSValue> textIndent = zoomAdjustedPixelValueForLength(style- >textIndent(), style.get()); 2255 RefPtr<CSSValue> textIndent = zoomAdjustedPixelValueForLength(style- >textIndent(), style.get());
2256 #if ENABLE(CSS3_TEXT) 2256 if (RuntimeEnabledFeatures::css3TextEnabled() && style->textIndentLi ne() == TextIndentEachLine) {
Julien - ping for review 2013/08/21 23:54:50 Let's keep in mind that we would like to test this
2257 if (style->textIndentLine() == TextIndentEachLine) {
2258 RefPtr<CSSValueList> list = CSSValueList::createSpaceSeparated() ; 2257 RefPtr<CSSValueList> list = CSSValueList::createSpaceSeparated() ;
2259 list->append(textIndent.release()); 2258 list->append(textIndent.release());
2260 list->append(cssValuePool().createIdentifierValue(CSSValueWebkit EachLine)); 2259 list->append(cssValuePool().createIdentifierValue(CSSValueEachLi ne));
2261 return list.release(); 2260 return list.release();
2262 } 2261 }
2263 #endif
2264 return textIndent.release(); 2262 return textIndent.release();
2265 } 2263 }
2266 case CSSPropertyTextShadow: 2264 case CSSPropertyTextShadow:
2267 return valueForShadow(renderer, style->textShadow(), propertyID, sty le.get()); 2265 return valueForShadow(renderer, style->textShadow(), propertyID, sty le.get());
2268 case CSSPropertyTextRendering: 2266 case CSSPropertyTextRendering:
2269 return cssValuePool().createValue(style->fontDescription().textRende ringMode()); 2267 return cssValuePool().createValue(style->fontDescription().textRende ringMode());
2270 case CSSPropertyTextOverflow: 2268 case CSSPropertyTextOverflow:
2271 if (style->textOverflow()) 2269 if (style->textOverflow())
2272 return cssValuePool().createIdentifierValue(CSSValueEllipsis); 2270 return cssValuePool().createIdentifierValue(CSSValueEllipsis);
2273 return cssValuePool().createIdentifierValue(CSSValueClip); 2271 return cssValuePool().createIdentifierValue(CSSValueClip);
(...skipping 888 matching lines...) Expand 10 before | Expand all | Expand 10 after
3162 static const CSSPropertyID propertiesAfterSlashSeperator[3] = { CSSPropertyB ackgroundSize, CSSPropertyBackgroundOrigin, 3160 static const CSSPropertyID propertiesAfterSlashSeperator[3] = { CSSPropertyB ackgroundSize, CSSPropertyBackgroundOrigin,
3163 CSSPropertyB ackgroundClip }; 3161 CSSPropertyB ackgroundClip };
3164 3162
3165 RefPtr<CSSValueList> list = CSSValueList::createSlashSeparated(); 3163 RefPtr<CSSValueList> list = CSSValueList::createSlashSeparated();
3166 list->append(valuesForShorthandProperty(StylePropertyShorthand(CSSPropertyBa ckground, propertiesBeforeSlashSeperator, WTF_ARRAY_LENGTH(propertiesBeforeSlash Seperator)))); 3164 list->append(valuesForShorthandProperty(StylePropertyShorthand(CSSPropertyBa ckground, propertiesBeforeSlashSeperator, WTF_ARRAY_LENGTH(propertiesBeforeSlash Seperator))));
3167 list->append(valuesForShorthandProperty(StylePropertyShorthand(CSSPropertyBa ckground, propertiesAfterSlashSeperator, WTF_ARRAY_LENGTH(propertiesAfterSlashSe perator)))); 3165 list->append(valuesForShorthandProperty(StylePropertyShorthand(CSSPropertyBa ckground, propertiesAfterSlashSeperator, WTF_ARRAY_LENGTH(propertiesAfterSlashSe perator))));
3168 return list.release(); 3166 return list.release();
3169 } 3167 }
3170 3168
3171 } // namespace WebCore 3169 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698