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

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

Issue 2370053003: Clean up stale FIXMEs and TODOs in core/animation (Closed)
Patch Set: Enum change unchange 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 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "core/animation/LengthPropertyFunctions.h" 5 #include "core/animation/LengthPropertyFunctions.h"
6 6
7 #include "core/style/ComputedStyle.h" 7 #include "core/style/ComputedStyle.h"
8 8
9 namespace blink { 9 namespace blink {
10 10
11 // TODO(alancutter): Generate these functions.
12
alancutter (OOO until 2018) 2016/09/27 01:14:13 Generating these will be a consequence of work on
13 ValueRange LengthPropertyFunctions::getValueRange(CSSPropertyID property) 11 ValueRange LengthPropertyFunctions::getValueRange(CSSPropertyID property)
14 { 12 {
15 switch (property) { 13 switch (property) {
16 case CSSPropertyBorderBottomWidth: 14 case CSSPropertyBorderBottomWidth:
17 case CSSPropertyBorderLeftWidth: 15 case CSSPropertyBorderLeftWidth:
18 case CSSPropertyBorderRightWidth: 16 case CSSPropertyBorderRightWidth:
19 case CSSPropertyBorderTopWidth: 17 case CSSPropertyBorderTopWidth:
20 case CSSPropertyFlexBasis: 18 case CSSPropertyFlexBasis:
21 case CSSPropertyHeight: 19 case CSSPropertyHeight:
22 case CSSPropertyLineHeight: 20 case CSSPropertyLineHeight:
(...skipping 415 matching lines...) Expand 10 before | Expand all | Expand 10 after
438 case CSSPropertyWebkitTransformOriginZ: 436 case CSSPropertyWebkitTransformOriginZ:
439 case CSSPropertyWordSpacing: 437 case CSSPropertyWordSpacing:
440 return false; 438 return false;
441 439
442 default: 440 default:
443 return false; 441 return false;
444 } 442 }
445 } 443 }
446 444
447 } // namespace blink 445 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698