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

Side by Side Diff: content/browser/download/download_manager_impl_unittest.cc

Issue 209613002: Download shelf autohides on showing in shell, just same as regular open Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Move the 'user acted' flag into DownloadItemModelData and get rid of SetOpened/SetShown in Download… Created 6 years, 6 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 #include <set> 5 #include <set>
6 #include <string> 6 #include <string>
7 7
8 #include "base/bind.h" 8 #include "base/bind.h"
9 #include "base/files/scoped_temp_dir.h" 9 #include "base/files/scoped_temp_dir.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 142 matching lines...) Expand 10 before | Expand all | Expand 10 after
153 MOCK_METHOD1(SetOpenWhenComplete, void(bool)); 153 MOCK_METHOD1(SetOpenWhenComplete, void(bool));
154 MOCK_CONST_METHOD0(GetFileExternallyRemoved, bool()); 154 MOCK_CONST_METHOD0(GetFileExternallyRemoved, bool());
155 MOCK_CONST_METHOD0(GetDangerType, DownloadDangerType()); 155 MOCK_CONST_METHOD0(GetDangerType, DownloadDangerType());
156 MOCK_CONST_METHOD0(IsDangerous, bool()); 156 MOCK_CONST_METHOD0(IsDangerous, bool());
157 MOCK_METHOD0(GetAutoOpened, bool()); 157 MOCK_METHOD0(GetAutoOpened, bool());
158 MOCK_CONST_METHOD0(GetForcedFilePath, const base::FilePath&()); 158 MOCK_CONST_METHOD0(GetForcedFilePath, const base::FilePath&());
159 MOCK_CONST_METHOD0(HasUserGesture, bool()); 159 MOCK_CONST_METHOD0(HasUserGesture, bool());
160 MOCK_CONST_METHOD0(GetTransitionType, PageTransition()); 160 MOCK_CONST_METHOD0(GetTransitionType, PageTransition());
161 MOCK_CONST_METHOD0(IsTemporary, bool()); 161 MOCK_CONST_METHOD0(IsTemporary, bool());
162 MOCK_METHOD1(SetIsTemporary, void(bool)); 162 MOCK_METHOD1(SetIsTemporary, void(bool));
163 MOCK_METHOD1(SetOpened, void(bool));
164 MOCK_CONST_METHOD0(GetOpened, bool()); 163 MOCK_CONST_METHOD0(GetOpened, bool());
165 MOCK_CONST_METHOD0(GetLastModifiedTime, const std::string&()); 164 MOCK_CONST_METHOD0(GetLastModifiedTime, const std::string&());
166 MOCK_CONST_METHOD0(GetETag, const std::string&()); 165 MOCK_CONST_METHOD0(GetETag, const std::string&());
167 MOCK_CONST_METHOD0(GetLastReason, DownloadInterruptReason()); 166 MOCK_CONST_METHOD0(GetLastReason, DownloadInterruptReason());
168 MOCK_CONST_METHOD0(GetBrowserContext, BrowserContext*()); 167 MOCK_CONST_METHOD0(GetBrowserContext, BrowserContext*());
169 MOCK_CONST_METHOD0(GetWebContents, WebContents*()); 168 MOCK_CONST_METHOD0(GetWebContents, WebContents*());
170 MOCK_CONST_METHOD0(GetFileNameToReportUser, base::FilePath()); 169 MOCK_CONST_METHOD0(GetFileNameToReportUser, base::FilePath());
171 MOCK_METHOD1(SetDisplayName, void(const base::FilePath&)); 170 MOCK_METHOD1(SetDisplayName, void(const base::FilePath&));
172 MOCK_METHOD0(NotifyRemoved, void()); 171 MOCK_METHOD0(NotifyRemoved, void());
173 // May be called when vlog is on. 172 // May be called when vlog is on.
(...skipping 522 matching lines...) Expand 10 before | Expand all | Expand 10 after
696 .WillOnce(Return()); 695 .WillOnce(Return());
697 EXPECT_CALL(GetMockDownloadItem(3), Remove()) 696 EXPECT_CALL(GetMockDownloadItem(3), Remove())
698 .Times(0); 697 .Times(0);
699 698
700 download_manager_->RemoveAllDownloads(); 699 download_manager_->RemoveAllDownloads();
701 // Because we're mocking the download item, the Remove call doesn't 700 // Because we're mocking the download item, the Remove call doesn't
702 // result in them being removed from the DownloadManager list. 701 // result in them being removed from the DownloadManager list.
703 } 702 }
704 703
705 } // namespace content 704 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698