Closed
Bug 585630
Opened 13 years ago
Closed 13 years ago
disable mochitests that utilize ondragstart attributes at least until tiles die
Categories
(Firefox for Android Graveyard :: General, defect, P1)
Tracking
(fennec-)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fennec | - | --- |
People
(Reporter: jmaher, Unassigned)
References
Details
Attachments
(1 file)
1.01 KB,
patch
|
mfinkle
:
review+
|
Details | Diff | Splinter Review |
in looking at content/events/test/test_dragstart.html, I found that the test was not executing the code in ondragstart. In Fennec, we don't explicitly disable this, but the mouse events are not sent down to the browser. For now, we will disable the tests that are depending on this via makefile conditions. It was mentioned that once tiles disappear we might want to revisit this.
Reporter | ||
Comment 1•13 years ago
|
||
possible tests that I see in the tree: content/events/test/test_dragstart.html (will disable) content/base/test/test_htmlcopyencoder.xhtml content/base/test/test_bug424359-2.html content/base/test/test_htmlcopyencoder.html content/base/test/test_bug422403-2.xhtml toolkit/content/tests/widgets/test_mousecapture.xul dom/tests/mochitest/whatwg/test_bug477323.html testing/mochitest/tests/SimpleTest/tests/test_synthesizeDragStart.xul (chrome and I don't think this runs as the test listings ignore it)
Reporter | ||
Updated•13 years ago
|
Summary: disable mochitests that utility ondragstart attributes at least until tiles die → disable mochitests that utilize ondragstart attributes at least until tiles die
Reporter | ||
Comment 2•13 years ago
|
||
patch to disable the test_dragstart.html. I would like to leave this bug open so we can revisit it in the future.
Attachment #464098 -
Flags: review?(mark.finkle)
Comment 3•13 years ago
|
||
Comment on attachment 464098 [details] [diff] [review] disable test_dragstart.html for fennec We should definitely try to get this test working again when we kill off the tile manager.
Attachment #464098 -
Flags: review?(mark.finkle) → review+
Reporter | ||
Comment 4•13 years ago
|
||
mfinkle, can you land this?
Updated•13 years ago
|
tracking-fennec: --- → ?
Updated•13 years ago
|
tracking-fennec: ? → 2.0-
Priority: -- → P1
Reporter | ||
Comment 5•13 years ago
|
||
this appears to be landed, should we close this bug?
Comment 6•13 years ago
|
||
(In reply to comment #5) > this appears to be landed, should we close this bug? Can you find the hg checkin when you close this bug?
Reporter | ||
Comment 7•13 years ago
|
||
Landed August 9th 2010: http://hg.mozilla.org/mozilla-central/rev/ea7b16ce2f1e
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•