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

Side by Side Diff: Source/bindings/core/v8/custom/V8CSSStyleDeclarationCustom.cpp

Issue 397733004: Allow assertions to be enabled in Blink Release builds. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Rebased. Created 6 years, 5 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007-2011 Google Inc. All rights reserved. 2 * Copyright (C) 2007-2011 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 41 matching lines...) Expand 10 before | Expand all | Expand 10 after
52 52
53 namespace WebCore { 53 namespace WebCore {
54 54
55 // Check for a CSS prefix. 55 // Check for a CSS prefix.
56 // Passed prefix is all lowercase. 56 // Passed prefix is all lowercase.
57 // First character of the prefix within the property name may be upper or lowerc ase. 57 // First character of the prefix within the property name may be upper or lowerc ase.
58 // Other characters in the prefix within the property name must be lowercase. 58 // Other characters in the prefix within the property name must be lowercase.
59 // The prefix within the property name must be followed by a capital letter. 59 // The prefix within the property name must be followed by a capital letter.
60 static bool hasCSSPropertyNamePrefix(const String& propertyName, const char* pre fix) 60 static bool hasCSSPropertyNamePrefix(const String& propertyName, const char* pre fix)
61 { 61 {
62 #ifndef NDEBUG 62 #if ENABLE(ASSERT)
63 ASSERT(*prefix); 63 ASSERT(*prefix);
64 for (const char* p = prefix; *p; ++p) 64 for (const char* p = prefix; *p; ++p)
65 ASSERT(isASCIILower(*p)); 65 ASSERT(isASCIILower(*p));
66 ASSERT(propertyName.length()); 66 ASSERT(propertyName.length());
67 #endif 67 #endif
68 68
69 if (toASCIILower(propertyName[0]) != prefix[0]) 69 if (toASCIILower(propertyName[0]) != prefix[0])
70 return false; 70 return false;
71 71
72 unsigned length = propertyName.length(); 72 unsigned length = propertyName.length();
(...skipping 150 matching lines...) Expand 10 before | Expand all | Expand 10 after
223 ExceptionState exceptionState(ExceptionState::SetterContext, getPropertyName (static_cast<CSSPropertyID>(propInfo->propID)), "CSSStyleDeclaration", info.Hold er(), info.GetIsolate()); 223 ExceptionState exceptionState(ExceptionState::SetterContext, getPropertyName (static_cast<CSSPropertyID>(propInfo->propID)), "CSSStyleDeclaration", info.Hold er(), info.GetIsolate());
224 impl->setPropertyInternal(static_cast<CSSPropertyID>(propInfo->propID), prop ertyValue, false, exceptionState); 224 impl->setPropertyInternal(static_cast<CSSPropertyID>(propInfo->propID), prop ertyValue, false, exceptionState);
225 225
226 if (exceptionState.throwIfNeeded()) 226 if (exceptionState.throwIfNeeded())
227 return; 227 return;
228 228
229 v8SetReturnValue(info, value); 229 v8SetReturnValue(info, value);
230 } 230 }
231 231
232 } // namespace WebCore 232 } // namespace WebCore
OLDNEW
« no previous file with comments | « Source/bindings/core/v8/custom/V8BlobCustomHelpers.cpp ('k') | Source/bindings/core/v8/npruntime.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698