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

Side by Side Diff: Source/core/animation/css/CSSAnimatableValueFactory.cpp

Issue 234453002: Remove SVG1.1 kerning property (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: More TestExpectations updates 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 330 matching lines...) Expand 10 before | Expand all | Expand 10 after
341 case CSSPropertyFontSize: 341 case CSSPropertyFontSize:
342 // Must pass a specified size to setFontSize if Text Autosizing is enabl ed, but a computed size 342 // Must pass a specified size to setFontSize if Text Autosizing is enabl ed, but a computed size
343 // if text zoom is enabled (if neither is enabled it's irrelevant as the y're probably the same). 343 // if text zoom is enabled (if neither is enabled it's irrelevant as the y're probably the same).
344 // FIXME: Should we introduce an option to pass the computed font size h ere, allowing consumers to 344 // FIXME: Should we introduce an option to pass the computed font size h ere, allowing consumers to
345 // enable text zoom rather than Text Autosizing? See http://crbug.com/22 7545. 345 // enable text zoom rather than Text Autosizing? See http://crbug.com/22 7545.
346 return createFromDouble(style.specifiedFontSize()); 346 return createFromDouble(style.specifiedFontSize());
347 case CSSPropertyFontWeight: 347 case CSSPropertyFontWeight:
348 return createFromFontWeight(style.fontWeight()); 348 return createFromFontWeight(style.fontWeight());
349 case CSSPropertyHeight: 349 case CSSPropertyHeight:
350 return createFromLength(style.height(), style); 350 return createFromLength(style.height(), style);
351 case CSSPropertyKerning:
352 return AnimatableSVGLength::create(style.kerning());
353 case CSSPropertyLightingColor: 351 case CSSPropertyLightingColor:
354 return createFromColor(property, style); 352 return createFromColor(property, style);
355 case CSSPropertyListStyleImage: 353 case CSSPropertyListStyleImage:
356 return createFromStyleImage(style.listStyleImage()); 354 return createFromStyleImage(style.listStyleImage());
357 case CSSPropertyLeft: 355 case CSSPropertyLeft:
358 return createFromLength(style.left(), style); 356 return createFromLength(style.left(), style);
359 case CSSPropertyLetterSpacing: 357 case CSSPropertyLetterSpacing:
360 return createFromDouble(style.letterSpacing()); 358 return createFromDouble(style.letterSpacing());
361 case CSSPropertyLineHeight: 359 case CSSPropertyLineHeight:
362 return createFromLineHeight(style.specifiedLineHeight(), style); 360 return createFromLineHeight(style.specifiedLineHeight(), style);
(...skipping 146 matching lines...) Expand 10 before | Expand all | Expand 10 after
509 case CSSPropertyZoom: 507 case CSSPropertyZoom:
510 return createFromDouble(style.zoom()); 508 return createFromDouble(style.zoom());
511 default: 509 default:
512 ASSERT_NOT_REACHED(); 510 ASSERT_NOT_REACHED();
513 // This return value is to avoid a release crash if possible. 511 // This return value is to avoid a release crash if possible.
514 return AnimatableUnknown::create(nullptr); 512 return AnimatableUnknown::create(nullptr);
515 } 513 }
516 } 514 }
517 515
518 } // namespace WebCore 516 } // namespace WebCore
OLDNEW
« no previous file with comments | « LayoutTests/webexposed/css-properties-as-js-properties-expected.txt ('k') | Source/core/animation/css/CSSAnimations.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698