dom/security/nsContentSecurityManager.cpp has two #include "nsIStreamListener.h"

ASSIGNED
Assigned to

Status

()

P2
normal
ASSIGNED
2 years ago
2 years ago

People

(Reporter: ewong, Assigned: ewong)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
line #4 and line #8 contains #include "nsIStreamListener.h".

according to :jdm on irc, it's not necessary.

filing this bug.
Comment hidden (mozreview-request)
(Assignee)

Updated

2 years ago
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Flags: needinfo?(bzbarsky)
(Assignee)

Updated

2 years ago
Flags: needinfo?(bzbarsky)

Comment 2

2 years ago
mozreview-review
Comment on attachment 8860260 [details]
Bug 1358345 - Remove extra instance of #include 'nsIStreamListener.h'

https://reviewboard.mozilla.org/r/132274/#review135186

Thanks - please update the commit message though!
Attachment #8860260 - Flags: review+
Priority: -- → P2
Whiteboard: [domsecurity-active]
(Assignee)

Comment 3

2 years ago
mozreview-review-reply
Comment on attachment 8860260 [details]
Bug 1358345 - Remove extra instance of #include 'nsIStreamListener.h'

https://reviewboard.mozilla.org/r/132274/#review135186

pardon my ignorance, but what's the mozreview way of updating the commit message?  (more so, what exactly do I update it to?)
I am not really using mozreview, but I would imagine it's the same as a regular commit message:

hg qrefresh -m "Bug 1358345 - Remove extra instance of #include 'nsIStreamListener.h'. r=ckerschb"
Comment hidden (mozreview-request)
You need to log in before you can comment on or make changes to this bug.