-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
brave:// pages show up in browsing history #3493
Labels
bug
priority/P2
A bad problem. We might uplift this to the next planned release.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
regression
release-notes/include
Milestone
Comments
cndouglas
changed the title
History special page shows up in browsing history
brave:// pages shows up in browsing history
Feb 25, 2019
cndouglas
changed the title
brave:// pages shows up in browsing history
brave:// pages show up in browsing history
Feb 25, 2019
Reproduced on Linux as well |
rebron
added
the
priority/P2
A bad problem. We might uplift this to the next planned release.
label
Feb 26, 2019
cc: @simonhong @bridiver can you take a look? |
Not reproducible on Release and Beta
Reproducible on dev
|
19 tasks
Verification passed on
Verified passed with
Verified passed with
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
bug
priority/P2
A bad problem. We might uplift this to the next planned release.
QA Pass-Linux
QA Pass-macOS
QA Pass-Win64
QA/Yes
regression
release-notes/include
Description
Special pages (that begin with brave://, such as brave://history/) show up in the browsing history.
Steps to Reproduce
Actual result:
The history page is in the browsing history.
Expected result:
The history page should not be in the browsing history.
Reproduces how often:
Always
Brave version (brave://version info)
Brave | 0.62.8 Chromium: 73.0.3683.39 (Official Build) dev(64-bit)
Revision | cc53b0e12fcaf42e4bab8d6c23bd4fb7aae99f6c-refs/branch-heads/3683@{#413}
OS | Mac OS X
Reproducible on current release:
Website problems only:
Additional Information
The text was updated successfully, but these errors were encountered: