Remove 'using namespace mozilla::net' from BackgroundUtils.h

RESOLVED FIXED in Firefox 62

Status

()

enhancement
P3
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: ckerschb, Assigned: baku)

Tracking

unspecified
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

(Whiteboard: [domsecurity-backlog1])

Attachments

(1 attachment, 1 obsolete attachment)

That sort of namespace usage is not ok within headers, we should fix that.
Depends on: 1439713
Priority: -- → P3
Whiteboard: [domsecurity-backlog1]
Assignee

Comment 1

a year ago
Posted patch namespace.patch (obsolete) — Splinter Review
Assignee: nobody → amarchesini
Attachment #8981895 - Flags: review?(kyle)
Assignee

Comment 2

a year ago
Attachment #8981895 - Attachment is obsolete: true
Attachment #8981895 - Flags: review?(kyle)
Attachment #8981938 - Flags: review?(kyle)
Comment on attachment 8981938 [details] [diff] [review]
namespace.patch

Review of attachment 8981938 [details] [diff] [review]:
-----------------------------------------------------------------

::: netwerk/protocol/data/nsDataHandler.cpp
@@ +86,5 @@
>                  return NS_ERROR_OUT_OF_MEMORY;
>              }
>          }
>  
> +        rv = NS_MutateURI(new net::nsSimpleURI::Mutator())

Why is this just net:: instead of mozilla::net, as it is elsewhere in the file? 

This applies here, and pretty much everywhere else where it's net:: instead of mozilla::net:, though I think this is the only file that actually has a mozilla::net reference elsewhere in the file.
Attachment #8981938 - Flags: review?(kyle) → review+

Comment 4

a year ago
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/eb54a3e980ae
Remove 'using namespace mozilla::net' from BackgroundUtils.h, r=qdot

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/eb54a3e980ae
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.