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

Unified Diff: chrome/browser/sessions/session_service.cc

Issue 8873021: (Base)SessionService: Remove dead code which was only used by the unit tests. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Rebased. Created 9 years 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
« no previous file with comments | « chrome/browser/sessions/session_service.h ('k') | chrome/browser/sessions/session_service_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sessions/session_service.cc
diff --git a/chrome/browser/sessions/session_service.cc b/chrome/browser/sessions/session_service.cc
index 0dda61c1c432dd03f77f5ddc05a0e049c8fcf471..63ba56ac9df62999ea0b626adc94c6d272554238 100644
--- a/chrome/browser/sessions/session_service.cc
+++ b/chrome/browser/sessions/session_service.cc
@@ -84,7 +84,7 @@ class InternalSessionRequest
real_callback(real_callback) {
}
- // The callback supplied to GetLastSession and GetCurrentSession.
+ // The callback supplied to GetLastSession.
SessionService::SessionCallback real_callback;
private:
@@ -424,35 +424,6 @@ SessionService::Handle SessionService::GetLastSession(
consumer);
}
-SessionService::Handle SessionService::GetCurrentSession(
- CancelableRequestConsumerBase* consumer,
- const SessionCallback& callback) {
- if (pending_window_close_ids_.empty()) {
- // If there are no pending window closes, we can get the current session
- // from memory.
- scoped_refptr<InternalSessionRequest> request(new InternalSessionRequest(
- base::Bind(&SessionService::OnGotSessionCommands,
- base::Unretained(this)),
- callback));
- AddRequest(request, consumer);
- IdToRange tab_to_available_range;
- std::set<SessionID::id_type> windows_to_track;
- BuildCommandsFromBrowsers(&(request->commands),
- &tab_to_available_range,
- &windows_to_track);
- request->ForwardResult(request->handle(), request);
- return request->handle();
- } else {
- // If there are pending window closes, read the current session from disk.
- return ScheduleGetCurrentSessionCommands(
- new InternalSessionRequest(
- base::Bind(&SessionService::OnGotSessionCommands,
- base::Unretained(this)),
- callback),
- consumer);
- }
-}
-
void SessionService::Save() {
bool had_commands = !pending_commands().empty();
BaseSessionService::Save();
« no previous file with comments | « chrome/browser/sessions/session_service.h ('k') | chrome/browser/sessions/session_service_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698