839648 - Clean up testGreyLarry.js in default and aurora branches

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect, P5)

Description

[Jonathan French (:jfrench)]

Begat from Comment 100 in Bug 789916:
https://bugzilla.mozilla.org/show_bug.cgi?id=789916#c100

// Check the favicon has no label

> The patch on bug 803088 missed to remove that for
> the default and aurora branch.

So, I'd like to clean up the noted line, in testGreyLarry.js in default and aurora. Please assign me as programmer if appropriate. This will allow small patches which involve this file to be landed on all branches without requiring a subtly different patch. And to prevent the line from being accidentally propagated back to beta, release, and esr17.

Comment 1

[Jonathan French (:jfrench)]

I've built a patch in advance of the bug being assigned. I will attach it momentarily.

Comment 2

[Jonathan French (:jfrench)]

Attachment: clean up greyLarry (default and aurora)

Patch "clean up greyLarry (default and aurora)" for those respective branches. One file affected.

Tested with Default Nightly 21.0a1 20130211031055.
Tested with Aurora 20.0a2 20130211042016.

Tests pass as expected, in both branches and builds.

Comment 3

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 712488 [details] [diff] [review]
clean up greyLarry (default and aurora)

Review of attachment 712488 [details] [diff] [review]:
-----------------------------------------------------------------

Thanks Jonathan, looks great!

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/4fdf37e044ae (default)
http://hg.mozilla.org/qa/mozmill-tests/rev/908006f1baba (mozilla-aurora)