Open Bug 1358345 Opened 5 years ago Updated 4 years ago

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

Categories

(Core :: DOM: Security, enhancement, P3)

enhancement

Tracking

()

ASSIGNED

People

(Reporter: ewong, Assigned: ewong)

Details

(Whiteboard: [domsecurity-active])

Attachments

(1 file)

line #4 and line #8 contains #include "nsIStreamListener.h".

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

filing this bug.
Assignee: nobody → ewong
Status: NEW → ASSIGNED
Flags: needinfo?(bzbarsky)
Flags: needinfo?(bzbarsky)
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]
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"
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.