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

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

Issue 2906603002: Remove WTFLogAlways() usages from HTMLParserScriptRunner.cpp (Closed)
Patch Set: Remove WTFLogAlways() as suggested Created 3 years, 7 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 | « no previous file | 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) 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 283 matching lines...) Expand 10 before | Expand all | Expand 10 after
294 void EmitWarningForDocWriteScripts(const String& url, Document& document) { 294 void EmitWarningForDocWriteScripts(const String& url, Document& document) {
295 String message = 295 String message =
296 "The Parser-blocking, cross site (i.e. different eTLD+1) " 296 "The Parser-blocking, cross site (i.e. different eTLD+1) "
297 "script, " + 297 "script, " +
298 url + 298 url +
299 ", invoked via document.write was NOT BLOCKED on this page load, but MAY " 299 ", invoked via document.write was NOT BLOCKED on this page load, but MAY "
300 "be blocked by the browser in future page loads with poor network " 300 "be blocked by the browser in future page loads with poor network "
301 "connectivity."; 301 "connectivity.";
302 document.AddConsoleMessage( 302 document.AddConsoleMessage(
303 ConsoleMessage::Create(kJSMessageSource, kWarningMessageLevel, message)); 303 ConsoleMessage::Create(kJSMessageSource, kWarningMessageLevel, message));
304 WTFLogAlways("%s", message.Utf8().data());
305 } 304 }
306 305
307 void EmitErrorForDocWriteScripts(const String& url, Document& document) { 306 void EmitErrorForDocWriteScripts(const String& url, Document& document) {
308 String message = 307 String message =
309 "Network request for the parser-blocking, cross site " 308 "Network request for the parser-blocking, cross site "
310 "(i.e. different eTLD+1) script, " + 309 "(i.e. different eTLD+1) script, " +
311 url + 310 url +
312 ", invoked via document.write was BLOCKED by the browser due to poor " 311 ", invoked via document.write was BLOCKED by the browser due to poor "
313 "network connectivity. "; 312 "network connectivity. ";
314 document.AddConsoleMessage( 313 document.AddConsoleMessage(
315 ConsoleMessage::Create(kJSMessageSource, kErrorMessageLevel, message)); 314 ConsoleMessage::Create(kJSMessageSource, kErrorMessageLevel, message));
316 WTFLogAlways("%s", message.Utf8().data());
317 } 315 }
318 316
319 void HTMLParserScriptRunner::PossiblyFetchBlockedDocWriteScript( 317 void HTMLParserScriptRunner::PossiblyFetchBlockedDocWriteScript(
320 PendingScript* pending_script) { 318 PendingScript* pending_script) {
321 // If the script was blocked as part of document.write intervention, 319 // If the script was blocked as part of document.write intervention,
322 // then send an asynchronous GET request with an interventions header. 320 // then send an asynchronous GET request with an interventions header.
323 321
324 if (!ParserBlockingScript()) 322 if (!ParserBlockingScript())
325 return; 323 return;
326 324
(...skipping 351 matching lines...) Expand 10 before | Expand all | Expand 10 after
678 676
679 DEFINE_TRACE(HTMLParserScriptRunner) { 677 DEFINE_TRACE(HTMLParserScriptRunner) {
680 visitor->Trace(document_); 678 visitor->Trace(document_);
681 visitor->Trace(host_); 679 visitor->Trace(host_);
682 visitor->Trace(parser_blocking_script_); 680 visitor->Trace(parser_blocking_script_);
683 visitor->Trace(scripts_to_execute_after_parsing_); 681 visitor->Trace(scripts_to_execute_after_parsing_);
684 PendingScriptClient::Trace(visitor); 682 PendingScriptClient::Trace(visitor);
685 } 683 }
686 684
687 } // namespace blink 685 } // namespace blink
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698