Add spec for custom uploading parameters.

diff --git a/common/crash_reporter/crash_reporter.cc b/common/crash_reporter/crash_reporter.cc
index f52f335..c576541 100644 (file)
--- a/common/crash_reporter/crash_reporter.cc
+++ b/common/crash_reporter/crash_reporter.cc
@@ -36,6 +36,9 @@ void CrashReporter::Start(std::string product_name,
 void CrashReporter::SetUploadParameters(const StringMap& parameters) {
   upload_parameters_ = parameters;
   upload_parameters_["process_type"] = is_browser_ ? "browser" : "renderer";
+
+  // Setting platform dependent parameters.
+  SetUploadParameters();
 }
 
 }  // namespace crash_reporter

diff --git a/common/v8_conversions.h b/common/v8_conversions.h
index 378839a..cd50723 100644 (file)
--- a/common/v8_conversions.h
+++ b/common/v8_conversions.h
@@ -53,8 +53,9 @@ struct FromV8Value {
     v8::Handle<v8::Object> v8_dict = value_->ToObject();
     v8::Handle<v8::Array> v8_keys = v8_dict->GetOwnPropertyNames();
     for (uint32_t i = 0; i < v8_keys->Length(); ++i) {
-      std::string key = FromV8Value(v8_keys->Get(i));
-      dict[key] = std::string(FromV8Value(v8_dict->Get(i)));
+      v8::Handle<v8::Value> v8_key = v8_keys->Get(i);
+      std::string key = FromV8Value(v8_key);
+      dict[key] = std::string(FromV8Value(v8_dict->Get(v8_key)));
     }
 
     return dict;

diff --git a/spec/api/crash-reporter.coffee b/spec/api/crash-reporter.coffee
index 415c2a0..86d0a0b 100644 (file)
--- a/spec/api/crash-reporter.coffee
+++ b/spec/api/crash-reporter.coffee
@@ -13,13 +13,17 @@ 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['ver'], process.versions['atom-shell']
         assert.equal fields['process_type'], 'renderer'
+        assert.equal fields['platform'], process.platform
+        assert.equal fields['extra1'], 'extra1'
+        assert.equal fields['extra2'], 'extra2'
         assert.equal files['upload_file_minidump']['name'], 'minidump.dmp'
 
         w.destroy()
         res.end()
         server.close()
         done()
-    server.listen 901007, '127.0.0.1', ->
+    server.listen 1127, '127.0.0.1', ->
       w.loadUrl 'file://' + path.join(fixtures, 'api', 'crash.html')

diff --git a/spec/fixtures/api/crash.html b/spec/fixtures/api/crash.html
index 88063bf..8933288 100644 (file)
--- a/spec/fixtures/api/crash.html
+++ b/spec/fixtures/api/crash.html
@@ -3,9 +3,13 @@
 <script type="text/javascript" charset="utf-8">
 var crashReporter = require('crash-reporter');
 crashReporter.start({
-  submitUrl: 'http://127.0.0.1:901007',
+  submitUrl: 'http://127.0.0.1:1127',
   autoSubmit: true,
-  ignoreSystemCrashHandler: true
+  ignoreSystemCrashHandler: true,
+  extra: {
+    'extra1': 'extra1',
+    'extra2': 'extra2',
+  }
 });
 
 setImmediate(function() { process.crash(); });