diff options
author | primiano <primiano@chromium.org> | 2015-04-08 13:06:39 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-04-08 20:07:10 +0000 |
commit | 4950a30243ae4573b78033781830a113d220f938 (patch) | |
tree | 8429869170818475ba104be39f5b1963f9a08eec /DEPS | |
parent | 5384f002f839439ef666ed9688069e42ca5ccdca (diff) | |
download | chromium_src-4950a30243ae4573b78033781830a113d220f938.zip chromium_src-4950a30243ae4573b78033781830a113d220f938.tar.gz chromium_src-4950a30243ae4573b78033781830a113d220f938.tar.bz2 |
[tracing] Fix single-process detection logic of ChildTraceMessageFilter
Currently the instantiation of ChildTraceMessageFilter in
child_thread_impl.cc is guarded by a check on
CommandLine.HasSwitch(kSingleProcess)
The rationale is that in single-process mode we don't want to have any
message filter as the browser-side tracing will cover all the threads.
However, the aforementioned check is not complete, as it does not
covers the case of tests using the InProcessUtilityThreadHelper [1].
In those tests, the kSingleProcess flag is not present, but the
renderer is in-process anyways.
Replacing the the HasSwitch check in with IsInBrowserProcess(),
which covers also the latter case.
[1] for instance extensions/browser/sandboxed_unpacker_unittest.cc
BUG=474973
Review URL: https://codereview.chromium.org/1069253003
Cr-Commit-Position: refs/heads/master@{#324276}
Diffstat (limited to 'DEPS')
0 files changed, 0 insertions, 0 deletions