Last Comment Bug 777242 - Rename utils.assertElementVisible() to utils.isDisplayed()
: Rename utils.assertElementVisible() to utils.isDisplayed()
Product: Mozilla QA
Classification: Other
Component: Mozmill Tests (show other bugs)
: unspecified
: All All
: -- normal (vote)
: ---
Assigned To: Henrik Skupin (:whimboo)
Depends on:
  Show dependency treegraph
Reported: 2012-07-24 23:31 PDT by Henrik Skupin (:whimboo)
Modified: 2012-07-26 06:30 PDT (History)
2 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---

Patch v1 (11.64 KB, patch)
2012-07-24 23:32 PDT, Henrik Skupin (:whimboo)
dave.hunt: review+
Details | Diff | Splinter Review
Patch (backport esr10) (11.61 KB, patch)
2012-07-26 04:30 PDT, Henrik Skupin (:whimboo)
dave.hunt: review+
Details | Diff | Splinter Review

Description Henrik Skupin (:whimboo) 2012-07-24 23:31:17 PDT
There is no need to assert in this method. We now have the assertions module which gives us way nicer feedback. Also with this change tests could check first if an element is visible before clicking on it.
Comment 1 Henrik Skupin (:whimboo) 2012-07-24 23:32:41 PDT
Created attachment 645662 [details] [diff] [review]
Patch v1

Actually this patch depends on bug 777237.
Comment 2 Dave Hunt (:davehunt) 2012-07-25 03:51:11 PDT
Comment on attachment 645662 [details] [diff] [review]
Patch v1

I like it! Landed as: (default)
Comment 3 Henrik Skupin (:whimboo) 2012-07-26 04:29:53 PDT
Landed on other branches except esr10 which needs another patch: (aurora) (beta) (release)
Comment 4 Henrik Skupin (:whimboo) 2012-07-26 04:30:24 PDT
Created attachment 646090 [details] [diff] [review]
Patch (backport esr10)
Comment 5 Henrik Skupin (:whimboo) 2012-07-26 06:30:00 PDT
And landed on esr10:

Note You need to log in before you can comment on or make changes to this bug.