Merge pull request #511 from brave/browser-laptop-issue-13254

Fix empty devtool profile
brave · Feb 28, 2018 · 66d3cbb · 66d3cbb
1 parent 0303a42
commit 66d3cbb
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/atom/browser/common_web_contents_delegate.cc b/atom/browser/common_web_contents_delegate.cc
@@ -490,8 +490,12 @@ void CommonWebContentsDelegate::OnSaveFileSelected(
     const std::string& content,
     const std::vector<base::FilePath>& paths) {
   DCHECK(!paths.empty());
-  WriteToFile(paths[0], content);
-  OnDevToolsSaveToFile(url);
+  saved_files_[url] = paths[0];
+  BrowserThread::PostTaskAndReply(
+      BrowserThread::FILE, FROM_HERE,
+      base::Bind(&WriteToFile, paths[0], content),
+      base::Bind(&CommonWebContentsDelegate::OnDevToolsSaveToFile,
+                 base::Unretained(this), url));
 }
 void CommonWebContentsDelegate::OnSaveFileSelectionCancelled(
     const std::string url) {