Hi, getPath("logs") return the wrong path.
getPath("logs") return : %APPDATA% + ProductName from the executable of Electron
when it should return : "userData" (when edited) + productName from package.json

The consequence is that Electron creates an empty folder "logs" in the wrong place.

It seems to me that the problem has occurred since this fix : #10911

* Electron version: 1.8.2 (since 1.8.2-beta.2)
* Operating system: Windows 7 64 bit
atom/browser/api/atom_api_app.h
content::mojom::WindowContainerType container_type,
                       const GURL& target_url,
                       const content::Referrer& referrer,
                       const std::string& frame_name,
                       WindowOpenDisposition disposition,
                       const blink::mojom::WindowFeatures& features,
                       const std::vector<std::string>& additional_features,
                       const scoped_refptr<network::ResourceRequestBody>& body,
                       bool user_gesture,
                       bool opener_suppressed,
                       bool* no_javascript_access) override;
  
  void OnGpuProcessCrashed(base::TerminationStatus status) override;
  
  void BrowserChildProcessLaunchedAndConnected(
      const content::ChildProcessData& data) override;
  void BrowserChildProcessHostDisconnected(
      const content::ChildProcessData& data) override;
  void BrowserChildProcessCrashed(
      const content::ChildProcessData& data,
      const content::ChildProcessTerminationInfo& info) override;
  void BrowserChildProcessKilled(
      const content::ChildProcessData& data,
      const content::ChildProcessTerminationInfo& info) override;
 private:
  void SetAppPath(const base::FilePath& app_path);
  void ChildProcessLaunched(int process_type, base::ProcessHandle handle);
  void ChildProcessDisconnected(base::ProcessId pid);
  
  base::FilePath GetPath(mate::Arguments* args, const std::string& name);