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

Side by Side Diff: Source/platform/text/PlatformLocale.cpp

Issue 235273003: parsing input type=number should skip spaces (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2011,2012 Google Inc. All rights reserved. 2 * Copyright (C) 2011,2012 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 266 matching lines...) Expand 10 before | Expand all | Expand 10 after
277 case '5': 277 case '5':
278 case '6': 278 case '6':
279 case '7': 279 case '7':
280 case '8': 280 case '8':
281 case '9': 281 case '9':
282 builder.append(m_decimalSymbols[input[i] - '0']); 282 builder.append(m_decimalSymbols[input[i] - '0']);
283 break; 283 break;
284 case '.': 284 case '.':
285 builder.append(m_decimalSymbols[DecimalSeparatorIndex]); 285 builder.append(m_decimalSymbols[DecimalSeparatorIndex]);
286 break; 286 break;
287 case ' ':
tkent 2014/04/14 00:47:37 This change is unnecessary. The input string must
288 break;
287 default: 289 default:
288 ASSERT_NOT_REACHED(); 290 ASSERT_NOT_REACHED();
289 } 291 }
290 } 292 }
291 293
292 builder.append(isNegative ? m_negativeSuffix : m_positiveSuffix); 294 builder.append(isNegative ? m_negativeSuffix : m_positiveSuffix);
293 295
294 return builder.toString(); 296 return builder.toString();
295 } 297 }
296 298
(...skipping 114 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 builder.build(formatType == FormatTypeShort ? dateTimeFormatWithoutSecon ds() : dateTimeFormatWithSeconds()); 413 builder.build(formatType == FormatTypeShort ? dateTimeFormatWithoutSecon ds() : dateTimeFormatWithSeconds());
412 break; 414 break;
413 case DateComponents::Invalid: 415 case DateComponents::Invalid:
414 ASSERT_NOT_REACHED(); 416 ASSERT_NOT_REACHED();
415 break; 417 break;
416 } 418 }
417 return builder.toString(); 419 return builder.toString();
418 } 420 }
419 421
420 } 422 }
OLDNEW
« Source/core/html/parser/HTMLParserIdioms.cpp ('K') | « Source/core/html/parser/HTMLParserIdioms.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698