Only generate suggested filenames when actually dragging an image.

third_party/WebKit/Source/platform/network/ResourceResponse.h

 /*
  * Copyright (C) 2006, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 120 matching lines @@
 
   explicit ResourceResponse(CrossThreadResourceResponseData*);
 
   // Gets a copy of the data suitable for passing to another thread.
   std::unique_ptr<CrossThreadResourceResponseData> copyData() const;
 
   ResourceResponse();
   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&);
 
   bool isNull() const { return m_isNull; }
   bool isHTTP() const;
 
   // The URL of the resource. Note that if a service worker responded to the
   // request for this resource, it may have fetched an entirely different URL
   // and responded with that resource. wasFetchedViaServiceWorker() and
   // originalURLViaServiceWorker() can be used to determine whether and how a
@@ skipping 13 matching lines @@
 
   const AtomicString& mimeType() const;
   void setMimeType(const AtomicString&);
 
   long long expectedContentLength() const;
   void setExpectedContentLength(long long);
 
   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);
 
   const AtomicString& httpStatusText() const;
   void setHTTPStatusText(const AtomicString&);
 
   const AtomicString& httpHeaderField(const AtomicString& name) const;
   void setHTTPHeaderField(const AtomicString& name, const AtomicString& value);
   void addHTTPHeaderField(const AtomicString& name, const AtomicString& value);
   void clearHTTPHeaderField(const AtomicString& name);
@@ skipping 197 matching lines @@
   // This method doesn't compare the all members.
   static bool compare(const ResourceResponse&, const ResourceResponse&);
 
  private:
   void updateHeaderParsedState(const AtomicString& name);
 
   KURL m_url;
   AtomicString m_mimeType;
   long long m_expectedContentLength;
   AtomicString m_textEncodingName;
-  String m_suggestedFilename;
   int m_httpStatusCode;
   AtomicString m_httpStatusText;
   HTTPHeaderMap m_httpHeaderFields;
   time_t m_lastModifiedDate;
   bool m_wasCached : 1;
   unsigned m_connectionID;
   bool m_connectionReused : 1;
   RefPtr<ResourceLoadTiming> m_resourceLoadTiming;
   RefPtr<ResourceLoadInfo> m_resourceLoadInfo;
 
@@ skipping 127 matching lines @@
 struct CrossThreadResourceResponseData {
   WTF_MAKE_NONCOPYABLE(CrossThreadResourceResponseData);
   USING_FAST_MALLOC(CrossThreadResourceResponseData);
 
  public:
   CrossThreadResourceResponseData() {}
   KURL m_url;
   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;
   time_t m_lastModifiedDate;
   RefPtr<ResourceLoadTiming> m_resourceLoadTiming;
   bool m_hasMajorCertificateErrors;
   ResourceResponse::SecurityStyle m_securityStyle;
   ResourceResponse::SecurityDetails m_securityDetails;
   // This is |certificate| from SecurityDetails since that structure should
   // use an AtomicString but this temporary structure is sent across threads.
@@ skipping 19 matching lines @@
   long long m_encodedDataLength;
   long long m_encodedBodyLength;
   long long m_decodedBodyLength;
   String m_downloadedFilePath;
   RefPtr<BlobDataHandle> m_downloadedFileHandle;
 };
 
 }  // namespace blink
 
 #endif  // ResourceResponse_h