| Index: third_party/WebKit/Source/core/layout/ng/ng_length_utils.cc
|
| diff --git a/third_party/WebKit/Source/core/layout/ng/ng_length_utils.cc b/third_party/WebKit/Source/core/layout/ng/ng_length_utils.cc
|
| index 8e2fa7f696353fe741dce04d11995ed2dd6852eb..d6fdcb327c60ef5c58fbe0462a04531afd3ad099 100644
|
| --- a/third_party/WebKit/Source/core/layout/ng/ng_length_utils.cc
|
| +++ b/third_party/WebKit/Source/core/layout/ng/ng_length_utils.cc
|
| @@ -55,12 +55,8 @@ LayoutUnit resolveInlineLength(const NGConstraintSpace& constraintSpace,
|
| return LayoutUnit();
|
| case DeviceWidth:
|
| case DeviceHeight:
|
| - // TODO(layout-ng): Do we need to handle this here or does the style
|
| - // system already compute these?
|
| - return LayoutUnit();
|
| case ExtendToZoom:
|
| - NOTREACHED()
|
| - << "ExtendToZoom should only be used for viewport definitions";
|
| + NOTREACHED() << "These should only be used for viewport definitions";
|
| case MaxSizeNone:
|
| default:
|
| NOTREACHED();
|
| @@ -111,12 +107,8 @@ LayoutUnit resolveBlockLength(const NGConstraintSpace& constraintSpace,
|
| return contentSize;
|
| case DeviceWidth:
|
| case DeviceHeight:
|
| - // TODO(layout-ng): Do we need to handle this here or does the style
|
| - // system already compute these?
|
| - return LayoutUnit();
|
| case ExtendToZoom:
|
| - NOTREACHED()
|
| - << "ExtendToZoom should only be used for viewport definitions";
|
| + NOTREACHED() << "These should only be used for viewport definitions";
|
| case MaxSizeNone:
|
| default:
|
| NOTREACHED();
|
|
|