[SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """, since bug 527935 landing

NEW
Unassigned

Status

()

--
major
8 years ago
7 years ago

People

(Reporter: sgautherie, Unassigned)

Tracking

(Blocks: 1 bug, {regression})

1.9.1 Branch
x86
All
regression
Points:
---

Firefox Tracking Flags

(status2.0 unaffected, status1.9.1 ?)

Details

(Whiteboard: [almost perma-o‌range], URL)

Attachments

(1 obsolete attachment)

(Reporter)

Description

8 years ago
Last good:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey2.0/1296705165.1296707956.16725.gz&fulltext=1
Linux comm-1.9.1 test mochitests on 2011/02/02 19:52:45
rev:07c40b67dae9
moz:cf6f4bb351f7
{
88180 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | Checking form1 input - "v" should equal "v"
}
+
Same changesets for OSX and Windows.

First bad:
http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey2.0/1296869668.1296871918.12812.gz
Linux comm-1.9.1 test mochitests on 2011/02/04 17:34:28
rev:07c40b67dae9
moz:27ddaf8a6c89
{
88188 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | Starting test #211
88189 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | This test is privledged.
88190 INFO TEST-PASS | /tests/toolkit/components/satchel/test/test_bug_511615.html | popup should be closed - false should equal false
88191 ERROR TEST-UNEXPECTED-FAIL | /tests/toolkit/components/satchel/test/test_bug_511615.html | Checking form1 input - "v" should equal ""
}
+
Same changesets for OSX and Windows.


Regression timeframe:
http://hg.mozilla.org/releases/mozilla-1.9.1/pushloghtml?fromchange=cf6f4bb351f7&tochange=27ddaf8a6c89
Bug 527935!
"You are not authorized to access bug #527935."
(Reporter)

Updated

8 years ago
Summary: [SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """ → [SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """, since bug 527935 landing

Comment 1

8 years ago
Serge: does this only happen on 1.9.1?

Dolske: can you please look into this?  This looks like a test problem to me, the same kind of stuff you fixed in bug 527935...

Comment 2

8 years ago
Probably something in <http://hg.mozilla.org/releases/mozilla-1.9.1/rev/27ddaf8a6c89#l12.1> is wrong.  I tested it locally in Firefox before landing the patch both on 1.9.2 and 1.9.1, but I'm not sure what SeaMonkey could have different here...
(Reporter)

Comment 3

8 years ago
(In reply to comment #1)
> Serge: does this only happen on 1.9.1?

Yes, no failure report with SeaMonkey 2.1 (m-2.0).
status2.0: --- → unaffected
(Reporter)

Comment 4

8 years ago
With SM 2.0, if I manually press escape after test has ended, the 'v' is removed.

Yet, I kind of wonder why, with (FF 3.x or) SM 2.1, this escape both 1) closes the popup and 2) removes the 'v' ... as I am ++/- unable to reproduce that behavior after the test has ended :-|

Note that, before bug 527935, the test didn't expect the 'v' to be removed.
(Reporter)

Comment 5

8 years ago
Created attachment 519454 [details] [diff] [review]
(Av1) s/privledged/priviledged/g, Add 2 comments
[Moved to bug 705957]

Unrelated, but while there.
Attachment #519454 - Flags: review?(ehsan)
(Reporter)

Comment 6

8 years ago
(Guessing:) Could it be related to some XPFE vs Toolkit autocomplete widget difference?

Comment 7

8 years ago
Probably what happens in XPFE is that ESC closes the popup but is eaten and so the autocomplete doesn't revert the text value.

Comment 8

8 years ago
Having said that, the test passes on my build, so...

Comment 9

8 years ago
Comment on attachment 519454 [details] [diff] [review]
(Av1) s/privledged/priviledged/g, Add 2 comments
[Moved to bug 705957]

r=me only if you land this on mozilla-central too.

And please change the commit message to something like "Bug 641716 - Correct the spelling of doKeyUnprivledged".
Attachment #519454 - Flags: review?(ehsan) → review+

Comment 10

8 years ago
Comment on attachment 519454 [details] [diff] [review]
(Av1) s/privledged/priviledged/g, Add 2 comments
[Moved to bug 705957]

Actually, r-.  Landing this patch as part of this bug will only make things more confusing.  Please file a new bug and attach the patch there.
Attachment #519454 - Flags: review+ → review-

Updated

8 years ago
Attachment #519454 - Attachment is obsolete: true

Updated

8 years ago
Whiteboard: [perma-orange]
(Reporter)

Updated

8 years ago
Whiteboard: [almost perma-orange]

Updated

8 years ago
Whiteboard: [almost perma-orange] → [almost perma-o‌range]
(Reporter)

Comment 11

8 years ago
Bug still there.
(Reporter)

Updated

7 years ago
Attachment #519454 - Attachment description: (Av1) s/privledged/priviledged/g, Add 2 comments → (Av1) s/privledged/priviledged/g, Add 2 comments [Moved to bug 705957]
You need to log in before you can comment on or make changes to this bug.