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

Side by Side Diff: content/browser/android/download_controller_android_impl.h

Issue 152683003: Pass the filename obtained from content-disposition header for download (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // This class pairs with DownloadController on Java side to forward requests 5 // This class pairs with DownloadController on Java side to forward requests
6 // for GET downloads to the current DownloadListener. POST downloads are 6 // for GET downloads to the current DownloadListener. POST downloads are
7 // handled on the native side. 7 // handled on the native side.
8 // 8 //
9 // Both classes are Singleton classes. C++ object owns Java object. 9 // Both classes are Singleton classes. C++ object owns Java object.
10 // 10 //
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 // redirection by the server for |original_url_|. 58 // redirection by the server for |original_url_|.
59 GURL url; 59 GURL url;
60 // The original URL before any redirection by the server for this URL. 60 // The original URL before any redirection by the server for this URL.
61 GURL original_url; 61 GURL original_url;
62 int64 total_bytes; 62 int64 total_bytes;
63 std::string content_disposition; 63 std::string content_disposition;
64 std::string original_mime_type; 64 std::string original_mime_type;
65 std::string user_agent; 65 std::string user_agent;
66 std::string cookie; 66 std::string cookie;
67 std::string referer; 67 std::string referer;
68 std::string filename;
boliu 2014/02/07 18:11:47 nothing uses this afaict?
Jinsuk Kim 2014/02/08 02:49:28 Ah right. Saw it wrong. Reverted.
68 69
69 WebContents* web_contents; 70 WebContents* web_contents;
70 // Default copy constructor is used for passing this struct by value. 71 // Default copy constructor is used for passing this struct by value.
71 }; 72 };
72 struct JavaObject; 73 struct JavaObject;
73 friend struct DefaultSingletonTraits<DownloadControllerAndroidImpl>; 74 friend struct DefaultSingletonTraits<DownloadControllerAndroidImpl>;
74 DownloadControllerAndroidImpl(); 75 DownloadControllerAndroidImpl();
75 virtual ~DownloadControllerAndroidImpl(); 76 virtual ~DownloadControllerAndroidImpl();
76 77
77 // DownloadControllerAndroid implementation. 78 // DownloadControllerAndroid implementation.
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 JavaObject* GetJavaObject(); 121 JavaObject* GetJavaObject();
121 122
122 JavaObject* java_object_; 123 JavaObject* java_object_;
123 124
124 DISALLOW_COPY_AND_ASSIGN(DownloadControllerAndroidImpl); 125 DISALLOW_COPY_AND_ASSIGN(DownloadControllerAndroidImpl);
125 }; 126 };
126 127
127 } // namespace content 128 } // namespace content
128 129
129 #endif // CONTENT_BROWSER_ANDROID_DOWNLOAD_CONTROLLER_ANDROID_IMPL_H_ 130 #endif // CONTENT_BROWSER_ANDROID_DOWNLOAD_CONTROLLER_ANDROID_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698