New issue
Advanced search Search tips

Issue 650828 link

Starred by 1 user
Status: Fixed
Owner:
thestig@chromium.org
Closed: Sep 2016
Cc:
dominickn@chromium.org
timvolod...@chromium.org
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

PermissionContextBaseTests fail with --single-process-tests

Project Member Reported by thestig@chromium.org, Sep 27 2016 
Do a normal build and run: unit_tests --gtest_filter=PermissionContextBaseTests.* --single-process-tests --gtest_shuffle

As seen on DrMemory bots, and see crashes like:

[ RUN      ] PermissionContextBaseTests.TestAskAndBlockNoPersist
Xlib:  extension "RANDR" missing on display ":2".
Received signal 11 SEGV_MAPERR 0000000000c0
#0 0x7fb964472f0e base::debug::StackTrace::StackTrace()
#1 0x7fb964472a4f base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x7fb967762330 <unknown>
#3 0x7fb96447fe21 base::FeatureList::GetAssociatedFieldTrial()
#4 0x7fb96447fdad base::FeatureList::GetFieldTrial()
#5 0x000004a38a56 variations::GetVariationParamValueByFeature()
#6 0x0000069265e8 ntp_snippets::(anonymous namespace)::GetParamValue()
#7 0x00000692638c ntp_snippets::(anonymous namespace)::GetDiscountRatePerHour()
#8 0x0000069268a4 ntp_snippets::UserClassifier::RegisterProfilePrefs()
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 28 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9bdf7f221dcf693eebce8c57567314e058368c36

commit 9bdf7f221dcf693eebce8c57567314e058368c36
Author: thestig <thestig@chromium.org>
Date: Wed Sep 28 00:56:20 2016

Clean up PermissionContextBaseTests code.

Fix crashes when running with --single-process-tests

BUG= 650828 

Review-Url: https://codereview.chromium.org/2373883002
Cr-Commit-Position: refs/heads/master@{#421404}

[modify] https://crrev.com/9bdf7f221dcf693eebce8c57567314e058368c36/chrome/browser/permissions/permission_context_base_unittest.cc

Comment 2 by thestig@chromium.org, Sep 28 2016

Owner: thestig@chromium.org
Status: Fixed (was: Untriaged)
►

Sign in to add a comment