Skip to content

Commit

Permalink
Bug 1147178 - remove InternalResponse::mTerminationReason r=dom-worke…
Browse files Browse the repository at this point in the history
…rs-and-storage-reviewers,sg

The termination reason doesn't exist in the Fetch spec anymore. The member
variable isn't ever used either.

Differential Revision: https://phabricator.services.mozilla.com/D66539

--HG--
extra : moz-landing-system : lando
  • Loading branch information
perryjiang committed Mar 12, 2020
1 parent e304dee commit 0e90b8b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 3 deletions.
2 changes: 0 additions & 2 deletions dom/fetch/InternalResponse.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,6 @@ already_AddRefed<InternalResponse> InternalResponse::OpaqueResponse() {
"Can't OpaqueResponse a already wrapped response");
RefPtr<InternalResponse> response = new InternalResponse(0, EmptyCString());
response->mType = ResponseType::Opaque;
response->mTerminationReason = mTerminationReason;
response->mChannelInfo = mChannelInfo;
if (mPrincipalInfo) {
response->mPrincipalInfo =
Expand All @@ -335,7 +334,6 @@ already_AddRefed<InternalResponse> InternalResponse::OpaqueRedirectResponse() {
already_AddRefed<InternalResponse> InternalResponse::CreateIncompleteCopy() {
RefPtr<InternalResponse> copy = new InternalResponse(mStatus, mStatusText);
copy->mType = mType;
copy->mTerminationReason = mTerminationReason;
copy->mURLList = mURLList;
copy->mChannelInfo = mChannelInfo;
if (mPrincipalInfo) {
Expand Down
1 change: 0 additions & 1 deletion dom/fetch/InternalResponse.h
Original file line number Diff line number Diff line change
Expand Up @@ -331,7 +331,6 @@ class InternalResponse final {
already_AddRefed<InternalResponse> CreateIncompleteCopy();

ResponseType mType;
nsCString mTerminationReason;
// A response has an associated url list (a list of zero or more fetch URLs).
// Unless stated otherwise, it is the empty list. The current url is the last
// element in mURLlist
Expand Down

0 comments on commit 0e90b8b

Please sign in to comment.