Closed
Bug 1382548
Opened 7 years ago
Closed 7 years ago
Sending message that cannot be cloned at FormAutofillContent.jsm:161:6
Categories
(Toolkit :: Form Manager, defect)
Toolkit
Form Manager
Tracking
()
VERIFIED
FIXED
mozilla57
People
(Reporter: MattN, Assigned: lchang)
References
()
Details
(Whiteboard: [form autofill:MVP])
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
ralin
:
review+
lizzard
:
approval-mozilla-beta+
|
Details |
Initially we were explicitly avoiding this issue by not sending the element references but then I was told that we no longer needed to avoid this warning and that special-casing was removed. It seems like we're sending XPCOM objects again… > Sending message that cannot be cloned. Are you trying to send an XPCOM object? FormAutofillContent.jsm:161:6 https://dxr.mozilla.org/mozilla-central/rev/1b065ffd8a535a0ad4c39a912af18e948e6a42c1/browser/extensions/formautofill/FormAutofillContent.jsm#161
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → lchang
Status: NEW → ASSIGNED
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8902527 [details] Bug 1382548 - [Form Autofill] Avoid sending element references in messages. https://reviewboard.mozilla.org/r/174124/#review179366 LGTM, thanks. I'm thinking if we can add an unit test in _getRecords using like sinon.matcher to verify whether the element reference is passed?
Attachment #8902527 -
Flags: review?(ralin) → review+
Comment 3•7 years ago
|
||
Ignore what I just said :P It seems unnecessary as that's unlikely be regressed by other patches in the future, plus, even though it doesn't cause any fatal error.
Assignee | ||
Comment 4•7 years ago
|
||
Thanks.
Pushed by lchang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0357cf75ec0c [Form Autofill] Avoid sending element references in messages. r=ralin
Comment 6•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0357cf75ec0c
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Reporter | ||
Updated•7 years ago
|
Status: RESOLVED → VERIFIED
Reporter | ||
Comment 7•7 years ago
|
||
Comment on attachment 8902527 [details] Bug 1382548 - [Form Autofill] Avoid sending element references in messages. Approval Request Comment [Feature/Bug causing the regression]: Not sure of the specific bug but it was a month or so ago [User impact if declined]: Not really user-facing but it will spam Fx/addon (not web) developer consoles when autofill is used [Is this code covered by automated tests?]: Yes but it's harmless in this case. Automated tests would catch cases where it isn't harmless [Has the fix been verified in Nightly?]: Yes [Needs manual test from QE? If yes, steps to reproduce]: No [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No [Why is the change risky/not risky?]: Just cloning an object and deleting the element reference from the clone before sending across the process boundary [String changes made/needed]: None
Attachment #8902527 -
Flags: approval-mozilla-beta?
Comment 8•7 years ago
|
||
Assuming from MattN's request that this affects 56 (since we are launching the feature in 56)
status-firefox56:
--- → affected
Comment 9•7 years ago
|
||
Comment on attachment 8902527 [details] Bug 1382548 - [Form Autofill] Avoid sending element references in messages. Avoid sending pointless log messages, let's uplift. We are hitting some errors with the form autofill uplifts related to the differences in preferences - not sure if this will suffer from the same issue or not but if so it may need rebasing for beta.
Attachment #8902527 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 10•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/577b2db694a7
Comment 11•7 years ago
|
||
(In reply to Matthew N. [:MattN] (huge backlog; PM if requests are blocking you) from comment #7) > [Needs manual test from QE? If yes, steps to reproduce]: No PEr Matthew's assessment, this does not require manual testing. Marking as qe-verify-.
Flags: qe-verify-
You need to log in
before you can comment on or make changes to this bug.
Description
•