Skip to content

Commit

Permalink
Fix crash with --mojo-local-storage
Browse files Browse the repository at this point in the history
null process id was being sent for threads created in the shell process. Use shell process id instead.

R=jam@chromium.org
BUG=

Review URL: https://codereview.chromium.org/1922043003

Cr-Commit-Position: refs/heads/master@{#389841}
  • Loading branch information
ben authored and Commit bot committed Apr 26, 2016
1 parent 4368f22 commit 9e0b9dc
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion services/shell/runner/host/in_process_native_runner.cc
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@
#include "base/callback_helpers.h"
#include "base/location.h"
#include "base/memory/ptr_util.h"
#include "base/process/process.h"
#include "base/strings/utf_string_conversions.h"
#include "base/task_runner.h"
#include "base/thread_task_runner_handle.h"
Expand Down Expand Up @@ -58,7 +59,7 @@ mojom::ShellClientPtr InProcessNativeRunner::Start(
#endif
thread_.reset(new base::DelegateSimpleThread(this, thread_name));
thread_->Start();
pid_available_callback.Run(base::kNullProcessId);
pid_available_callback.Run(base::Process::Current().Pid());

return client;
}
Expand Down

0 comments on commit 9e0b9dc

Please sign in to comment.