Open
Bug 1234367
Opened 9 years ago
Updated 2 years ago
It's possible to create new <body> elements by editing <head> as HTML
Categories
(DevTools :: Inspector, defect, P3)
DevTools
Inspector
Tracking
(Not tracked)
NEW
People
(Reporter: arni2033, Unassigned)
References
()
Details
(Whiteboard: [btpp-backlog])
Attachments
(1 file)
|
117 bytes,
text/html
|
Details |
>>> My Info: Win7_64, Nightly 46, 32bit, ID 20151221030239
STR:
1. Open page "testcase 1" or "data:" url in the form above
2. Open devtools -> Inspector
3. Right-click <head> in markup view, click "Edit as HTML"
4. Click between "</style>" and "</head>", type "<div>" without quotes. Press Ctrl+Enter to save
5. Repeat Steps 3-4 several times
Result: New <body> is created after Step 4 every time
Expectations: No new <body> elements
Attachment #8700786 -
Attachment description: 2015.12.22 01-18-12.htm → testcase 1 - add [div] after [style] to get new [body] elements.htm
|
||
Comment 2•9 years ago
|
||
Triaging. Filter on CLIMBING SHOES.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Priority: -- → P3
Whiteboard: [btpp-backlog]
I want to solve for this bug. Can you help me on how should I approach for it?
|
|
||
Comment 4•9 years ago
|
||
(In reply to kshitija from comment #3)
> I want to solve for this bug. Can you help me on how should I approach for
> it?
First of all, make sure you go through our contribution guide here: https://developer.mozilla.org/en-US/docs/Tools/Contributing
And once you have the dev environment set up, make sure you are able to make changes, build, and test your changes.
Once that's in place, you'll be able to work on a fix.
The file where the HTML is changed is: /devtools/server/actors/inspector.js and in particular, the method called `setOuterHTML`.
You can browse the code online too using the DXR tool, here is a link to this method: https://dxr.mozilla.org/mozilla-central/source/devtools/server/actors/inspector.js#2581
As you can see, there's a comment in this method about "special cases". I'm guessing that this bug is yet another special case that should be handled somehow in this code.
Brian (bgrins) worked on this feature originally, so he will be able to help you much better than me on this.
I suggest you try and investigate this issue first a little bit and then come back with questions here by setting the "need info" flag to bgrins.
Assignee: nobody → kshitijakarkar
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
|
||
Updated•3 years ago
|
Assignee: kshitijakarkar → nobody
|
|
||
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•