Open
Bug 447829
Opened 17 years ago
Updated 3 years ago
Automate litmus test Testcase ID #5061 - Completed download - context menus
Categories
(Toolkit :: Downloads API, defect)
Toolkit
Downloads API
Tracking
()
NEW
People
(Reporter: poonaatsoc, Unassigned)
References
()
Details
Attachments
(1 file, 1 obsolete file)
|
6.05 KB,
patch
|
sdwilsh
:
review-
|
Details | Diff | Splinter Review |
Proposed type of test: chrome
Proposed location of test: toolkit/mozapps/downloads/tests/chrome
|
Reporter | |
Comment 1•17 years ago
|
||
The test passes. Actually I thought of clubbing this with 447830 and 447831. But since the litmus tests were separate and the tests seemed to give a separate identity to contextmenus of each kind of download, I separated the 3 tests.
Attachment #331156 -
Flags: review?(sdwilsh)
|
|
Reporter | |
Comment 2•17 years ago
|
||
Test updated to accommodate the updated utils.js. The test passes.
Attachment #331156 -
Attachment is obsolete: true
Attachment #331499 -
Flags: review?(sdwilsh)
Attachment #331156 -
Flags: review?(sdwilsh)
|
Assignee | |
Updated•17 years ago
|
Product: Firefox → Toolkit
|
||
Comment 3•17 years ago
|
||
Comment on attachment 331499 [details] [diff] [review]
v2.0
I was about to say that the litmus test in question wasn't worth being a test, but then it occurred to me that we have different menus for different states of the downloads.
So, instead of checking string values, we should just be checking the command attribute on the menuitems. Those won't ever change, and it's more reliable and easier than getting the strings.
Attachment #331499 -
Flags: review?(sdwilsh) → review-
|
||
Comment 4•3 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.
Assignee: poonaatsoc → nobody
Status: ASSIGNED → NEW
|
||
Updated•3 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•