FetchDriver should check content load policy before proceeding

RESOLVED FIXED in Firefox 39

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: nsm, Assigned: nsm)

Tracking

unspecified
mozilla39
x86_64
Linux
Points:
---

Firefox Tracking Flags

(firefox39 fixed)

Details

Attachments

(1 attachment)

Attachment #8572910 - Attachment is patch: true
Attachment #8572910 - Flags: review?(amarchesini)
Comment on attachment 8572910 [details] [diff] [review]
fix

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

I don't know enough to say yes/no for this LOAD_BACKGROUND thing.

::: dom/fetch/FetchDriver.cpp
@@ +368,5 @@
>                       nsILoadInfo::SEC_NORMAL,
>                       mRequest->ContentPolicyType(),
>                       mLoadGroup,
>                       nullptr, /* aCallbacks */
> +                     nsIRequest::LOAD_BACKGROUND | credentialsFlag,

Why this?

::: dom/fetch/FetchDriver.h
@@ +13,5 @@
>  #include "nsIStreamListener.h"
>  #include "nsRefPtr.h"
>  
>  #include "mozilla/DebugOnly.h"
>  #include "mozilla/net/ReferrerPolicy.h"

should we remove this header?
Attachment #8572910 - Flags: review?(amarchesini) → review+
I removed the LOAD_BACKGROUND change. The header is required since we use DebugOnly<bool>.
https://hg.mozilla.org/mozilla-central/rev/25ded6c22f64
Assignee: nobody → nsm.nikhil
Status: NEW → RESOLVED
Last Resolved: 4 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.