Index: chrome/browser/ui/webui/ntp/foreign_session_handler.cc |
diff --git a/chrome/browser/ui/webui/ntp/foreign_session_handler.cc b/chrome/browser/ui/webui/ntp/foreign_session_handler.cc |
index 4a6bbaed1601a893c36c1a9f65c796fe3fc52ed2..820774badf783ed2ebe88a1e8f3c1d3aba5913fb 100644 |
--- a/chrome/browser/ui/webui/ntp/foreign_session_handler.cc |
+++ b/chrome/browser/ui/webui/ntp/foreign_session_handler.cc |
@@ -99,12 +99,11 @@ void ForeignSessionHandler::HandleGetForeignSessions(const ListValue* args) { |
return; |
} |
- // Note: we don't own the SyncedSessions themselves. |
- if (!associator->GetAllForeignSessions(&sessions)) { |
- LOG(ERROR) << "ForeignSessionHandler failed to get session data from" |
- "SessionModelAssociator."; |
+ if (associator->GetAllForeignSessions(&sessions)) { |
Dan Beam
2012/03/23 21:12:47
no curlies, also should there be a ! in front of t
Patrick Dubroy
2012/03/26 15:16:46
Ummm...yeah. Nice catch. :-) Done.
|
return; |
} |
+ |
+ // Note: we don't own the SyncedSessions themselves. |
int added_count = 0; |
ListValue session_list; |
for (std::vector<const SyncedSession*>::const_iterator i = |