Attachment 8790731 Details for Bug 1282408

[patch] add ignore initialization check flag for variables in AltSvcMapping

Bug 1282408_4.diff (text/plain), 1.45 KB, created by Andi [:andi]

(hide)

Description:

Filename:

MIME Type:

Creator: Andi [:andi]

Size: 1.45 KB

patch

obsolete

># HG changeset patch
># User Andi-Bogdan Postelnicu <bpostelnicu@mozilla.com>
># Date 1473776640 -10800
>#      Tue Sep 13 17:24:00 2016 +0300
># Node ID 85020ae76f88e9836393ada7be3841563fee6541
># Parent  b1156b0eb96fcb49966b20e5fcf6a01f634ea2ee
>Bug 1282408 - add ignore initialization check flag for variables in AltSvcMapping. r?valentin
>
>MozReview-Commit-ID: GRWqLr1pW3q
>
>diff --git a/netwerk/protocol/http/AlternateServices.h b/netwerk/protocol/http/AlternateServices.h
>--- a/netwerk/protocol/http/AlternateServices.h
>+++ b/netwerk/protocol/http/AlternateServices.h
>@@ -105,21 +105,21 @@ private:
> 
>   // If you change any of these members, update Serialize()
>   nsCString mAlternateHost;
>-  int32_t mAlternatePort;
>+  MOZ_INIT_OUTSIDE_CTOR int32_t mAlternatePort;
> 
>   nsCString mOriginHost;
>-  int32_t mOriginPort;
>+  MOZ_INIT_OUTSIDE_CTOR int32_t mOriginPort;
> 
>   nsCString mUsername;
>-  bool mPrivate;
>-
>-  uint32_t mExpiresAt; // alt-svc mappping
>+  MOZ_INIT_OUTSIDE_CTOR bool mPrivate;
> 
>-  bool mValidated;
>-  bool mHttps; // origin is https://
>-  bool mMixedScheme; // .wk allows http and https on same con
>+  MOZ_INIT_OUTSIDE_CTOR uint32_t mExpiresAt; // alt-svc mappping
> 
>-  nsCString        mNPNToken;
>+  MOZ_INIT_OUTSIDE_CTOR bool mValidated;
>+  MOZ_INIT_OUTSIDE_CTOR bool mHttps; // origin is https://
>+  MOZ_INIT_OUTSIDE_CTOR bool mMixedScheme; // .wk allows http and https on same con
>+
>+  nsCString mNPNToken;
> };
> 
> class AltSvcOverride : public nsIInterfaceRequestor

Flags:

mcmanus: review+

Actions: View | Diff | Review

Attachments on bug 1282408: 8765437 | 8765842 | 8765844 | 8765891 | 8765901 | 8766283 | 8766288 | 8774695 | 8774702 | 8774704 | 8774727 | 8774738 | 8775045 | 8775107 | 8775117 | 8775121 | 8775130 | 8775139 | 8775141 | 8775519 | 8775560 | 8775577 | 8775599 | 8783946 | 8783955 | 8783956 | 8783963 | 8786330 | 8786345 | 8787093 | 8790295 | 8790304 | 8790645 | 8790699 | 8790702 | 8790731 | 8791536 | 8791622 | 8791937 | 8792801 | 8794853 | 8794855 | 8795169 | 8795228 | 8795236 | 8795246 | 8795282 | 8795333 | 8795623 | 8795632 | 8795642 | 8795644 | 8795690 | 8796067 | 8796100