Closed
Bug 1472426
Opened 7 years ago
Closed 7 years ago
[FIX] HTMLFormElement.cpp should probably use IsInComposedDoc
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla63
| Tracking | Status | |
|---|---|---|
| firefox63 | --- | fixed |
People
(Reporter: smaug, Assigned: smaug)
References
Details
Attachments
(2 files)
|
6.42 KB,
patch
|
baku
:
review+
|
Details | Diff | Splinter Review |
|
6.31 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
|
||
Updated•7 years ago
|
Priority: -- → P3
|
Assignee | |
Updated•7 years ago
|
Assignee: nobody → bugs
|
|
Assignee | |
Comment 1•7 years ago
|
||
ended up writing a new simple test using the same backend which test_formSubmission.html has.
In the code there is just one IsInUncomposedDoc -> IsInComposedDoc change.
remote: View your change here:
remote: https://hg.mozilla.org/try/rev/123eab83f4d03abce0cbbd2c211f1c6925108b66
remote:
remote: Follow the progress of your build on Treeherder:
remote: https://treeherder.mozilla.org/#/jobs?repo=try&revision=123eab83f4d03abce0cbbd2c211f1c6925108b66
remote: recorded changegroup in replication log in 0.097s
Attachment #8989730 -
Flags: review?(amarchesini)
|
|
Assignee | |
Updated•7 years ago
|
Summary: HTMLFormElement.cpp should probably use IsInComposedDoc → [FIX] HTMLFormElement.cpp should probably use IsInComposedDoc
|
||
Comment 2•7 years ago
|
||
Comment on attachment 8989730 [details] [diff] [review]
formcontrol_isincomposedoc.diff
Review of attachment 8989730 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/html/test/test_bug1472426.html
@@ +28,5 @@
> + .replace(/\r/g, " ")
> + .replace(/\n/g, " ")
> + .replace(/\"/g, "\\\"");
> + }
> +
extra space here and elsewhere in this file.
@@ +117,5 @@
> + </template>
> +</head>
> +<body>
> +<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1472426">Mozilla Bug 1472426</a>
> +<p id="display"></p>
remove this <p>, the following <div> and the following <pre>
Attachment #8989730 -
Flags: review?(amarchesini) → review+
|
|
Assignee | |
Comment 3•7 years ago
|
||
FWIW, I just use the template generator when creating test files, and it adds all sorts of stuff.
|
|
Assignee | |
Comment 4•7 years ago
|
||
And when using it, it isn't clear what elements are needed by mochitest framework.
|
|
Assignee | |
Comment 5•7 years ago
|
||
Pushed by opettay@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f60f27d4d133
form submission should work in shadow DOM, r=baku
|
|
||
Comment 7•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
|
||
Updated•7 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•