860705 - [browserconsole] Do not display messages from open tabs (only show messages from chrome)

Status: RESOLVED DUPLICATE of bug 1260877

(DevTools :: Console, defect, P3)

Description

[Mihai Sucan [:msucan]]

To lower the number the number of messages we get in the Browser Console we should look into how we can filter out all of the messages that belong to open tabs.

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

See bug 593540 for removing content messages from the Error Console.

Comment 2

[Mihai Sucan [:msucan]]

(In reply to Jared Wein [:jaws] from comment #1)
> See bug 593540 for removing content messages from the Error Console.

Thank you!

Comment 5

[bmax1337]

Attachment: Patch that fixes the bug, still need test.

Hey,

I think this solves the problem just am kind of stuck on writing the test. I'm not sure the best way to go about writing the test.

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8577697 [details] [diff] [review]
Patch that fixes the bug, still need test.

Were you able to take a look at the patches on bug 593540? This also needs to account for about: and resource: URIs for example.

Comment 7

[Matthew N. [:MattN]]

Comment on attachment 8577697 [details] [diff] [review]
Patch that fixes the bug, still need test.

Review of attachment 8577697 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/devtools/webconsole/webconsole.js
@@ +1234,5 @@
>      let node = null;
>  
> +    // if this is an instance of browserconsole only show chrome messages.
> +    if (this.owner._browserConsole && sourceURL.substr(0,9) != "chrome://") {
> +      return null;

Have you checked if sourceURL can sometimes be missing? IIRC, I don't always see one in the console. If we need to use the scheme, it may be easier to blocklist https/http for the Browser Console. Since the scheme checking isn't going to be perfect, I think we may want it to be a pref in the Browser Console. Note that you can use .startsWith too

Comment 8

[Alice0775 White]

Where is messages from page in popup window?
AFAIK, I can not see the message in Web Console. It is only Browser Console.

So, I think this Bug should be wontfix.

Comment 9

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Should this be marked as a dupe of bug 1260877?  Wasn't sure which way to dupe if so.

Comment 10

[Brian Grinstead [:bgrins]]

(In reply to J. Ryan Stinnett [:jryans] (use ni?) from comment #9)
> Should this be marked as a dupe of bug 1260877?  Wasn't sure which way to
> dupe if so.

Yes I think we should dupe this to bug 1260877.  People have different preferences for this behvaior so that bug is about adding a UI to pick / filter.