872065 - Console showing chrome code warnings/errors is very surprising and confusing

Status: RESOLVED DUPLICATE of bug 860312

(DevTools :: Console, defect)

Description

[Victor Porof [:vporof][:vp]]

Attachment: screenshot

STR:

1. Open http://htmlpad.org/debugger/
2. Open debugger
3. Open console
4. Click "Click me" in the page
5. Evaluate "pAsProto"

[19:11:33.219] Ignoring get or set of property that has [LenientThis] because the "this" object is incorrect. @ chrome://global/content/devtools/dbg-script-actors.js:1568

My first reaction as a user would be "wtf?" :)

Comment 1

[Victor Porof [:vporof][:vp]]

Attachment: screenshot 2

Also: 

[19:14:42.590] Use of inputEncoding is deprecated. @ chrome://global/content/devtools/dbg-script-actors.js:1568

..if you evaluate document in that same frame.

Comment 2

[Panos Astithas (he/him) [:past] (please ni?)]

We have another bug on file about this that I can't find atm, and the problem is that I don't know of any way to tell the engine not to log those errors, or to tell them apart in the console observer.

Comment 3

[Girish Sharma [:Optimizer]]

There is bug 802543 for just the opposite, if anyone is interested.

Comment 4

[Mihai Sucan [:msucan]]

(In reply to Panos Astithas [:past] from comment #2)
> We have another bug on file about this that I can't find atm, and the
> problem is that I don't know of any way to tell the engine not to log those
> errors, or to tell them apart in the console observer.

This issue Victor reported seems to be related to bz's explanation of LenientThis, see bug 871510. If we fix that, we also probably fix this issue.

Comment 5

[Panos Astithas (he/him) [:past] (please ni?)]

We'll fix this specific LenientThis warning, but I don't know of any way to have the web console filter out nsIConsoleService messages caused by chrome code (that should only be displayed in the browser console).

Comment 6

[Mihai Sucan [:msucan]]

(In reply to Panos Astithas [:past] from comment #5)
> We'll fix this specific LenientThis warning, but I don't know of any way to
> have the web console filter out nsIConsoleService messages caused by chrome
> code (that should only be displayed in the browser console).

The LenientThis warning is caused by the debugger while it executes code in the web page context. The web console can't tell the difference because the nsIScriptError has the web page window ID. Do we have a warning/error filename location we can filter out? Afaik we do, but in some cases we want those warnings/errors to show (eg. when the user evaluates some code in vview or in jsterm).

Comment 7

[Panos Astithas (he/him) [:past] (please ni?)]

The symptoms mentioned so far have been cured. Should we resolve this bug or add new STR since the underlying issue is still present?

Comment 8

[Mihai Sucan [:msucan]]

(In reply to Panos Astithas [:past] from comment #7)
> The symptoms mentioned so far have been cured. Should we resolve this bug or
> add new STR since the underlying issue is still present?

New STR should make sense.