Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(49)

Side by Side Diff: third_party/WebKit/Source/core/fetch/Resource.cpp

Issue 1977303003: Adds a feature to MHTML serialization that omits subframes and subresources marked no-store. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@no-store
Patch Set: Address dcheng's comments. Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de) 2 Copyright (C) 1998 Lars Knoll (knoll@mpi-hd.mpg.de)
3 Copyright (C) 2001 Dirk Mueller (mueller@kde.org) 3 Copyright (C) 2001 Dirk Mueller (mueller@kde.org)
4 Copyright (C) 2002 Waldo Bastian (bastian@kde.org) 4 Copyright (C) 2002 Waldo Bastian (bastian@kde.org)
5 Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com) 5 Copyright (C) 2006 Samuel Weinig (sam.weinig@gmail.com)
6 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved. 6 Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
7 7
8 This library is free software; you can redistribute it and/or 8 This library is free software; you can redistribute it and/or
9 modify it under the terms of the GNU Library General Public 9 modify it under the terms of the GNU Library General Public
10 License as published by the Free Software Foundation; either 10 License as published by the Free Software Foundation; either
(...skipping 927 matching lines...) Expand 10 before | Expand all | Expand 10 after
938 { 938 {
939 for (auto& redirect : m_redirectChain) { 939 for (auto& redirect : m_redirectChain) {
940 if (!canUseResponse(redirect.m_redirectResponse, m_responseTimestamp)) 940 if (!canUseResponse(redirect.m_redirectResponse, m_responseTimestamp))
941 return false; 941 return false;
942 if (redirect.m_request.cacheControlContainsNoCache() || redirect.m_reque st.cacheControlContainsNoStore()) 942 if (redirect.m_request.cacheControlContainsNoCache() || redirect.m_reque st.cacheControlContainsNoStore())
943 return false; 943 return false;
944 } 944 }
945 return true; 945 return true;
946 } 946 }
947 947
948 bool Resource::hasCacheControlNoStoreHeader() 948 bool Resource::hasCacheControlNoStoreHeader() const
949 { 949 {
950 return m_response.cacheControlContainsNoStore() || m_resourceRequest.cacheCo ntrolContainsNoStore(); 950 return m_response.cacheControlContainsNoStore() || m_resourceRequest.cacheCo ntrolContainsNoStore();
951 } 951 }
952 952
953 bool Resource::hasVaryHeader() const 953 bool Resource::hasVaryHeader() const
954 { 954 {
955 return !m_response.httpHeaderField(HTTPNames::Vary).isNull(); 955 return !m_response.httpHeaderField(HTTPNames::Vary).isNull();
956 } 956 }
957 957
958 bool Resource::mustRevalidateDueToCacheHeaders() 958 bool Resource::mustRevalidateDueToCacheHeaders()
(...skipping 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
1129 case Resource::Media: 1129 case Resource::Media:
1130 return "Media"; 1130 return "Media";
1131 case Resource::Manifest: 1131 case Resource::Manifest:
1132 return "Manifest"; 1132 return "Manifest";
1133 } 1133 }
1134 ASSERT_NOT_REACHED(); 1134 ASSERT_NOT_REACHED();
1135 return "Unknown"; 1135 return "Unknown";
1136 } 1136 }
1137 1137
1138 } // namespace blink 1138 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698