From 458ff5c41f5f4c3acdbfc6eceade959c66cc01c0 Mon Sep 17 00:00:00 2001 From: Cheng Zhao Date: Tue, 19 Nov 2013 12:29:34 +0800 Subject: [PATCH] Always use atom-shell as product name. Since the symbols of atom-shell would be used by all third party applications, it would make server handling more complicated if we allow users to custom the uploaded product name. We should always use atom-shell as product name when uploading so the server would not be confused when searching symbols, and the user specified product name could be show in UI. --- common/crash_reporter/crash_reporter_mac.mm | 2 +- spec/api/crash-reporter.coffee | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/common/crash_reporter/crash_reporter_mac.mm b/common/crash_reporter/crash_reporter_mac.mm index 47b399d591..72ae8c913f 100644 --- a/common/crash_reporter/crash_reporter_mac.mm +++ b/common/crash_reporter/crash_reporter_mac.mm @@ -34,7 +34,7 @@ void CrashReporterMac::InitBreakpad(const std::string& product_name, NSMutableDictionary* parameters = [NSMutableDictionary dictionaryWithCapacity:4]; - [parameters setValue:base::SysUTF8ToNSString(product_name) + [parameters setValue:@"atom-shell" forKey:@BREAKPAD_PRODUCT]; [parameters setValue:base::SysUTF8ToNSString(product_name) forKey:@BREAKPAD_PRODUCT_DISPLAY]; diff --git a/spec/api/crash-reporter.coffee b/spec/api/crash-reporter.coffee index b612356dd2..0b5d1450d0 100644 --- a/spec/api/crash-reporter.coffee +++ b/spec/api/crash-reporter.coffee @@ -13,7 +13,7 @@ describe 'crash-reporter module', -> server = http.createServer (req, res) -> form = new formidable.IncomingForm() form.parse req, (error, fields, files) -> - assert.equal fields['prod'], 'Atom-Shell' + assert.equal fields['prod'], 'atom-shell' assert.equal fields['ver'], process.versions['atom-shell'] assert.equal fields['process_type'], 'renderer' assert.equal fields['platform'], process.platform