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

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

Issue 2394683005: Remove ASSERT_UNUSED (Closed)
Patch Set: Created 4 years, 2 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) 2010 Google, Inc. All Rights Reserved. 2 * Copyright (C) 2010 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 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 364 matching lines...) Expand 10 before | Expand all | Expand 10 after
375 void HTMLScriptRunner::executeParsingBlockingScripts() { 375 void HTMLScriptRunner::executeParsingBlockingScripts() {
376 while (hasParserBlockingScript() && 376 while (hasParserBlockingScript() &&
377 isPendingScriptReady(m_parserBlockingScript.get())) 377 isPendingScriptReady(m_parserBlockingScript.get()))
378 executeParsingBlockingScript(); 378 executeParsingBlockingScript();
379 } 379 }
380 380
381 void HTMLScriptRunner::executeScriptsWaitingForLoad(Resource* resource) { 381 void HTMLScriptRunner::executeScriptsWaitingForLoad(Resource* resource) {
382 TRACE_EVENT0("blink", "HTMLScriptRunner::executeScriptsWaitingForLoad"); 382 TRACE_EVENT0("blink", "HTMLScriptRunner::executeScriptsWaitingForLoad");
383 ASSERT(!isExecutingScript()); 383 ASSERT(!isExecutingScript());
384 ASSERT(hasParserBlockingScript()); 384 ASSERT(hasParserBlockingScript());
385 ASSERT_UNUSED(resource, m_parserBlockingScript->resource() == resource); 385 DCHECK_EQ(resource, m_parserBlockingScript->resource());
386 ASSERT(m_parserBlockingScript->isReady()); 386 ASSERT(m_parserBlockingScript->isReady());
387 executeParsingBlockingScripts(); 387 executeParsingBlockingScripts();
388 } 388 }
389 389
390 void HTMLScriptRunner::executeScriptsWaitingForResources() { 390 void HTMLScriptRunner::executeScriptsWaitingForResources() {
391 TRACE_EVENT0("blink", "HTMLScriptRunner::executeScriptsWaitingForResources"); 391 TRACE_EVENT0("blink", "HTMLScriptRunner::executeScriptsWaitingForResources");
392 ASSERT(m_document); 392 ASSERT(m_document);
393 // Callers should check hasScriptsWaitingForResources() before calling 393 // Callers should check hasScriptsWaitingForResources() before calling
394 // to prevent parser or script re-entry during </style> parsing. 394 // to prevent parser or script re-entry during </style> parsing.
395 ASSERT(hasScriptsWaitingForResources()); 395 ASSERT(hasScriptsWaitingForResources());
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
529 529
530 DEFINE_TRACE(HTMLScriptRunner) { 530 DEFINE_TRACE(HTMLScriptRunner) {
531 visitor->trace(m_document); 531 visitor->trace(m_document);
532 visitor->trace(m_host); 532 visitor->trace(m_host);
533 visitor->trace(m_parserBlockingScript); 533 visitor->trace(m_parserBlockingScript);
534 visitor->trace(m_scriptsToExecuteAfterParsing); 534 visitor->trace(m_scriptsToExecuteAfterParsing);
535 ScriptResourceClient::trace(visitor); 535 ScriptResourceClient::trace(visitor);
536 } 536 }
537 537
538 } // namespace blink 538 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698