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

Fixed copy to clipboard(Flash plugin) blocked on Reverso.net #4020

Closed
srirambv opened this issue Sep 15, 2016 · 6 comments
Closed

Fixed copy to clipboard(Flash plugin) blocked on Reverso.net #4020

srirambv opened this issue Sep 15, 2016 · 6 comments

Comments

@srirambv
Copy link
Collaborator

srirambv commented Sep 15, 2016

Test Plan:

  1. go to reverso.net with flash enabled in about:preferences and shields down
  2. enter something in the box and click translate
  3. you should see a flash notification bar. click 'allow'
  4. click on the copy-paste icon in the results box. the page should tell you that the text was copied.

Original issue description

Did you search for similar issues before submitting this one?
Yes

Describe the issue you encountered:
Copy to clipboard(Flash plugin) blocked on Reverso.net.

Expected behavior:
Should allow the plugin to run to enable clipboard functionality

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Windows 10 x64

  • Brave Version:
    0.12.1 RC1 (Clean Install)

  • Steps to reproduce:

    1. Visit http://reverso.net
    2. Type in a word in the Enter/Paste textbox
    3. Select a language to translate
    4. After page loads the clipboard function loads and then gets blocked.
  • Screenshot if needed:
    Flash content on Chrome
    image

    Brave
    image
    Clipboard item blocked after page refresh
    1

  • Any related issues:

cc: @diracdeltas

@diracdeltas
Copy link
Member

@srirambv i don't see a 'copy' element in either Chrome or Brave.
screen shot 2017-03-16 at 6 27 22 pm

@diracdeltas
Copy link
Member

diracdeltas commented Mar 16, 2017

nvm, i found it. there is an issue with flash detection when shields are down.

diracdeltas added a commit that referenced this issue Mar 16, 2017
Separates plugin content setting from flashEnabled setting so that shields down
allows tiny flash elements to be detected. Also improves hidden element detection
for slow/delayed element loads.

Fix #4020

Auditors: @srirambv @bbondy

Test Plan:
1. go to reverso.net with flash enabled in about:preferences and shields down
2. enter something in the box and click translate
3. you should see a flash notification bar. click 'allow'
4. click on the copy-paste icon in the results box. the page should tell you that the text was copied.
@luixxiul luixxiul added this to the 0.14.0 milestone Mar 17, 2017
@luixxiul
Copy link
Contributor

Note: needs next preview to test this (it's on the master branch yet)

jonathansampson pushed a commit that referenced this issue Mar 18, 2017
Separates plugin content setting from flashEnabled setting so that shields down
allows tiny flash elements to be detected. Also improves hidden element detection
for slow/delayed element loads.

Fix #4020

Auditors: @srirambv @bbondy

Test Plan:
1. go to reverso.net with flash enabled in about:preferences and shields down
2. enter something in the box and click translate
3. you should see a flash notification bar. click 'allow'
4. click on the copy-paste icon in the results box. the page should tell you that the text was copied.
@srirambv
Copy link
Collaborator Author

srirambv commented Mar 21, 2017

Not working on Windows x64 0.14.0 Preview 4 build.
reverso

@srirambv srirambv reopened this Mar 21, 2017
@diracdeltas
Copy link
Member

@srirambv as stated in the PR test plan, you have to turn shields down on reverso.net first

@srirambv
Copy link
Collaborator Author

Sorry my bad. Missed the shields down part.. Works as expected.

@alexwykoff alexwykoff changed the title Copy to clipboard(Flash plugin) blocked on Reverso.net Fixed copy to clipboard(Flash plugin) blocked on Reverso.net Mar 28, 2017
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

6 participants