chrome/test/data/permissions/flash.html - Issue 2361453004: Add browsertest for Flash permission prompt

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Side by Side Diff: chrome/test/data/permissions/flash.html

Issue 2361453004: Add browsertest for Flash permission prompt (Closed)

Patch Set: Add browesrtest for Flash permission prompt Created 4 years, 3 months ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View unified diff | Download patch

OLD	NEW
(Empty)
	1 <!DOCTYPE html>

	2 <html>

	3 <script>

	4 function triggerPrompt() {

	5 document.getElementById('flash-link').click();

	6 window.domAutomationController.send(true);

	7 }

	8

	9 function flashIsEnabled() {

	10 var plugin = document.getElementById('flash-object');

	11 function handleEvent(event) {

	12 if (event.data.source === 'getPowerSaverStatusResponse') {

	13 plugin.removeEventListener('message', handleEvent);

	14 window.domAutomationController.send(true);

	15 }

	16 }

	17 plugin.addEventListener('message', handleEvent);
	tommycli 2016/09/26 22:54:49 nit: I've seen this cool pattern recently: plugin nit: I've seen this cool pattern recently: plugin.addEventListener('message', function handleEvent(e) { if (event.data.source === 'getPowerSaverStatusResponse') { plugin.removeEventListener('message', handleEvent); window.domAutomationController.send(true); } }); (The handleEvent is inline) raymes 2016/09/27 04:05:41 Cool, thanks! Cool, thanks!
	18 if (plugin.postMessage) {

	19 plugin.postMessage('getPowerSaverStatus');

	20 } else {

	21 window.domAutomationController.send(false);

	22 }

	23 }

	24 </script>

	25 <body>

	26 <object id="flash-object" name="plugin" data="test.swf"

	27 type="application/x-shockwave-flash" width="400" height="100">

	28 Flash not supported.

	29 <br>

	30 <a href="https://get.adobe.com/flashplayer/" id="flash-link">Download Flash. </a>
	tommycli 2016/09/26 22:54:49 need line wrap here? need line wrap here? raymes 2016/09/27 04:05:41 Done. Show quoted text On 2016/09/26 22:54:49, tommycli wrote: > need line wrap here? Done.
	31 </object>

	32 </body>

	33 </html>

OLD	NEW

« chrome/browser/permissions/permissions_browsertest.cc ('K') | « chrome/test/BUILD.gn ('k') | content/public/test/ppapi_test_utils.h » ('j') | content/public/test/ppapi_test_utils.h » ('J')