Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Desktop] brave://discards is blank #10451

Closed
dentistformyeye opened this issue Jun 24, 2020 · 8 comments
Closed

[Desktop] brave://discards is blank #10451

dentistformyeye opened this issue Jun 24, 2020 · 8 comments

Comments

@dentistformyeye
Copy link

Description

brave://discards should show information for each tab open in the browser and should allow for manual freezing/discarding of tabs, but a blank, white screen shows up instead.

Steps to Reproduce

  1. Start the Brave Desktop app.
  2. Go to DuckDuckGo in the current new tab.
  3. Open a new tab and go to chrome://discards. On Brave, this will change to brave://discards automatically.

Actual result:

Blank, white page.

Expected result:

image

Reproduces how often:

Always.

Brave version (brave://version info)

Brave 1.10.97 Chromium: 83.0.4103.116 (Official Build) (64-bit)
Revision 8f0c18b4dca9b6699eb629be0f51810c24fb6428-refs/branch-heads/4103@{#716}
OS macOS Version 10.15.4 (Build 19E266)

Version/Channel Information:

  • Can you reproduce this issue with the current release? Yes.
  • Can you reproduce this issue with the beta channel? Haven't tried.
  • Can you reproduce this issue with the dev channel? Haven't tried.
  • Can you reproduce this issue with the nightly channel? Haven't tried.

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? N/A.
  • Does the issue resolve itself when disabling Brave Rewards? N/A.
  • Is the issue reproducible on the latest version of Chrome? No. chrome://discards works normally on Chrome 83.0.4103.116 (tested on WIndows 10). OS might not matter though.

Miscellaneous Information:

chrome://discards is listed in chrome://chrome-urls on Brave Desktop (though not for Brave Android).

@srirambv
Copy link
Contributor

This is a regression possibly on CR 83. I remember it used to work before. Thanks for reporting

@dentistformyeye
Copy link
Author

I remember the bug existed as far back as Brave Desktop 1.8.XX (81.0.4044.129). I don't remember if the bug was in 1.7.XX (81.0.4044.113).

@rebron rebron added the priority/P4 Planned work. We expect to get to it "soon". label Jun 30, 2020
@jonathansampson
Copy link
Contributor

This page fails to load completely due to the following errors:

Refused to load the script 'chrome://brave-resources/br_elements/shared_vars_css.m.js' because it violates the following Content Security Policy directive: "script-src chrome://resources chrome://test 'self'". Note that 'script-src-elem' was not explicitly set, so 'script-src' is used as a fallback.

and

(WEBUI context) extensions::safeMethods:20: Uncaught TypeError: Illegal invocation{TypeError: Illegal invocation
    at Object.define (extensions::safeMethods:20:22)
    at extensions::guestViewContainerElement:67:28
    at registerElement (extensions::guestViewContainerElement:56:28)
    at extensions::chromeWebView:220:1}

@bsclifton
Copy link
Member

bsclifton commented Dec 30, 2020

This is fixed with Chromium 88 (Nightly) 😄 I bet we broke this in Chromium 83/84 upgrade when our code touching Polymer UI needed to be adjusted

@hornetnz
Copy link

hornetnz commented Jan 5, 2021

When will this fix be released?

@kjozwiak
Copy link
Member

kjozwiak commented Jan 5, 2021

When will this fix be released?

As per @bsclifton, this appears to have been fixed via CR88 which is scheduled to hit the release channel on January 19, 2021 as per https://github.com/brave/brave-browser/wiki/Brave-Release-Schedule.

This is fixed with Chromium 88 (Nightly) 😄 I bet we broke this in Chromium 83/84 upgrade when our code touching Polymer UI needed to be adjusted

If that's the case, can we close this off and move it into https://github.com/brave/brave-browser/milestone/155?

@bsclifton
Copy link
Member

@kjozwiak SGTM - will close this and add to milestone. We can call out on release notes since our rebase on Chromium 88 seems to have addressed the root cause

@bsclifton bsclifton added this to the 1.19.x - Release milestone Jan 15, 2021
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Jan 15, 2021

Verification passed on


Brave | 1.19.84 Chromium: 88.0.4324.87 (Official Build) (64-bit)
-- | --
Revision | dd01ff8f58c65af81127ad5c105c79d5b571d8f3-refs/branch-heads/4324@{#1702}
OS | Windows 10 OS Version 2004 (Build 19041.685)

  • Verified the STR from the description
    image

Verified passed with

Brave | 1.19.84 Chromium: 88.0.4324.87 (Official Build) (x86_64)
-- | --
Revision | dd01ff8f58c65af81127ad5c105c79d5b571d8f3-refs/branch-heads/4324@{#1702}
OS | macOS Version 10.15.7 (Build 19H15)

Verified STR from description. Encountered #13600.

Dark Light
Brave-dark Brave-light

Verification passed on

Brave 1.19.84 Chromium: 88.0.4324.87 (Official Build) (64-bit)
Revision dd01ff8f58c65af81127ad5c105c79d5b571d8f3-refs/branch-heads/4324@{#1702}
OS Ubuntu 18.04 LTS

Verified STR from description.
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

10 participants