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

Unified Diff: chrome/browser/download/download_test_file_activity_observer.h

Issue 12850002: Move download filename determintion into a separate class. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 7 years, 8 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: chrome/browser/download/download_test_file_activity_observer.h
diff --git a/chrome/browser/download/download_test_file_activity_observer.h b/chrome/browser/download/download_test_file_activity_observer.h
index 736b9fd005ce26219a8d63b898f90a499dc655eb..760282c8d8e7042214b46e821d7d4ede2b92593f 100644
--- a/chrome/browser/download/download_test_file_activity_observer.h
+++ b/chrome/browser/download/download_test_file_activity_observer.h
@@ -5,7 +5,7 @@
#ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_TEST_FILE_ACTIVITY_OBSERVER_H_
#define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_TEST_FILE_ACTIVITY_OBSERVER_H_
-#include "base/memory/ref_counted.h"
+#include "base/memory/weak_ptr.h"
class Profile;
@@ -32,9 +32,9 @@ class DownloadTestFileActivityObserver {
bool TestAndResetDidShowFileChooser();
private:
- class MockDownloadManagerDelegate;
+ class TestDownloadFilePickerFactory;
- scoped_refptr<MockDownloadManagerDelegate> test_delegate_;
+ base::WeakPtr<TestDownloadFilePickerFactory> test_file_picker_factory_;
};
#endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_TEST_FILE_ACTIVITY_OBSERVER_H_

Powered by Google App Engine
This is Rietveld 408576698