Bug 1000513 - Tests for menugroup keyboard navigation. r=Unfocused

--HG--
extra : rebase_source : 925b6991f3444ff197b166f1e4234f1f19fe321d
This commit is contained in:
Jared Wein 2014-05-15 18:14:15 -04:00
parent 062700e6ea
commit 9aca6f8285
8 changed files with 218 additions and 78 deletions

View File

@ -269,7 +269,8 @@ nsContextMenu.prototype = {
// Use "Bookmark This Link" if on a link.
this.showItem("context-bookmarkpage",
!(this.isContentSelected || this.onTextInput || this.onLink ||
this.onImage || this.onVideo || this.onAudio || this.onSocial));
this.onImage || this.onVideo || this.onAudio || this.onSocial ||
this.onCanvas));
this.showItem("context-bookmarklink", (this.onLink && !this.onMailtoLink &&
!this.onSocial) || this.onPlainTextLink);
this.showItem("context-keywordfield",

View File

@ -26,11 +26,13 @@ function test() {
EventUtils.synthesizeMouseAtCenter(video1,
{ type: "contextmenu", button: 2 },
gBrowser.contentWindow);
info("context menu click on video1");
});
});
function contextMenuOpened(event) {
event.currentTarget.removeEventListener("popupshown", contextMenuOpened);
info("context menu opened on video1");
// Create the folder the video will be saved into.
var destDir = createTemporarySaveDirectory();
@ -56,6 +58,7 @@ function test() {
// Select "Save Video As" option from context menu
var saveVideoCommand = document.getElementById("context-savevideo");
saveVideoCommand.doCommand();
info("context-savevideo command executed");
event.target.hidePopup();
}

View File

@ -96,10 +96,23 @@ function getVisibleMenuItems(aMenu, aData) {
}
items.push(item.id);
items.push(!item.disabled);
// Add a dummy item to that the indexes in checkMenu are the same
// Add a dummy item so that the indexes in checkMenu are the same
// for expectedItems and actualItems.
items.push([]);
items.push(null);
} else if (item.nodeName == "menugroup") {
ok(item.id, "child menugroup #" + i + " has an ID");
items.push(item.id);
items.push(!item.disabled);
var menugroupChildren = [];
for (var child of item.children) {
if (child.hidden)
continue;
menugroupChildren.push([child.id, !child.disabled]);
}
items.push(menugroupChildren);
items.push(null);
} else {
ok(false, "child #" + i + " of menu ID " + aMenu.id +
" has an unknown type (" + item.nodeName + ")");
@ -114,6 +127,43 @@ function checkContextMenu(expectedItems) {
checkMenu(contextMenu, expectedItems, data);
}
function checkMenuItem(actualItem, actualEnabled, expectedItem, expectedEnabled, index) {
is(actualItem, expectedItem,
"checking item #" + index/2 + " (" + expectedItem + ") name");
if (typeof expectedEnabled == "object" && expectedEnabled != null ||
typeof actualEnabled == "object" && actualEnabled != null) {
ok(!(actualEnabled == null), "actualEnabled is not null");
ok(!(expectedEnabled == null), "expectedEnabled is not null");
is(typeof actualEnabled, typeof expectedEnabled, "checking types");
if (typeof actualEnabled != typeof expectedEnabled ||
actualEnabled == null || expectedEnabled == null)
return;
is(actualEnabled.type, expectedEnabled.type,
"checking item #" + index/2 + " (" + expectedItem + ") type attr value");
var icon = actualEnabled.icon;
if (icon) {
var tmp = "";
var j = icon.length - 1;
while (j && icon[j] != "/") {
tmp = icon[j--] + tmp;
}
icon = tmp;
}
is(icon, expectedEnabled.icon,
"checking item #" + index/2 + " (" + expectedItem + ") icon attr value");
is(actualEnabled.checked, expectedEnabled.checked,
"checking item #" + index/2 + " (" + expectedItem + ") has checked attr");
is(actualEnabled.disabled, expectedEnabled.disabled,
"checking item #" + index/2 + " (" + expectedItem + ") has disabled attr");
} else if (expectedEnabled != null)
is(actualEnabled, expectedEnabled,
"checking item #" + index/2 + " (" + expectedItem + ") enabled state");
}
/*
* checkMenu - checks to see if the specified <menupopup> contains the
* expected items and state.
@ -136,50 +186,26 @@ function checkMenu(menu, expectedItems, data) {
var expectedItem = expectedItems[i];
var expectedEnabled = expectedItems[i + 1];
if (expectedItem instanceof Array) {
ok(true, "Checking submenu...");
var menuID = expectedItems[i - 2]; // The last item was the menu ID.
var submenu = menu.getElementsByAttribute("id", menuID)[0];
ok(submenu, "got a submenu element of id='" + menuID + "'");
if (submenu) {
is(submenu.nodeName, "menu", "submenu element of id='" + menuID +
ok(true, "Checking submenu/menugroup...");
var previousId = expectedItems[i - 2]; // The last item was the menu ID.
var previousItem = menu.getElementsByAttribute("id", previousId)[0];
ok(previousItem, (previousItem ? previousItem.nodeName : "item") + " with previous id (" + previousId + ") found");
if (previousItem && previousItem.nodeName == "menu") {
ok(previousItem, "got a submenu element of id='" + previousId + "'");
is(previousItem.nodeName, "menu", "submenu element of id='" + previousId +
"' has expected nodeName");
checkMenu(submenu.menupopup, expectedItem, data);
checkMenu(previousItem.menupopup, expectedItem, data, i);
} else if (previousItem && previousItem.nodeName == "menugroup") {
ok(expectedItem.length, "menugroup must not be empty");
for (var j = 0; j < expectedItem.length / 2; j++) {
checkMenuItem(actualItems[i][j][0], actualItems[i][j][1], expectedItem[j*2], expectedItem[j*2+1], i+j*2);
}
i += j;
} else {
ok(false, "previous item is not a menu or menugroup");
}
} else {
is(actualItem, expectedItem,
"checking item #" + i/2 + " (" + expectedItem + ") name");
if (typeof expectedEnabled == "object" && expectedEnabled != null ||
typeof actualEnabled == "object" && actualEnabled != null) {
ok(!(actualEnabled == null), "actualEnabled is not null");
ok(!(expectedEnabled == null), "expectedEnabled is not null");
is(typeof actualEnabled, typeof expectedEnabled, "checking types");
if (typeof actualEnabled != typeof expectedEnabled ||
actualEnabled == null || expectedEnabled == null)
continue;
is(actualEnabled.type, expectedEnabled.type,
"checking item #" + i/2 + " (" + expectedItem + ") type attr value");
var icon = actualEnabled.icon;
if (icon) {
var tmp = "";
var j = icon.length - 1;
while (j && icon[j] != "/") {
tmp = icon[j--] + tmp;
}
icon = tmp;
}
is(icon, expectedEnabled.icon,
"checking item #" + i/2 + " (" + expectedItem + ") icon attr value");
is(actualEnabled.checked, expectedEnabled.checked,
"checking item #" + i/2 + " (" + expectedItem + ") has checked attr");
is(actualEnabled.disabled, expectedEnabled.disabled,
"checking item #" + i/2 + " (" + expectedItem + ") has disabled attr");
} else if (expectedEnabled != null)
is(actualEnabled, expectedEnabled,
"checking item #" + i/2 + " (" + expectedItem + ") enabled state");
checkMenuItem(actualItem, actualEnabled, expectedItem, expectedEnabled, i);
}
}
// Could find unexpected extra items at the end...

View File

@ -91,11 +91,12 @@ function runTest(testNum) {
case 2:
// Context menu for plain text
plainTextItems = ["context-back", false,
"context-forward", false,
"context-reload", true,
plainTextItems = ["context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,
@ -163,7 +164,6 @@ function runTest(testNum) {
// Context menu for a canvas
checkContextMenu(["context-viewimage", true,
"context-saveimage", true,
"context-bookmarkpage", true,
"context-selectall", true
].concat(inspectItems));
closeContextMenu();
@ -264,11 +264,12 @@ function runTest(testNum) {
case 11:
// Context menu for an iframe
checkContextMenu(["context-back", false,
"context-forward", false,
"context-reload", true,
checkContextMenu(["context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,
@ -468,11 +469,12 @@ function runTest(testNum) {
"---", null,
"+Checkbox", {type: "checkbox", icon: "", checked: false, disabled: false}], null,
"---", null,
"context-back", false,
"context-forward", false,
"context-reload", true,
"context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,
@ -501,11 +503,12 @@ function runTest(testNum) {
// Context menu for DOM Fullscreen mode (NOTE: this is *NOT* on an img)
checkContextMenu(["context-leave-dom-fullscreen", true,
"---", null,
"context-back", false,
"context-forward", false,
"context-reload", true,
"context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,
@ -529,11 +532,12 @@ function runTest(testNum) {
case 21:
// Context menu for element with assigned content context menu
// The shift key should bypass content context menu processing
checkContextMenu(["context-back", false,
"context-forward", false,
"context-reload", true,
checkContextMenu(["context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,
@ -693,11 +697,12 @@ function runTest(testNum) {
checkContextMenu(["context-ctp-play", true,
"context-ctp-hide", true,
"---", null,
"context-back", false,
"context-forward", false,
"context-reload", true,
"context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,
@ -730,11 +735,12 @@ function runTest(testNum) {
case 29:
// Context menu for an iframe with srcdoc attribute set
checkContextMenu(["context-back", false,
"context-forward", false,
"context-reload", true,
checkContextMenu(["context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,

View File

@ -189,11 +189,12 @@ function runTest(testNum) {
case 13: // type='number'
case 14: // type='color'
case 15: // type='range'
checkContextMenu(["context-back", false,
"context-forward", false,
"context-reload", true,
checkContextMenu(["context-navigation", null,
["context-back", false,
"context-forward", false,
"context-reload", true,
"context-bookmarkpage", true], null,
"---", null,
"context-bookmarkpage", true,
"context-savepage", true,
"---", null,
"context-viewbgimage", false,

View File

@ -74,9 +74,12 @@ function test1() {
// When the popupshown DOM event is fired, the actual showing of the popup
// may still be pending. Clear the event loop before continuing so that
// subsequently-opened popups aren't cancelled by accident.
let goToNext = function() {
let goToNext = function(aEvent) {
window.document.removeEventListener("popupshown", goToNext, false);
executeSoon(test2);
executeSoon(function() {
test2();
aEvent.target.hidePopup();
});
};
window.document.addEventListener("popupshown", goToNext, false);
EventUtils.synthesizeMouseAtCenter(plugin,

View File

@ -5,6 +5,7 @@ support-files =
window_menubar.xul
[test_contextmenu_nested.xul]
[test_contextmenu_menugroup.xul]
[test_editor_currentURI.xul]
[test_menubar.xul]
skip-if = os == 'mac'

View File

@ -0,0 +1,99 @@
<?xml version="1.0"?>
<?xml-stylesheet href="chrome://global/skin" type="text/css"?>
<?xml-stylesheet href="/tests/SimpleTest/test.css" type="text/css"?>
<window title="Context menugroup Tests"
xmlns="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<script type="application/javascript" src="chrome://mochikit/content/tests/SimpleTest/SimpleTest.js"></script>
<script type="application/javascript" src="chrome://mochikit/content/tests/SimpleTest/EventUtils.js"></script>
<script type="application/javascript" src="popup_shared.js"></script>
<menupopup id="context">
<menugroup>
<menuitem id="a"/>
<menuitem id="b"/>
</menugroup>
<menuitem id="c" label="c"/>
</menupopup>
<button label="Check"/>
<vbox id="popuparea" popup="context" width="20" height="20"/>
<script type="application/javascript">
<![CDATA[
SimpleTest.waitForExplicitFinish();
var gMenuPopup = $("context");
ok(gMenuPopup, "Got the reference to the context menu");
var popupTests = [
{
testname: "one-down-key",
condition: function() { return (navigator.platform.indexOf("Mac") == -1); },
events: [ "popupshowing context", "popupshown context", "DOMMenuItemActive a" ],
test: function () {
synthesizeMouse($("popuparea"), 4, 4, {});
synthesizeKey("VK_DOWN", {});
},
result: function (testname) {
checkActive(gMenuPopup, "a", testname);
}
},
{
testname: "two-down-keys",
condition: function() { return (navigator.platform.indexOf("Mac") == -1); },
events: [ "DOMMenuItemInactive a", "DOMMenuItemActive b" ],
test: function () synthesizeKey("VK_DOWN", {}),
result: function (testname) {
checkActive(gMenuPopup, "b", testname);
}
},
{
testname: "three-down-keys",
condition: function() { return (navigator.platform.indexOf("Mac") == -1); },
events: [ "DOMMenuItemInactive b", "DOMMenuItemActive c" ],
test: function () synthesizeKey("VK_DOWN", {}),
result: function (testname) {
checkActive(gMenuPopup, "c", testname);
}
},
{
testname: "three-down-keys-one-up-key",
condition: function() { return (navigator.platform.indexOf("Mac") == -1); },
events: [ "DOMMenuItemInactive c", "DOMMenuItemActive b" ],
test: function () synthesizeKey("VK_UP", {}),
result: function (testname) {
checkActive(gMenuPopup, "b", testname);
}
},
{
testname: "three-down-keys-two-up-keys",
condition: function() { return (navigator.platform.indexOf("Mac") == -1); },
events: [ "DOMMenuItemInactive b", "DOMMenuItemActive a" ],
test: function () synthesizeKey("VK_UP", {}),
result: function (testname) {
checkActive(gMenuPopup, "a", testname);
}
},
{
testname: "three-down-keys-three-up-key",
condition: function() { return (navigator.platform.indexOf("Mac") == -1); },
events: [ "DOMMenuItemInactive a", "DOMMenuItemActive c" ],
test: function () synthesizeKey("VK_UP", {}),
result: function (testname) {
checkActive(gMenuPopup, "c", testname);
}
},
];
SimpleTest.waitForFocus(function runTest() startPopupTests(popupTests));
]]>
</script>
<body xmlns="http://www.w3.org/1999/xhtml"><p id="display"/></body>
</window>