[Webkit-unassigned] [Bug 105280] New: Console log sometimes prefixed with line number

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=105280

           Summary: Console log sometimes prefixed with line number
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: WebCore Misc.
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: jussi.kukkonen at intel.com
                CC: dominik.rottsches at intel.com, mkwst at chromium.org


Several XHR tests have started doing this randomly (~10% of the time) after http://trac.webkit.org/changeset/136657 on at least the EFL WK2 bot:

-CONSOLE MESSAGE: XMLHttpRequest cannot load http://localhost:8000/xmlhttprequest/resources/get.txt. Origin http://127.0.0.1:8000 is not allowed by Access-Control-Allow-Origin.
+CONSOLE MESSAGE: line 58: XMLHttpRequest cannot load http://localhost:8000/xmlhttprequest/resources/get.txt. Origin http://127.0.0.1:8000 is not allowed by Access-Control-Allow-Origin.

I can reproduce this locally. Line 58 of the test in question (http/tests/xmlhttprequest/origin-whitelisting-https.html) is end of file.


I believe this code in WebCore::Console::addMessage is relevant:
  unsigned line = 0;
  if (document && document->parsing() && !document->isInDocumentWrite() && document->scriptableDocumentParser()) {
      ScriptableDocumentParser* parser = document->scriptableDocumentParser();
      if (!parser->isWaitingForScripts() && !parser->isExecutingScript())
          line = parser->lineNumber().oneBasedInt();
  }
  addMessage(source, level, message, url, line, 0, 0, requestIdentifier);

'line' sometimes ends up being set to a value other than zero.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.