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

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

Issue 2384263003: Reflow comments in core/animation and subdirs (Closed)
Patch Set: Created 4 years, 2 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) 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 412 matching lines...) Expand 10 before | Expand all | Expand 10 after
423 return createFromDouble(style.flexGrow()); 423 return createFromDouble(style.flexGrow());
424 case CSSPropertyFlexShrink: 424 case CSSPropertyFlexShrink:
425 return createFromDouble(style.flexShrink()); 425 return createFromDouble(style.flexShrink());
426 case CSSPropertyFlexBasis: 426 case CSSPropertyFlexBasis:
427 return createFromLength(style.flexBasis(), style); 427 return createFromLength(style.flexBasis(), style);
428 case CSSPropertyFloodColor: 428 case CSSPropertyFloodColor:
429 return createFromColor(property, style); 429 return createFromColor(property, style);
430 case CSSPropertyFloodOpacity: 430 case CSSPropertyFloodOpacity:
431 return createFromDouble(style.floodOpacity()); 431 return createFromDouble(style.floodOpacity());
432 case CSSPropertyFontSize: 432 case CSSPropertyFontSize:
433 // Must pass a specified size to setFontSize if Text Autosizing is enabled , but a computed size 433 // Must pass a specified size to setFontSize if Text Autosizing is
434 // if text zoom is enabled (if neither is enabled it's irrelevant as they' re probably the same). 434 // enabled, but a computed size if text zoom is enabled (if neither is
435 // FIXME: Should we introduce an option to pass the computed font size her e, allowing consumers to 435 // enabled it's irrelevant as they're probably the same).
436 // enable text zoom rather than Text Autosizing? See http://crbug.com/2275 45. 436 // FIXME: Should we introduce an option to pass the computed font size
437 // here, allowing consumers to enable text zoom rather than Text
438 // Autosizing? See http://crbug.com/227545.
437 return createFromDouble(style.specifiedFontSize()); 439 return createFromDouble(style.specifiedFontSize());
438 case CSSPropertyFontSizeAdjust: 440 case CSSPropertyFontSizeAdjust:
439 return style.hasFontSizeAdjust() 441 return style.hasFontSizeAdjust()
440 ? createFromDouble(style.fontSizeAdjust()) 442 ? createFromDouble(style.fontSizeAdjust())
441 : AnimatableUnknown::create(CSSValueNone); 443 : AnimatableUnknown::create(CSSValueNone);
442 case CSSPropertyFontStretch: 444 case CSSPropertyFontStretch:
443 return createFromFontStretch(style.fontStretch()); 445 return createFromFontStretch(style.fontStretch());
444 case CSSPropertyFontWeight: 446 case CSSPropertyFontWeight:
445 return createFromFontWeight(style.fontWeight()); 447 return createFromFontWeight(style.fontWeight());
446 case CSSPropertyHeight: 448 case CSSPropertyHeight:
(...skipping 220 matching lines...) Expand 10 before | Expand all | Expand 10 after
667 if (style.hasAutoZIndex()) 669 if (style.hasAutoZIndex())
668 return AnimatableUnknown::create(CSSValueAuto); 670 return AnimatableUnknown::create(CSSValueAuto);
669 return createFromDouble(style.zIndex()); 671 return createFromDouble(style.zIndex());
670 default: 672 default:
671 NOTREACHED(); 673 NOTREACHED();
672 return nullptr; 674 return nullptr;
673 } 675 }
674 } 676 }
675 677
676 } // namespace blink 678 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698