Serge Gautherie (:sgautherie) Assignee
	Description • 13 years ago

While working on bug 726505, I noticed bug 658738 comment 102 changeset...

	Serge Gautherie (:sgautherie) Assignee
	Comment 1 • 13 years ago

... and the only other changeset to port (so far) is bug 658738 comment 17 changeset.

*****

In the end,
http://mxr.mozilla.org/comm-central/search?string=addEventListener&case=1&find=%2Fsuite%2F.*%2Ftest.*%2Fbrowser
"Found 151 matching lines in 54 files"
and
http://mxr.mozilla.org/comm-central/search?string=removeEventListener&case=1&find=%2Fsuite%2F.*%2Ftest.*%2Fbrowser
"Found 148 matching lines in 50 files"
should match, in total and also wrt each test.

	Serge Gautherie (:sgautherie) Assignee
	Comment 3 • 13 years ago

... and none of bug 658738 blocking bugs fixed in so far affects SeaMonkey.

	Serge Gautherie (:sgautherie) Assignee
	Comment 4 • 13 years ago

Comment on attachment 597284 [details] [diff] [review]
(Av1) Port bug 658738 comment 17 (Add missing removeEventListener() calls), Fix a mismatch in browser_394759.js
[Checked in: Comment 4]

http://hg.mozilla.org/comm-central/rev/074ad890780f

	Serge Gautherie (:sgautherie) Assignee
	Comment 5 • 13 years ago

A step to more sync'.

	Serge Gautherie (:sgautherie) Assignee
	Comment 6 • 13 years ago

*FF changeset cd888c5d77f0 and changeset a6fe7bb368a9: usual Dao's "no bug no review".
*browserWindowsCount() calls: missed in SM bug 633711.
*s/gBrowser/getBrowser()/: mistakenly undone in SM bug 510890.

	Serge Gautherie (:sgautherie) Assignee
	Comment 7 • 13 years ago

Comment on attachment 604432 [details] [diff] [review]
(Bv1) Port |Bug 595292 - browser_394759.js seems to hit network with dns failures|
[Checked in: Comment 10]

Whoever reviews this first.

	neil@parkwaycc.co.uk
	Comment 8 • 13 years ago

Comment on attachment 604464 [details] [diff] [review]
(Cv1) browser_394759.js: Port FF changeset cd888c5d77f0 and changeset a6fe7bb368a9, Add browserWindowsCount() calls, Do 3 s/gBrowser/getBrowser()/
[Checked in: Comment 19]

r=me on the window/win rename; where was the browserWindowsCount() added?

	Serge Gautherie (:sgautherie) Assignee
	Comment 9 • 13 years ago

Comment on attachment 604464 [details] [diff] [review]
(Cv1) browser_394759.js: Port FF changeset cd888c5d77f0 and changeset a6fe7bb368a9, Add browserWindowsCount() calls, Do 3 s/gBrowser/getBrowser()/
[Checked in: Comment 19]

