Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

google asking for flash permissions when clicking the adobe test page via search results #13500

Closed
kjozwiak opened this issue Mar 19, 2018 · 4 comments
Assignees

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Mar 19, 2018

Test plan

See #14269

Original issue description

Description

Clicking on https://get.adobe.com/flashplayer/about/ via a Google search result will prompt the user the Allow https://www.google.ca to run Flash Player? notification. Google shouldn't be asking for Flash permissions in this case.

Steps to Reproduce

  1. launch 0.21.20 and enable Flash via about:preferences#plugins
  2. using google, search for flash test
  3. click on the https://get.adobe.com/flashplayer/about/ search result

You'll receive a Allow https://www.google.ca to run Flash Player?. Clicking on Allow or Deny doesn't do anything either.

Actual result:

notificationflash

Expected result:

Clicking on the https://get.adobe.com/flashplayer/about/ search result shouldn't display the Flash permission notification.

Reproduces how often:

100% reproducible when using the STR mentioned above.

Brave Version

about:brave info:

Reproducible on current live release:

Yes, currently reproducible using 0.21.18 f359e6e which is the latest release build.

Additional Information

@kjozwiak kjozwiak added this to the Triage Backlog milestone Mar 19, 2018
@diracdeltas diracdeltas self-assigned this Mar 19, 2018
Slava pushed a commit to Slava/browser-laptop that referenced this issue May 29, 2018
@diracdeltas diracdeltas modified the milestones: Triage Backlog, 0.24.x (Nightly Channel) May 29, 2018
@diracdeltas
Copy link
Member

for reference, this was originally fixed in 4ffcfcb

Slava pushed a commit to Slava/browser-laptop that referenced this issue May 29, 2018
@diracdeltas
Copy link
Member

re-fixed by #14269

@alexwykoff alexwykoff added priority/P5 Cosmetic. Spelling, copy, layout. New features (which should also be part of an initiative). wontfix release-notes/include and removed wontfix labels Jun 5, 2018
@alexwykoff
Copy link
Contributor

Looks like it's merged.

@LaurenWags
Copy link
Member

LaurenWags commented Aug 14, 2018

Verified with macOS 10.12.6 using

  • 0.23.80 53a429f
  • Muon 8.0.8
  • libchromiumcontent 68.0.3440.84

Verified on Ubuntu 18 x64 using

  • 0.23.80 53a429f
  • Muon 8.0.8
  • libchromiumcontent 68.0.3440.84

Verified on Windows x64 with
• 0.23.80 53a429f
• Muon 8.0.8
• libchromiumcontent 68.0.3440.84

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

No branches or pull requests

7 participants