<!--  As an open source project with a dedicated but small maintainer team, it can sometimes take a long time for issues to be addressed so please be patient and we will get back to you as soon as we can.
-->

### Issue Details

* **Electron Version:**
  * 9-x-y and above
* **Operating System:**
  * linux
* **Last Known Working Electron version:**
  * 8-x-y

### Expected Behavior

crash reports should be saved locally when `uploadToServer: false`

### Actual Behavior
<!-- A clear and concise description of what actually happened. -->

crashDumps path is empty

### To Reproduce
<!--
Your best chance of getting this bug looked at quickly is to provide an example.
-->

https://gist.github.com/deepak1556/4e98046880c9ff4fecc03813708a2264
shell/browser/electron_browser_client.cc
content::RenderFrameHost* opener,
    const GURL& opener_url,
    const GURL& opener_top_level_frame_url,
    const url::Origin& source_origin,
    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::string& raw_features,
    const scoped_refptr<network::ResourceRequestBody>& body,
    bool user_gesture,
    bool opener_suppressed,
    bool* no_javascript_access) {
  DCHECK_CURRENTLY_ON(BrowserThread::UI);
  int opener_render_process_id = opener->GetProcess()->GetID();
  if (RendererUsesNativeWindowOpen(opener_render_process_id)) {
    if (RendererDisablesPopups(opener_render_process_id)) {
      
      
      return false;
    } else {