Open Bug 1725850 Opened 3 years ago Updated 1 year ago

Move crashtests under editor/ to editing/crashtests

Categories

(Core :: DOM: Editor, task, P3)

task

Tracking

()

ASSIGNED

People

(Reporter: masayuki, Assigned: masayuki)

References

(Blocks 1 open bug, )

Details

(Keywords: leave-open)

Attachments

(35 files)

48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
48 bytes, text/x-phabricator-request
Details | Review
No description provided.
Severity: -- → N/A
Priority: -- → P3
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Keywords: leave-open

The test used CRLF. Therefore, you may see all lines are changed.

Depends on D124846

Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/232802020e08
part 1: Port editor/libeditor/crashtests/336104.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/3da11343fbf2
part 2: Port editor/libeditor/crashtests/382527-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/d97f699612eb
part 3: Port editor/libeditor/crashtests/382778-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/f744eb441242
part 4: Port editor/libeditor/crashtests/402172-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/b7e917f0c9cc
part 5: Port editor/libeditor/crashtests/407074-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/105b4e292d99
part 6: Port editor/libeditor/crashtests/407079-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/dcd667a9d8e6
part 7: Port editor/libeditor/crashtests/407256-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/0b7cd5022143
part 8: Port editor/libeditor/crashtests/407277-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/1cbe007d5c33
part 9: Port editor/libeditor/crashtests/414178-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/044710dfd5d1
part 10: Port editor/libeditor/crashtests/418923-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/30545 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot

The test requires to support allowing nested execCommand if we want to run
completely. From point of view of the purose of crashtests, we should allow
it for testing in more complicated situation. Therefore, this patch allows
it while the test is running by the pref.

I think that the crashtests marked as needs-focus should wait focus event
in the document. I think that focus event is fired only when the window has
focus (*1) and Element.focus() steals focus unless chrome UI has focus (*2).
If the testing document hasn't get focus yet, focus event will be dispatched
later (*3).

1: https://searchfox.org/mozilla-central/rev/d8d7539593d5a3071841289dee72f971650073ce/dom/base/nsFocusManager.cpp#1672-1681
2: https://searchfox.org/mozilla-central/rev/d8d7539593d5a3071841289dee72f971650073ce/dom/base/nsFocusManager.cpp#1683-1691
3: https://searchfox.org/mozilla-central/rev/d8d7539593d5a3071841289dee72f971650073ce/dom/base/nsFocusManager.cpp#759-777

Depends on D125920

Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/4b96fdb4c8f7
part 11: Port editor/libeditor/crashtests/420439.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/48943ed3ffd0
part 12: Port editor/libeditor/crashtests/430624-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/68f6d9d84d82
part 13: Port editor/libeditor/crashtests/448329-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/b14b5f1f3a98
part 14: Port editor/libeditor/crashtests/448329-2.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/5e31e98f09af
part 15: Port editor/libeditor/crashtests/448329-3.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/8178c73014a9
part 16: Port editor/libeditor/crashtests/456727-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/1c196e74bfc9
part 17: Port editor/libeditor/crashtests/456727-2.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/9de37f824fef
part 18: Port editor/libeditor/crashtests/467647-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31009 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/7765e08df4fa
part 19: Port editor/libeditor/crashtests/459613.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/b52d6f61c208
part 20: Port editor/libeditor/crashtests/499044-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/8fa7d9e2a4c9
part 21: Port editor/libeditor/crashtests/612565-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/b849c1dd7585
part 22: Port editor/libeditor/crashtests/615015-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/057de5e308fe
part 23: Port editor/libeditor/crashtests/615450-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/16d0c8cdc167
part 24: Port editor/libeditor/crashtests/636074-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31129 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/97e71464e24e
part 25: Port editor/libeditor/crashtests/643706-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/f23111778411
part 26: Port editor/libeditor/crashtests/650572-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/113669447639
part 27: Port editor/libeditor/crashtests/667321-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/6e9d68ba9c21
part 28: Port editor/libeditor/crashtests/682650-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/d209f3ee4d5b
part 29: Port editor/libeditor/crashtests/713427-1.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/c974b2b8004c
part 30: Port editor/libeditor/crashtests/716456-1.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/31435 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/3cd7cabd4459
part 31: Port editor/libeditor/crashtests/745494.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/beba43f4ce5b
part 32: Port editor/libeditor/crashtests/759748.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/a0069e962af5
part 33: Port editor/libeditor/crashtests/761861.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/8e47246b16bd
part 34: Port editor/libeditor/crashtests/762103.html to WPT r=saschanaz
https://hg.mozilla.org/integration/autoland/rev/428f37efeef6
part 35: Port editor/libeditor/crashtests/766305.html to WPT r=saschanaz
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/33260 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot

The leave-open keyword is there and there is no activity for 6 months.
:hsivonen, maybe it's time to close this bug?
For more information, please visit auto_nag documentation.

Flags: needinfo?(hsivonen)

No, we should keep moving them to WPT at least for *.html.

Flags: needinfo?(hsivonen)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: