Closed
Bug 1279148
Opened 8 years ago
Closed 8 years ago
Clarify non-nullness of nsHeaderVisitor::{mXHR,mHttpChannel}
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla50
Tracking | Status | |
---|---|---|
firefox50 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: n.nethercote)
Details
(Whiteboard: btpp-active)
Attachments
(1 file)
9.55 KB,
patch
|
baku
:
review+
|
Details | Diff | Splinter Review |
nsHeaderVisitor has two pointer members that are guaranteed to be non-null. We can use different types (references and mozilla::NotNull) to make this more obvious.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8761476 -
Flags: review?(amarchesini)
Updated•8 years ago
|
Attachment #8761476 -
Flags: review?(amarchesini) → review+
Assignee | ||
Comment 2•8 years ago
|
||
Thank you for the fast review.
Updated•8 years ago
|
Whiteboard: btpp-active
Assignee | ||
Comment 3•8 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/759cd5d6285b29e10fb121db1e831c558158f3be Bug 1279148 - Clarify non-nullness of nsHeaderVisitor::{mXHR,mHttpChannel}. r=baku.
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/759cd5d6285b
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox50:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•