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

Side by Side Diff: Source/core/css/CSSPrimitiveValueMappings.h

Issue 57493002: Remove 'alphabetic' value from text-underline-position (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Patch for landing Created 7 years, 1 month 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
« no previous file with comments | « Source/core/css/CSSParser-in.cpp ('k') | Source/core/rendering/InlineTextBox.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007 Alexey Proskuryakov <ap@nypop.com>. 2 * Copyright (C) 2007 Alexey Proskuryakov <ap@nypop.com>.
3 * Copyright (C) 2008, 2009, 2010, 2011 Apple Inc. All rights reserved. 3 * Copyright (C) 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
4 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/) 4 * Copyright (C) 2009 Torch Mobile Inc. All rights reserved. (http://www.torchmo bile.com/)
5 * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com> 5 * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com>
6 * Copyright (C) Research In Motion Limited 2010. All rights reserved. 6 * Copyright (C) Research In Motion Limited 2010. All rights reserved.
7 * 7 *
8 * Redistribution and use in source and binary forms, with or without 8 * Redistribution and use in source and binary forms, with or without
9 * modification, are permitted provided that the following conditions 9 * modification, are permitted provided that the following conditions
10 * are met: 10 * are met:
(...skipping 2487 matching lines...) Expand 10 before | Expand all | Expand 10 after
2498 } 2498 }
2499 2499
2500 template<> inline CSSPrimitiveValue::CSSPrimitiveValue(TextUnderlinePosition e) 2500 template<> inline CSSPrimitiveValue::CSSPrimitiveValue(TextUnderlinePosition e)
2501 : CSSValue(PrimitiveClass) 2501 : CSSValue(PrimitiveClass)
2502 { 2502 {
2503 m_primitiveUnitType = CSS_VALUE_ID; 2503 m_primitiveUnitType = CSS_VALUE_ID;
2504 switch (e) { 2504 switch (e) {
2505 case TextUnderlinePositionAuto: 2505 case TextUnderlinePositionAuto:
2506 m_value.valueID = CSSValueAuto; 2506 m_value.valueID = CSSValueAuto;
2507 break; 2507 break;
2508 case TextUnderlinePositionAlphabetic:
2509 m_value.valueID = CSSValueAlphabetic;
2510 break;
2511 case TextUnderlinePositionUnder: 2508 case TextUnderlinePositionUnder:
2512 m_value.valueID = CSSValueUnder; 2509 m_value.valueID = CSSValueUnder;
2513 break; 2510 break;
2514 } 2511 }
2515 2512
2516 // FIXME: Implement support for 'under left' and 'under right' values. 2513 // FIXME: Implement support for 'under left' and 'under right' values.
2517 } 2514 }
2518 2515
2519 template<> inline CSSPrimitiveValue::operator TextUnderlinePosition() const 2516 template<> inline CSSPrimitiveValue::operator TextUnderlinePosition() const
2520 { 2517 {
2521 ASSERT(isValueID()); 2518 ASSERT(isValueID());
2522 switch (m_value.valueID) { 2519 switch (m_value.valueID) {
2523 case CSSValueAuto: 2520 case CSSValueAuto:
2524 return TextUnderlinePositionAuto; 2521 return TextUnderlinePositionAuto;
2525 case CSSValueAlphabetic:
2526 return TextUnderlinePositionAlphabetic;
2527 case CSSValueUnder: 2522 case CSSValueUnder:
2528 return TextUnderlinePositionUnder; 2523 return TextUnderlinePositionUnder;
2529 default: 2524 default:
2530 break; 2525 break;
2531 } 2526 }
2532 2527
2533 // FIXME: Implement support for 'under left' and 'under right' values. 2528 // FIXME: Implement support for 'under left' and 'under right' values.
2534 2529
2535 ASSERT_NOT_REACHED(); 2530 ASSERT_NOT_REACHED();
2536 return TextUnderlinePositionAuto; 2531 return TextUnderlinePositionAuto;
(...skipping 2421 matching lines...) Expand 10 before | Expand all | Expand 10 after
4958 break; 4953 break;
4959 } 4954 }
4960 4955
4961 ASSERT_NOT_REACHED(); 4956 ASSERT_NOT_REACHED();
4962 return TouchActionDelayNone; 4957 return TouchActionDelayNone;
4963 } 4958 }
4964 4959
4965 } 4960 }
4966 4961
4967 #endif 4962 #endif
OLDNEW
« no previous file with comments | « Source/core/css/CSSParser-in.cpp ('k') | Source/core/rendering/InlineTextBox.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698