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

Side by Side Diff: third_party/WebKit/Source/core/html/parser/HTMLTokenizer.cpp

Issue 2971683003: Change code to make jumbo exceptions unnecessary in core/html (Closed)
Patch Set: Removed testing code after showing that it worked Created 3 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
« no previous file with comments | « third_party/WebKit/Source/core/html/HTMLIFrameElementSandbox.cpp ('k') | no next file » | 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) 2008 Apple Inc. All Rights Reserved. 2 * Copyright (C) 2008 Apple Inc. All Rights Reserved.
3 * Copyright (C) 2009 Torch Mobile, Inc. http://www.torchmobile.com/ 3 * Copyright (C) 2009 Torch Mobile, Inc. http://www.torchmobile.com/
4 * Copyright (C) 2010 Google, Inc. All Rights Reserved. 4 * Copyright (C) 2010 Google, Inc. All Rights Reserved.
5 * 5 *
6 * Redistribution and use in source and binary forms, with or without 6 * Redistribution and use in source and binary forms, with or without
7 * modification, are permitted provided that the following conditions 7 * modification, are permitted provided that the following conditions
8 * are met: 8 * are met:
9 * 1. Redistributions of source code must retain the above copyright 9 * 1. Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 19 matching lines...) Expand all
30 #include "core/HTMLNames.h" 30 #include "core/HTMLNames.h"
31 #include "core/HTMLTokenizerNames.h" 31 #include "core/HTMLTokenizerNames.h"
32 #include "core/html/parser/HTMLEntityParser.h" 32 #include "core/html/parser/HTMLEntityParser.h"
33 #include "core/html/parser/HTMLParserIdioms.h" 33 #include "core/html/parser/HTMLParserIdioms.h"
34 #include "core/html/parser/HTMLTreeBuilder.h" 34 #include "core/html/parser/HTMLTreeBuilder.h"
35 #include "core/html/parser/MarkupTokenizerInlines.h" 35 #include "core/html/parser/MarkupTokenizerInlines.h"
36 #include "platform/wtf/ASCIICType.h" 36 #include "platform/wtf/ASCIICType.h"
37 #include "platform/wtf/text/Unicode.h" 37 #include "platform/wtf/text/Unicode.h"
38 38
39 // Please don't use DEFINE_STATIC_LOCAL in this file. The HTMLTokenizer is used 39 // Please don't use DEFINE_STATIC_LOCAL in this file. The HTMLTokenizer is used
40 // from multiple threads and DEFINE_STATIC_LOCAL isn't threadsafe. 40 // from multiple threads and DEFINE_STATIC_LOCAL isn't threadsafe.
kouhei (in TOK) 2017/07/11 02:10:49 This comment is no longer true. We can just remove
41 #pragma push_macro("DEFINE_STATIC_LOCAL")
41 #undef DEFINE_STATIC_LOCAL 42 #undef DEFINE_STATIC_LOCAL
42 43
43 namespace blink { 44 namespace blink {
44 45
45 using namespace HTMLNames; 46 using namespace HTMLNames;
46 47
47 static inline UChar ToLowerCase(UChar cc) { 48 static inline UChar ToLowerCase(UChar cc) {
48 DCHECK(IsASCIIAlpha(cc)); 49 DCHECK(IsASCIIAlpha(cc));
49 return cc | 0x20; 50 return cc | 0x20;
50 } 51 }
(...skipping 1446 matching lines...) Expand 10 before | Expand all | Expand 10 after
1497 return true; 1498 return true;
1498 } 1499 }
1499 1500
1500 inline void HTMLTokenizer::ParseError() { 1501 inline void HTMLTokenizer::ParseError() {
1501 #if DCHECK_IS_ON() 1502 #if DCHECK_IS_ON()
1502 DVLOG(1) << "Not implemented."; 1503 DVLOG(1) << "Not implemented.";
1503 #endif 1504 #endif
1504 } 1505 }
1505 1506
1506 } // namespace blink 1507 } // namespace blink
1508
1509 #pragma pop_macro("DEFINE_STATIC_LOCAL")
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/html/HTMLIFrameElementSandbox.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698