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

Side by Side Diff: chrome/browser/ui/webui/crashes_ui.cc

Issue 26536002: chrome://crashes: add a link on CrOS for triggering crash uploading (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: tweak handling of dummy dbus client Created 6 years, 9 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 | Annotate | Revision Log
« no previous file with comments | « chrome/browser/resources/crashes.js ('k') | chromeos/dbus/debug_daemon_client.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "chrome/browser/ui/webui/crashes_ui.h" 5 #include "chrome/browser/ui/webui/crashes_ui.h"
6 6
7 #include <vector> 7 #include <vector>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/bind_helpers.h" 10 #include "base/bind_helpers.h"
(...skipping 10 matching lines...) Expand all
21 #include "content/public/browser/web_ui.h" 21 #include "content/public/browser/web_ui.h"
22 #include "content/public/browser/web_ui_data_source.h" 22 #include "content/public/browser/web_ui_data_source.h"
23 #include "content/public/browser/web_ui_message_handler.h" 23 #include "content/public/browser/web_ui_message_handler.h"
24 #include "grit/browser_resources.h" 24 #include "grit/browser_resources.h"
25 #include "grit/chromium_strings.h" 25 #include "grit/chromium_strings.h"
26 #include "grit/generated_resources.h" 26 #include "grit/generated_resources.h"
27 #include "grit/theme_resources.h" 27 #include "grit/theme_resources.h"
28 #include "ui/base/l10n/l10n_util.h" 28 #include "ui/base/l10n/l10n_util.h"
29 #include "ui/base/resource/resource_bundle.h" 29 #include "ui/base/resource/resource_bundle.h"
30 30
31 #if defined(OS_CHROMEOS)
32 #include "chromeos/dbus/dbus_thread_manager.h"
33 #include "chromeos/dbus/debug_daemon_client.h"
34 #endif
35
31 using content::WebContents; 36 using content::WebContents;
32 using content::WebUIMessageHandler; 37 using content::WebUIMessageHandler;
33 38
34 namespace { 39 namespace {
35 40
36 content::WebUIDataSource* CreateCrashesUIHTMLSource() { 41 content::WebUIDataSource* CreateCrashesUIHTMLSource() {
37 content::WebUIDataSource* source = 42 content::WebUIDataSource* source =
38 content::WebUIDataSource::Create(chrome::kChromeUICrashesHost); 43 content::WebUIDataSource::Create(chrome::kChromeUICrashesHost);
39 source->SetUseJsonJSFormatV2(); 44 source->SetUseJsonJSFormatV2();
40 45
41 source->AddLocalizedString("crashesTitle", IDS_CRASHES_TITLE); 46 source->AddLocalizedString("crashesTitle", IDS_CRASHES_TITLE);
42 source->AddLocalizedString("crashCountFormat", 47 source->AddLocalizedString("crashCountFormat",
43 IDS_CRASHES_CRASH_COUNT_BANNER_FORMAT); 48 IDS_CRASHES_CRASH_COUNT_BANNER_FORMAT);
44 source->AddLocalizedString("crashHeaderFormat", 49 source->AddLocalizedString("crashHeaderFormat",
45 IDS_CRASHES_CRASH_HEADER_FORMAT); 50 IDS_CRASHES_CRASH_HEADER_FORMAT);
46 source->AddLocalizedString("crashTimeFormat", IDS_CRASHES_CRASH_TIME_FORMAT); 51 source->AddLocalizedString("crashTimeFormat", IDS_CRASHES_CRASH_TIME_FORMAT);
47 source->AddLocalizedString("bugLinkText", IDS_CRASHES_BUG_LINK_LABEL); 52 source->AddLocalizedString("bugLinkText", IDS_CRASHES_BUG_LINK_LABEL);
48 source->AddLocalizedString("noCrashesMessage", 53 source->AddLocalizedString("noCrashesMessage",
49 IDS_CRASHES_NO_CRASHES_MESSAGE); 54 IDS_CRASHES_NO_CRASHES_MESSAGE);
50 source->AddLocalizedString("disabledHeader", IDS_CRASHES_DISABLED_HEADER); 55 source->AddLocalizedString("disabledHeader", IDS_CRASHES_DISABLED_HEADER);
51 source->AddLocalizedString("disabledMessage", IDS_CRASHES_DISABLED_MESSAGE); 56 source->AddLocalizedString("disabledMessage", IDS_CRASHES_DISABLED_MESSAGE);
57 source->AddLocalizedString("uploadCrashesLinkText",
58 IDS_CRASHES_UPLOAD_MESSAGE);
52 source->SetJsonPath("strings.js"); 59 source->SetJsonPath("strings.js");
53 source->AddResourcePath("crashes.js", IDR_CRASHES_JS); 60 source->AddResourcePath("crashes.js", IDR_CRASHES_JS);
54 source->SetDefaultResource(IDR_CRASHES_HTML); 61 source->SetDefaultResource(IDR_CRASHES_HTML);
55 return source; 62 return source;
56 } 63 }
57 64
58 //////////////////////////////////////////////////////////////////////////////// 65 ////////////////////////////////////////////////////////////////////////////////
59 // 66 //
60 // CrashesDOMHandler 67 // CrashesDOMHandler
61 // 68 //
62 //////////////////////////////////////////////////////////////////////////////// 69 ////////////////////////////////////////////////////////////////////////////////
63 70
64 // The handler for Javascript messages for the chrome://crashes/ page. 71 // The handler for Javascript messages for the chrome://crashes/ page.
65 class CrashesDOMHandler : public WebUIMessageHandler, 72 class CrashesDOMHandler : public WebUIMessageHandler,
66 public CrashUploadList::Delegate { 73 public CrashUploadList::Delegate {
67 public: 74 public:
68 explicit CrashesDOMHandler(); 75 explicit CrashesDOMHandler();
69 virtual ~CrashesDOMHandler(); 76 virtual ~CrashesDOMHandler();
70 77
71 // WebUIMessageHandler implementation. 78 // WebUIMessageHandler implementation.
72 virtual void RegisterMessages() OVERRIDE; 79 virtual void RegisterMessages() OVERRIDE;
73 80
74 // CrashUploadList::Delegate implemenation. 81 // CrashUploadList::Delegate implemenation.
75 virtual void OnUploadListAvailable() OVERRIDE; 82 virtual void OnUploadListAvailable() OVERRIDE;
76 83
77 private: 84 private:
78 // Asynchronously fetches the list of crashes. Called from JS. 85 // Asynchronously fetches the list of crashes. Called from JS.
79 void HandleRequestCrashes(const base::ListValue* args); 86 void HandleRequestCrashes(const base::ListValue* args);
80 87
88 #if defined(OS_CHROMEOS)
89 // Asynchronously triggers crash uploading. Called from JS.
90 void HandleRequestUploads(const base::ListValue* args);
91 #endif
92
81 // Sends the recent crashes list JS. 93 // Sends the recent crashes list JS.
82 void UpdateUI(); 94 void UpdateUI();
83 95
84 scoped_refptr<CrashUploadList> upload_list_; 96 scoped_refptr<CrashUploadList> upload_list_;
85 bool list_available_; 97 bool list_available_;
86 bool first_load_; 98 bool first_load_;
87 99
88 DISALLOW_COPY_AND_ASSIGN(CrashesDOMHandler); 100 DISALLOW_COPY_AND_ASSIGN(CrashesDOMHandler);
89 }; 101 };
90 102
91 CrashesDOMHandler::CrashesDOMHandler() 103 CrashesDOMHandler::CrashesDOMHandler()
92 : list_available_(false), first_load_(true) { 104 : list_available_(false), first_load_(true) {
93 upload_list_ = CrashUploadList::Create(this); 105 upload_list_ = CrashUploadList::Create(this);
94 } 106 }
95 107
96 CrashesDOMHandler::~CrashesDOMHandler() { 108 CrashesDOMHandler::~CrashesDOMHandler() {
97 upload_list_->ClearDelegate(); 109 upload_list_->ClearDelegate();
98 } 110 }
99 111
100 void CrashesDOMHandler::RegisterMessages() { 112 void CrashesDOMHandler::RegisterMessages() {
101 upload_list_->LoadUploadListAsynchronously(); 113 upload_list_->LoadUploadListAsynchronously();
102 web_ui()->RegisterMessageCallback("requestCrashList", 114 web_ui()->RegisterMessageCallback("requestCrashList",
103 base::Bind(&CrashesDOMHandler::HandleRequestCrashes, 115 base::Bind(&CrashesDOMHandler::HandleRequestCrashes,
104 base::Unretained(this))); 116 base::Unretained(this)));
117
118 #if defined(OS_CHROMEOS)
119 web_ui()->RegisterMessageCallback("requestCrashUpload",
120 base::Bind(&CrashesDOMHandler::HandleRequestUploads,
121 base::Unretained(this)));
122 #endif
105 } 123 }
106 124
107 void CrashesDOMHandler::HandleRequestCrashes(const base::ListValue* args) { 125 void CrashesDOMHandler::HandleRequestCrashes(const base::ListValue* args) {
108 if (first_load_) { 126 if (first_load_) {
109 first_load_ = false; 127 first_load_ = false;
110 if (list_available_) 128 if (list_available_)
111 UpdateUI(); 129 UpdateUI();
112 } else { 130 } else {
113 list_available_ = false; 131 list_available_ = false;
114 upload_list_->LoadUploadListAsynchronously(); 132 upload_list_->LoadUploadListAsynchronously();
115 } 133 }
116 } 134 }
117 135
136 #if defined(OS_CHROMEOS)
137 void CrashesDOMHandler::HandleRequestUploads(const base::ListValue* args) {
138 chromeos::DebugDaemonClient* debugd_client =
139 chromeos::DBusThreadManager::Get()->GetDebugDaemonClient();
140 DCHECK(debugd_client);
141
142 debugd_client->UploadCrashes();
143 }
144 #endif
145
118 void CrashesDOMHandler::OnUploadListAvailable() { 146 void CrashesDOMHandler::OnUploadListAvailable() {
119 list_available_ = true; 147 list_available_ = true;
120 if (!first_load_) 148 if (!first_load_)
121 UpdateUI(); 149 UpdateUI();
122 } 150 }
123 151
124 void CrashesDOMHandler::UpdateUI() { 152 void CrashesDOMHandler::UpdateUI() {
125 bool crash_reporting_enabled = 153 bool crash_reporting_enabled =
126 MetricsServiceHelper::IsCrashReportingEnabled(); 154 MetricsServiceHelper::IsCrashReportingEnabled();
127 base::ListValue crash_list; 155 base::ListValue crash_list;
156 bool system_crash_reporter = false;
157
158 #if defined(OS_CHROMEOS)
159 // Chrome OS has a system crash reporter.
160 system_crash_reporter = true;
161 #endif
162
xiyuan 2014/03/31 20:29:18 nit: keep only one empty line
vapier 2014/03/31 21:53:28 Done.
128 163
129 if (crash_reporting_enabled) { 164 if (crash_reporting_enabled) {
130 std::vector<CrashUploadList::UploadInfo> crashes; 165 std::vector<CrashUploadList::UploadInfo> crashes;
131 upload_list_->GetUploads(50, &crashes); 166 upload_list_->GetUploads(50, &crashes);
132 167
133 for (std::vector<CrashUploadList::UploadInfo>::iterator i = crashes.begin(); 168 for (std::vector<CrashUploadList::UploadInfo>::iterator i = crashes.begin();
134 i != crashes.end(); ++i) { 169 i != crashes.end(); ++i) {
135 base::DictionaryValue* crash = new base::DictionaryValue(); 170 base::DictionaryValue* crash = new base::DictionaryValue();
136 crash->SetString("id", i->id); 171 crash->SetString("id", i->id);
137 crash->SetString("time", base::TimeFormatFriendlyDateAndTime(i->time)); 172 crash->SetString("time", base::TimeFormatFriendlyDateAndTime(i->time));
138 crash_list.Append(crash); 173 crash_list.Append(crash);
139 } 174 }
140 } 175 }
141 176
142 base::FundamentalValue enabled(crash_reporting_enabled); 177 base::FundamentalValue enabled(crash_reporting_enabled);
178 base::FundamentalValue dynamic_backend(system_crash_reporter);
143 179
144 const chrome::VersionInfo version_info; 180 const chrome::VersionInfo version_info;
145 base::StringValue version(version_info.Version()); 181 base::StringValue version(version_info.Version());
146 182
147 web_ui()->CallJavascriptFunction("updateCrashList", enabled, crash_list, 183 web_ui()->CallJavascriptFunction("updateCrashList", enabled, dynamic_backend,
148 version); 184 crash_list, version);
149 } 185 }
150 186
151 } // namespace 187 } // namespace
152 188
153 /////////////////////////////////////////////////////////////////////////////// 189 ///////////////////////////////////////////////////////////////////////////////
154 // 190 //
155 // CrashesUI 191 // CrashesUI
156 // 192 //
157 /////////////////////////////////////////////////////////////////////////////// 193 ///////////////////////////////////////////////////////////////////////////////
158 194
159 CrashesUI::CrashesUI(content::WebUI* web_ui) : WebUIController(web_ui) { 195 CrashesUI::CrashesUI(content::WebUI* web_ui) : WebUIController(web_ui) {
160 web_ui->AddMessageHandler(new CrashesDOMHandler()); 196 web_ui->AddMessageHandler(new CrashesDOMHandler());
161 197
162 // Set up the chrome://crashes/ source. 198 // Set up the chrome://crashes/ source.
163 Profile* profile = Profile::FromWebUI(web_ui); 199 Profile* profile = Profile::FromWebUI(web_ui);
164 content::WebUIDataSource::Add(profile, CreateCrashesUIHTMLSource()); 200 content::WebUIDataSource::Add(profile, CreateCrashesUIHTMLSource());
165 } 201 }
166 202
167 // static 203 // static
168 base::RefCountedMemory* CrashesUI::GetFaviconResourceBytes( 204 base::RefCountedMemory* CrashesUI::GetFaviconResourceBytes(
169 ui::ScaleFactor scale_factor) { 205 ui::ScaleFactor scale_factor) {
170 return ResourceBundle::GetSharedInstance(). 206 return ResourceBundle::GetSharedInstance().
171 LoadDataResourceBytesForScale(IDR_SAD_FAVICON, scale_factor); 207 LoadDataResourceBytesForScale(IDR_SAD_FAVICON, scale_factor);
172 } 208 }
OLDNEW
« no previous file with comments | « chrome/browser/resources/crashes.js ('k') | chromeos/dbus/debug_daemon_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698