Closed
Bug 641716
Opened 14 years ago
Closed 3 years ago
[SeaMonkey 2.0] mochitest-plain: permanent "test_bug_511615.html | Checking form1 input - "v" should equal """, since bug 527935 landing
Categories
(Toolkit :: Form Manager, defect)
Tracking
()
RESOLVED
INCOMPLETE
Tracking | Status | |
---|---|---|
status2.0 | --- | unaffected |
status1.9.1 | --- | ? |
People
(Reporter: sgautherie, Unassigned)
References
(Blocks 1 open bug, )
Details
(Keywords: regression, Whiteboard: [almost perma-orange])
Attachments
(1 obsolete file)
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•14 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•14 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•14 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•14 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•14 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•14 years ago
|
||
Unrelated, but while there.
Attachment #519454 -
Flags: review?(ehsan)
Reporter | ||
Comment 6•14 years ago
|
||
(Guessing:) Could it be related to some XPFE vs Toolkit autocomplete widget difference?
Comment 7•14 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•14 years ago
|
||
Having said that, the test passes on my build, so...
Comment 9•14 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•14 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•14 years ago
|
Attachment #519454 -
Attachment is obsolete: true
Updated•14 years ago
|
Whiteboard: [perma-orange]
Reporter | ||
Updated•14 years ago
|
Whiteboard: [almost perma-orange]
Updated•14 years ago
|
Whiteboard: [almost perma-orange] → [almost perma-orange]
Reporter | ||
Comment 11•14 years ago
|
||
Bug still there.
Reporter | ||
Updated•13 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]
Comment 12•3 years ago
|
||
There haven't been any activity on this bug for more than a decade.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•