Closed
Bug 585770
Opened 14 years ago
Closed 14 years ago
Make Mozmill-test testPasswordNotSaved local
Categories
(Mozilla QA Graveyard :: Mozmill Tests, defect)
Mozilla QA Graveyard
Mozmill Tests
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: u279076, Assigned: aaronmt)
References
Details
(Whiteboard: [litmus-data])
Attachments
(3 files)
4.14 KB,
patch
|
u279076
:
review+
whimboo
:
review-
|
Details | Diff | Splinter Review |
3.98 KB,
patch
|
u279076
:
review-
|
Details | Diff | Splinter Review |
3.98 KB,
patch
|
u279076
:
review+
whimboo
:
review+
|
Details | Diff | Splinter Review |
Module: testPasswordManager/testPasswordNotSaved.js Test-page: test-files/password_manager/login_form.html
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → aaron.train
Status: NEW → ASSIGNED
Assignee | ||
Comment 1•14 years ago
|
||
Cleanup and simple swap to local-test data
Attachment #466321 -
Flags: review?(anthony.s.hughes)
Attachment #466321 -
Flags: review?(hskupin)
Attachment #466321 -
Flags: review?(anthony.s.hughes)
Attachment #466321 -
Flags: review+
Comment 2•14 years ago
|
||
Comment on attachment 466321 [details] [diff] [review] Patch v1 - (default) >- controller.assertProperty(removeLogin, 'disabled', 'true'); >+ controller.assertDOMProperty(removeLogin, 'disabled', 'true'); Don't use assertDOMProperty yet. We haven't upgraded Mozmill to 1.5 on our boxes.
Attachment #466321 -
Flags: review?(hskupin) → review-
Assignee | ||
Comment 3•14 years ago
|
||
Now that it has, is the same patch ok?
Comment 4•14 years ago
|
||
Use the old way. We will update all instances at once.
Assignee | ||
Comment 5•14 years ago
|
||
revert back to assertProperty
Attachment #471519 -
Flags: review?(anthony.s.hughes)
Comment on attachment 471519 [details] [diff] [review] Patch v1.1 - (default) >+ pm = Cc["@mozilla.org/login-manager;1"] >+ .getService(Ci.nsILoginManager); Fix . and indentation
Attachment #471519 -
Flags: review?(anthony.s.hughes) → review-
Assignee | ||
Comment 7•14 years ago
|
||
Attachment #473514 -
Flags: review?(anthony.s.hughes)
Attachment #473514 -
Flags: review?(hskupin)
Attachment #473514 -
Flags: review?(anthony.s.hughes)
Attachment #473514 -
Flags: review+
Comment 8•14 years ago
|
||
Comment on attachment 473514 [details] [diff] [review] Patch v1.2 - [nit fix] - (default) r=me. Please file a follow-up bug to separate the two test functions into their own modules.
Attachment #473514 -
Flags: review?(hskupin) → review+
Keywords: checkin-needed
(In reply to comment #8) > Please file a follow-up bug to separate the two test functions into their own > modules. Filed -> bug 599148
Reporter | ||
Comment 10•14 years ago
|
||
(In reply to comment #7) > Created attachment 473514 [details] [diff] [review] > Patch v1.2 - [nit fix] - (default) Landed: http://hg.mozilla.org/qa/mozmill-tests/rev/4f11520b6ac2 [default] http://hg.mozilla.org/qa/mozmill-tests/rev/625a7b91b3cf [mozilla1.9.2] http://hg.mozilla.org/qa/mozmill-tests/rev/1e98eca8ee8b [mozilla1.9.1]
Comment 11•14 years ago
|
||
Move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Product: Testing → Mozilla QA
Updated•5 years ago
|
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•