LayoutTests/fast/dom/HTMLElement/contextmenu.html - Issue 464903002: Add contextmenu attribute for HTMLElement.

Side by Side Diff: LayoutTests/fast/dom/HTMLElement/contextmenu.html

Issue 464903002: Add contextmenu attribute for HTMLElement. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master

Patch Set: Created 6 years, 4 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 PUBLIC "-//IETF//DTD HTML//EN">

	2 <html>

	3 <head>

	4 <script src="../../../resources/js-test.js"></script>

	5 </head>

	6 <script>

	7

	8 description('Tests the contextmenu attribute.');

	9

	10 debug("Test with valid contextmenu attribute");

	11 document.write("<div id='container1' contextmenu='menu1'><menu id='menu1' type=' popup' label='menu 1'></menu></div>");

	12 shouldBeEqualToString("document.getElementById('container1').contextMenu.id", "m enu1");

	13 shouldBeEqualToString("document.getElementById('container1').contextMenu.type", "popup");

	14 shouldBeEqualToString("document.getElementById('container1').contextMenu.label", "menu 1");

	15

	16 debug("Test with invalid contextmenu attribute");

	17 document.write("<div id='container2' contextmenu='nonexistent_menu'><menu id='me nu2' type='popup' label='menu 2'></menu></div>");

	18 shouldBeNull("document.getElementById('container2').contextMenu");

	19

	20 debug("Test setting contextmenu attribute");

	21 document.write("<div id='container3'></div>");

	22 var menu = document.createElement('menu');

	23 menu.id = 'menu3';

	24 menu.type = 'popup';

	25 menu.label = 'menu 3';

	26 document.getElementById('container3').appendChild(menu);

	27 var container3 = document.getElementById('container3');

	28 container3.contextMenu = menu;

	29 shouldBeEqualToString("container3.contextMenu.id", "menu3");

	30 shouldBeEqualToString("container3.contextMenu.type", "popup");

	31 shouldBeEqualToString("container3.contextMenu.label", "menu 3");

	32

	33 debug("Test setting contextmenu attribute with menu element without id");

	34 document.write("<div id='container4'></div>");

	35 var menu1 = document.createElement('menu');

	36 menu1.type = 'popup';

	37 menu1.label = 'menu 4';

	38 document.getElementById('container4').appendChild(menu1);

	39 var container4 = document.getElementById('container4');

	40 container4.contextMenu = menu1;

	41 shouldBeNull("container4.contextMenu");

	42

	43 debug("Test setting contextmenu attribute with multiple menu element with same i d");

	44 document.write("<div id='container5'></div>");

	45 var menu2 = document.createElement('menu');

	46 menu2.id = 'menu5';

	47 menu2.type = 'popup';

	48 menu2.label = 'menu 5';

	49 var menu3 = document.createElement('menu');

	50 menu3.id = 'menu5';

	51 menu3.type = 'popup';

	52 menu3.label = 'menu 6';

	53 document.getElementById('container5').appendChild(menu2);

	54 document.getElementById('container5').appendChild(menu3);

	55 var container5 = document.getElementById('container5');

	56 container5.contextMenu = menu3;
	pals 2014/08/17 17:18:12 Specification http://www.whatwg.org/specs/web-apps Specification http://www.whatwg.org/specs/web-apps/current-work/#reflect under section "On setting, if the given element has an id attribute, and has the same home subtree as the element of the attribute being set, and the given element is the FIRST ELEMENT in that home subtree whose ID is the value of that id attribute, then the content attribute must be set to the value of that id attribute. Otherwise, the content attribute must be set to the empty string." tkent 2014/08/18 02:24:52 Please write this paragraph as a code comment of H Show quoted text On 2014/08/17 17:18:12, OOO_13th-18thAug wrote: > Specification > http://www.whatwg.org/specs/web-apps/current-work/#reflect > under section > "On setting, if the given element has an id attribute, and has the same home > subtree as the element of the attribute being set, and the given element is the > FIRST ELEMENT in that home subtree whose ID is the value of that id attribute, > then the content attribute must be set to the value of that id attribute. > Otherwise, the content attribute must be set to the empty string." Please write this paragraph as a code comment of HTMLElement::setContextMenu. \|menu3\| is NOT the first element. The first element with id=menu5 is \|menu2\|. So \|container5.contextMenu\| should be null. pals 2014/08/18 14:23:24 Done. Show quoted text On 2014/08/18 02:24:52, tkent wrote: > On 2014/08/17 17:18:12, OOO_13th-18thAug wrote: > > Specification > > http://www.whatwg.org/specs/web-apps/current-work/#reflect > > under section > > "On setting, if the given element has an id attribute, and has the same home > > subtree as the element of the attribute being set, and the given element is > the > > FIRST ELEMENT in that home subtree whose ID is the value of that id attribute, > > then the content attribute must be set to the value of that id attribute. > > Otherwise, the content attribute must be set to the empty string." > > Please write this paragraph as a code comment of HTMLElement::setContextMenu. > > \|menu3\| is NOT the first element. The first element with id=menu5 is \|menu2\|. > So \|container5.contextMenu\| should be null. Done.
	57 shouldBeEqualToString("container5.contextMenu.id", "menu5");

	58 shouldBeEqualToString("container5.contextMenu.type", "popup");

	59 shouldBeEqualToString("container5.contextMenu.label", "menu 5");

	60

	61 debug("Test setting contextmenu attribute to null");

	62 document.write("<div id='container6'></div>");

	63 var container6 = document.getElementById('container6');

	64 container6.contextMenu = null;

	65 shouldBeNull("container6.contextMenu");

	66

	67 debug("Test setting contextmenu attribute to some text");

	68 document.write("<div id='container7'></div>");

	69 var container7 = document.getElementById('container7');

	70 container7.contextMenu = "foobar";

	71 shouldBeNull("container7.contextMenu");

	72

	73 debug("Test setting contextmenu attribute to comment");

	74 document.write("<div id='container8'></div>");

	75 var container8 = document.getElementById('container8');

	76 container8.contextMenu = document.createComment('foo');

	77 shouldBeNull("container8.contextMenu");

	78

	79 debug("Test setting contextmenu attribute to menu without type");

	80 document.write("<div id='container9'></div>");

	81 var menu4 = document.createElement('menu');

	82 menu4.id = 'menu_without_type';

	83 menu4.label = 'menu without type';

	84 document.getElementById('container9').appendChild(menu4);

	85 var container9 = document.getElementById('container9');

	86 container9.contextMenu = menu4;

	87 shouldBeNull("container9.contextMenu");
	pals 2014/08/17 17:18:12 type of menu should be "popup" for contextmenu att type of menu should be "popup" for contextmenu attribute.
	88 </script>

	89 </html>

OLD	NEW

« no previous file with comments | « no previous file | LayoutTests/fast/dom/HTMLElement/contextmenu-expected.txt » ('j') | LayoutTests/fast/dom/HTMLElement/contextmenu-expected.txt » ('J')