Do not append "Renderer" in product name when uploading.
This commit is contained in:
parent
ed3358e501
commit
20e368eb89
4 changed files with 8 additions and 5 deletions
|
@ -19,7 +19,7 @@ CrashReporter::CrashReporter() {
|
|||
CrashReporter::~CrashReporter() {
|
||||
}
|
||||
|
||||
void CrashReporter::Start(std::string product_name,
|
||||
void CrashReporter::Start(const std::string& product_name,
|
||||
const std::string& company_name,
|
||||
const std::string& submit_url,
|
||||
bool auto_submit,
|
||||
|
@ -27,8 +27,6 @@ void CrashReporter::Start(std::string product_name,
|
|||
const StringMap& extra_parameters) {
|
||||
SetUploadParameters(extra_parameters);
|
||||
|
||||
// Append "Renderer" for the renderer.
|
||||
product_name += " Renderer";
|
||||
InitBreakpad(product_name, ATOM_VERSION_STRING, company_name, submit_url,
|
||||
auto_submit, skip_system_crash_handler);
|
||||
}
|
||||
|
|
|
@ -18,7 +18,7 @@ class CrashReporter {
|
|||
|
||||
static CrashReporter* GetInstance();
|
||||
|
||||
void Start(std::string product_name,
|
||||
void Start(const std::string& product_name,
|
||||
const std::string& company_name,
|
||||
const std::string& submit_url,
|
||||
bool auto_submit,
|
||||
|
|
|
@ -28,11 +28,16 @@ void CrashReporterMac::InitBreakpad(const std::string& product_name,
|
|||
if (breakpad_ != NULL)
|
||||
BreakpadRelease(breakpad_);
|
||||
|
||||
std::string display_name = is_browser_ ? product_name :
|
||||
product_name + " Renderer";
|
||||
|
||||
NSMutableDictionary* parameters =
|
||||
[NSMutableDictionary dictionaryWithCapacity:4];
|
||||
|
||||
[parameters setValue:base::SysUTF8ToNSString(product_name)
|
||||
forKey:@BREAKPAD_PRODUCT];
|
||||
[parameters setValue:base::SysUTF8ToNSString(product_name)
|
||||
forKey:@BREAKPAD_PRODUCT_DISPLAY];
|
||||
[parameters setValue:base::SysUTF8ToNSString(version)
|
||||
forKey:@BREAKPAD_VERSION];
|
||||
[parameters setValue:base::SysUTF8ToNSString(company_name)
|
||||
|
|
|
@ -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 Renderer'
|
||||
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
|
||||
|
|
Loading…
Reference in a new issue