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

Side by Side Diff: chrome/browser/feedback/system_logs/log_sources/chrome_internal_log_source.cc

Issue 2354613002: [Sync] Fix namespaces for the browser_sync component. (Closed)
Patch Set: Address comments. Created 4 years, 3 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 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "chrome/browser/feedback/system_logs/log_sources/chrome_internal_log_so urce.h" 5 #include "chrome/browser/feedback/system_logs/log_sources/chrome_internal_log_so urce.h"
6 6
7 #include "base/json/json_string_value_serializer.h" 7 #include "base/json/json_string_value_serializer.h"
8 #include "base/sys_info.h" 8 #include "base/sys_info.h"
9 #include "build/build_config.h" 9 #include "build/build_config.h"
10 #include "chrome/browser/browser_process.h" 10 #include "chrome/browser/browser_process.h"
(...skipping 122 matching lines...) Expand 10 before | Expand all | Expand 10 after
133 #endif // defined(OS_CHROMEOS) 133 #endif // defined(OS_CHROMEOS)
134 } 134 }
135 135
136 void ChromeInternalLogSource::PopulateSyncLogs(SystemLogsResponse* response) { 136 void ChromeInternalLogSource::PopulateSyncLogs(SystemLogsResponse* response) {
137 // We are only interested in sync logs for the primary user profile. 137 // We are only interested in sync logs for the primary user profile.
138 Profile* profile = ProfileManager::GetPrimaryUserProfile(); 138 Profile* profile = ProfileManager::GetPrimaryUserProfile();
139 if (!profile || 139 if (!profile ||
140 !ProfileSyncServiceFactory::GetInstance()->HasProfileSyncService(profile)) 140 !ProfileSyncServiceFactory::GetInstance()->HasProfileSyncService(profile))
141 return; 141 return;
142 142
143 ProfileSyncService* service = 143 browser_sync::ProfileSyncService* service =
144 ProfileSyncServiceFactory::GetInstance()->GetForProfile(profile); 144 ProfileSyncServiceFactory::GetInstance()->GetForProfile(profile);
145 std::unique_ptr<base::DictionaryValue> sync_logs( 145 std::unique_ptr<base::DictionaryValue> sync_logs(
146 sync_driver::sync_ui_util::ConstructAboutInformation( 146 sync_driver::sync_ui_util::ConstructAboutInformation(
147 service, service->signin(), chrome::GetChannel())); 147 service, service->signin(), chrome::GetChannel()));
148 148
149 // Remove identity section. 149 // Remove identity section.
150 base::ListValue* details = NULL; 150 base::ListValue* details = NULL;
151 sync_logs->GetList(sync_driver::sync_ui_util::kDetailsKey, &details); 151 sync_logs->GetList(sync_driver::sync_ui_util::kDetailsKey, &details);
152 if (!details) 152 if (!details)
153 return; 153 return;
(...skipping 79 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 SystemLogsResponse* response) { 233 SystemLogsResponse* response) {
234 std::string reason; 234 std::string reason;
235 bool result = base::win::IsKeyboardPresentOnSlate(&reason); 235 bool result = base::win::IsKeyboardPresentOnSlate(&reason);
236 (*response)[kUsbKeyboardDetected] = result ? "Keyboard Detected:\n" : 236 (*response)[kUsbKeyboardDetected] = result ? "Keyboard Detected:\n" :
237 "No Keyboard:\n"; 237 "No Keyboard:\n";
238 (*response)[kUsbKeyboardDetected] += reason; 238 (*response)[kUsbKeyboardDetected] += reason;
239 } 239 }
240 #endif 240 #endif
241 241
242 } // namespace system_logs 242 } // namespace system_logs
OLDNEW
« no previous file with comments | « chrome/browser/extensions/external_pref_loader.cc ('k') | chrome/browser/history/web_history_service_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698