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

Unified Diff: third_party/WebKit/Source/platform/network/ResourceResponse.h

Issue 2674953003: Only generate suggested filenames when actually dragging an image. (Closed)
Patch Set: rebase Created 3 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/network/ResourceResponse.h
diff --git a/third_party/WebKit/Source/platform/network/ResourceResponse.h b/third_party/WebKit/Source/platform/network/ResourceResponse.h
index bb0f21a27e4330d8d2f32babff8629a352219ec5..fe72dc832e55a0f32c44fbd6469dcde53bc6f261 100644
--- a/third_party/WebKit/Source/platform/network/ResourceResponse.h
+++ b/third_party/WebKit/Source/platform/network/ResourceResponse.h
@@ -138,8 +138,7 @@ class PLATFORM_EXPORT ResourceResponse final {
ResourceResponse(const KURL&,
const AtomicString& mimeType,
long long expectedLength,
- const AtomicString& textEncodingName,
- const String& filename);
+ const AtomicString& textEncodingName);
ResourceResponse(const ResourceResponse&);
ResourceResponse& operator=(const ResourceResponse&);
@@ -173,13 +172,6 @@ class PLATFORM_EXPORT ResourceResponse final {
const AtomicString& textEncodingName() const;
void setTextEncodingName(const AtomicString&);
- // FIXME: Should compute this on the fly.
- // There should not be a setter exposed, as suggested file name is determined
- // based on other headers in a manner that WebCore does not necessarily know
- // about.
- const String& suggestedFilename() const;
- void setSuggestedFilename(const String&);
-
int httpStatusCode() const;
void setHTTPStatusCode(int);
@@ -397,7 +389,6 @@ class PLATFORM_EXPORT ResourceResponse final {
AtomicString m_mimeType;
long long m_expectedContentLength;
AtomicString m_textEncodingName;
- String m_suggestedFilename;
int m_httpStatusCode;
AtomicString m_httpStatusText;
HTTPHeaderMap m_httpHeaderFields;
@@ -545,7 +536,6 @@ struct CrossThreadResourceResponseData {
String m_mimeType;
long long m_expectedContentLength;
String m_textEncodingName;
- String m_suggestedFilename;
int m_httpStatusCode;
String m_httpStatusText;
std::unique_ptr<CrossThreadHTTPHeaderMapData> m_httpHeaders;

Powered by Google App Engine
This is Rietveld 408576698