593884 - Port to SeaMonkey changes to test_contextmenu.html from bug 558072 Allow getting of popup position and size from popupshowing event

Status: RESOLVED FIXED

(SeaMonkey :: Testing Infrastructure, defect)

Description

[Ian Neal]

Attachment: Port changes to SM patch v0.1 [Checked in: Comment 4]

Bug 558072 introduced a small changed to test_contextmenu.html, this should be ported to SM's version of this test.
We don't use Browser:Stop so using context-stop instead.

Comment 1

[[e-mail]]

Is this actually causing a test failure? If it is, does changing line 601 to SimpleText.executeSoon(startTest) help? (I unfortunately don't have an up-to-date build with tests, since my VM is out of disk space.)

Comment 2

[Ian Neal]

(In reply to comment #1)
> Is this actually causing a test failure? If it is, does changing line 601 to
> SimpleText.executeSoon(startTest) help? (I unfortunately don't have an
> up-to-date build with tests, since my VM is out of disk space.)

It is to fix a potential issue with the new core code that went in with the changes from bug 558072. I presume Neil Deakin did consider that option.

Comment 3

[[e-mail]]

Comment on attachment 472495 [details] [diff] [review]
Port changes to SM patch v0.1 [Checked in: Comment 4]

OK, so if it's not causing a test failure now, can you put in an isnot test too, referencing this bug, so that when it does start failing then at least we're expecting it (and then maybe we can try my other workaround too)? Thanks.

Comment 4

[Ian Neal]

Comment on attachment 472495 [details] [diff] [review]
Port changes to SM patch v0.1 [Checked in: Comment 4]

Checked in with isnot line:
http://hg.mozilla.org/comm-central/rev/fd03993430e4

Comment 5

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

|isnot(false, "Document still loading");| is never going to fail, is it?