Closed
Bug 1304061
Opened 8 years ago
Closed 8 years ago
Remove explicit ns declarations in inspector.xhtml
Categories
(DevTools :: Inspector, defect)
DevTools
Inspector
Tracking
(firefox52 fixed)
RESOLVED
FIXED
Firefox 52
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | fixed |
People
(Reporter: Honza, Assigned: Honza)
Details
Attachments
(1 file, 1 obsolete file)
|
22.14 KB,
patch
|
Honza
:
review+
|
Details | Diff | Splinter Review |
This is follow up for bug 1262443 https://bugzilla.mozilla.org/show_bug.cgi?id=1262443#c15 Remove all the html namespaces in the inspector.xhtml file. <html:div> => <div> Honza
|
Assignee | |
Comment 1•8 years ago
|
||
|
||
Comment 2•8 years ago
|
||
Comment on attachment 8793302 [details] [diff] [review] bug1304061.patch Review of attachment 8793302 [details] [diff] [review]: ----------------------------------------------------------------- Seems mechanical enough for a quick R+. Thanks
Attachment #8793302 -
Flags: review?(pbrosset) → review+
|
|
Assignee | |
Comment 3•8 years ago
|
||
Thanks for the review Patrick. Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=89eed04f71c8 There are some oranges, but unrelated. Honza
Keywords: checkin-needed
|
|
Assignee | |
Comment 5•8 years ago
|
||
Rebasing Honza
Attachment #8793302 -
Attachment is obsolete: true
Attachment #8794069 -
Flags: review+
|
|
Assignee | |
Updated•8 years ago
|
Keywords: checkin-needed
Pushed by cbook@mozilla.com: https://hg.mozilla.org/integration/fx-team/rev/78860a7d085b Remove explicit ns declarations in inspector.xhtml; r=pbro
Keywords: checkin-needed
|
||
Comment 7•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/78860a7d085b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•