Merge pull request electron#232 from electron/open-external

Use shell.openExternal instead of shell.openItem

Author: kevinsawicki

main-process/media/pdf.js

@@ -16,7 +16,7 @@ ipc.on('print-to-pdf', function (event) {
       if (error) {
         throw error
       }
-      shell.openItem(pdfPath)
+      shell.openExternal('file://' + pdfPath)
       event.sender.send('wrote-pdf', pdfPath)
     })
   })

renderer-process/media/desktop-capturer.js

@@ -24,7 +24,7 @@ screenshot.addEventListener('click', function (event) {
 
         fs.writeFile(screenshotPath, source.thumbnail.toPng(), function (error) {
           if (error) return console.log(error)
-          shell.openItem(screenshotPath)
+          shell.openExternal('file://' + screenshotPath)
           const message = `Saved screenshot to: ${screenshotPath}`
           screenshotMsg.textContent = message
         })