(In reply to neil@parkwaycc.co.uk from comment #8)
> where was the browserWindowsCount() added?

In Firefox?
http://hg.mozilla.org/mozilla-central/rev/37f7608bae72
http://hg.mozilla.org/mozilla-central/rev/13601aff9814

	Serge Gautherie (:sgautherie) Assignee
	Comment 10 • 13 years ago

Comment on attachment 604432 [details] [diff] [review]
(Bv1) Port |Bug 595292 - browser_394759.js seems to hit network with dns failures|
[Checked in: Comment 10]

http://hg.mozilla.org/comm-central/rev/3eaa70af5151

	Justin Wood (:Callek)
	Comment 11 • 13 years ago

Comment on attachment 604432 [details] [diff] [review]
(Bv1) Port |Bug 595292 - browser_394759.js seems to hit network with dns failures|
[Checked in: Comment 10]

too late for beta, That is already in comm-release now, and I don't want to take this in comm-release, if we get this into aurora before tuesday then it will be in beta as of then.

	Serge Gautherie (:sgautherie) Assignee
	Comment 12 • 13 years ago

Jens, context has changed on -central, please apply it manually to -aurora. Thanks.

	Jens Hatlak (:InvisibleSmiley)
	Comment 13 • 13 years ago

(In reply to Serge Gautherie (:sgautherie) from comment #12)
> Jens, context has changed on -central, please apply it manually to -aurora.

Now this is an understatement. The file to patch isn't there, and has only been created as a copy of browser_bug510890.js through changeset cff76f26dd7b (landed Wed Feb 15). Clearing c-n request.

	Serge Gautherie (:sgautherie) Assignee
	Comment 14 • 13 years ago

Same as Bv1, but for comm-aurora/2.9.


(In reply to Jens Hatlak (:InvisibleSmiley) from comment #13)
> copy of browser_bug510890.js

Sorry, I had forgotten it had been renamed.

	Serge Gautherie (:sgautherie) Assignee
	Comment 15 • 13 years ago

(In reply to Serge Gautherie (:sgautherie) from comment #10)
> http://hg.mozilla.org/comm-central/rev/3eaa70af5151

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1331364554.1331369643.15790.gz&fulltext=1
OS X 10.5 comm-central-trunk debug test mochitest-other on 2012/03/09 23:29:14

still succeeds.

	Jens Hatlak (:InvisibleSmiley)
	Comment 16 • 13 years ago

Comment on attachment 604643 [details] [diff] [review]
(BBv1-29) Port |Bug 595292 - browser_394759.js seems to hit network with dns failures| [Checkin: Comment 16]

http://hg.mozilla.org/releases/comm-aurora/rev/c0666f5a239c

	Jens Hatlak (:InvisibleSmiley)
	Comment 17 • 13 years ago

[No idea whether this is supposed to "fix" 2.9, so leaving flags untouched.]

	neil@parkwaycc.co.uk
	Comment 18 • 13 years ago

Comment on attachment 604464 [details] [diff] [review]
(Cv1) browser_394759.js: Port FF changeset cd888c5d77f0 and changeset a6fe7bb368a9, Add browserWindowsCount() calls, Do 3 s/gBrowser/getBrowser()/
[Checked in: Comment 19]

Ah, I see now, misak added browserWindowsCount() but not the checks.

	Serge Gautherie (:sgautherie) Assignee
	Comment 19 • 13 years ago

Comment on attachment 604464 [details] [diff] [review]
(Cv1) browser_394759.js: Port FF changeset cd888c5d77f0 and changeset a6fe7bb368a9, Add browserWindowsCount() calls, Do 3 s/gBrowser/getBrowser()/
[Checked in: Comment 19]

http://hg.mozilla.org/comm-central/rev/5d0e20c97905

	Serge Gautherie (:sgautherie) Assignee
	Comment 20 • 13 years ago

Retained differences with Firefox:
Services.prefs + getBrowser() + listener names + nits.

	Serge Gautherie (:sgautherie) Assignee
	Comment 21 • 13 years ago

(In reply to Serge Gautherie (:sgautherie) from comment #1)
> In the end,
> http://mxr.mozilla.org/comm-central/
> search?string=addEventListener&case=1&find=%2Fsuite%2F.*%2Ftest.*%2Fbrowser
> "Found 151 matching lines in 54 files"
> and
> http://mxr.mozilla.org/comm-central/
> search?string=removeEventListener&case=1&find=%2Fsuite%2F.*%2Ftest.
> *%2Fbrowser
> "Found 148 matching lines in 50 files"
> should match, in total and also wrt each test.

Add   : "Found 152 matching lines in 54 files"
Remove: "Found 151 matching lines in 53 files"

The differences in counts are:
*http://mxr.mozilla.org/comm-central/find?text=&string=browser_form_restore_events_sample.html
 has 2 document.addEventListener(), which should automatically go away with the document.
*http://mxr.mozilla.org/comm-central/find?string=browser_popupNotification.js
 has an extra (conditional) removeEventListener() call in its cleanup(), as a safeguard.

Assuming all the others match correctly, /suite should be fine wrt that.

	Serge Gautherie (:sgautherie) Assignee
	Comment 22 • 13 years ago

Comment on attachment 605052 [details] [diff] [review]
(Dv1) Port bug 658738 comment 102 (split browser_394759.js into three tests)
[Checked in: Comments 22 and 25]

http://hg.mozilla.org/comm-central/rev/5bdb6dfb3e00


[Approval Request Comment]
Just to go along the rest of this bug.

	Serge Gautherie (:sgautherie) Assignee
	Comment 23 • 13 years ago

Let's file blocking bugs if more work is needed.

	Justin Wood (:Callek)
	Comment 24 • 13 years ago

Comment on attachment 605052 [details] [diff] [review]
(Dv1) Port bug 658738 comment 102 (split browser_394759.js into three tests)
[Checked in: Comments 22 and 25]

Beta is already in release repo, and while I could still take final build with a beta-approved patch tonight, I'm not taking this.

	Jens Hatlak (:InvisibleSmiley)
	Comment 25 • 13 years ago

Comment on attachment 605052 [details] [diff] [review]
(Dv1) Port bug 658738 comment 102 (split browser_394759.js into three tests)
[Checked in: Comments 22 and 25]

http://hg.mozilla.org/releases/comm-aurora/rev/2e0666f06d6b

	Serge Gautherie (:sgautherie) Assignee
	Comment 26 • 13 years ago

(In reply to Justin Wood (:Callek) from comment #24)
> (Dv1) Port bug 658738 comment 102 (split browser_394759.js into three tests)
> 
> Beta is already in release repo, and while I could still take final build
> with a beta-approved patch tonight, I'm not taking this.

Agreed: that request had stalled for too long and bug 741070 isn't fixed yet.