New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 624864 link

Starred by 1 user
Status: Duplicate
Owner:
asvitk...@chromium.org
Closed: Jun 2016
Cc:
klo...@chromium.org
tedc...@chromium.org
jkrcal@chromium.org
skare@chromium.org
isherman@chromium.org
asvitk...@chromium.org
█ rkaplow@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

crash in CreateTrialFromStudy

Project Member Reported by klo...@chromium.org, Jun 30 2016 
Updating Chrome Dev from 53.0.2774.4 to 53.0.2782.2. Chrome crashed on the start up repeatedly.

Here is the snippet of the stack. You can get the full stack from the attached minidump.

Thread 0 (crashed)
 0  libchrome.so!variations::VariationsSeedProcessor::CreateTrialFromStudy [variations_seed_processor.cc : 117 + 0x0]
 1  libchrome.so!variations::VariationsSeedProcessor::CreateTrialsFromSeed [variations_seed_processor.cc : 200 + 0x4]
 2  libchrome.so!variations::VariationsService::CreateTrialsFromSeed [variations_service.cc : 353 + 0x3c]

After uninstall, reinstall, I can't repro the issue any more. I suspect it may be triggered by some experiment flags I had before.
crash.txt
52.0 KB View Download

Comment 1 by asvitk...@chromium.org, Jun 30 2016

Cc: jkrcal@chromium.org
+jkrcal who was investigating a crash as a result of some about:flags integration. Jan, does this look like the same issue?

Otherwise, I'm not really sure what could cause the above. If it really crashed on line 117 of variations_seed_processor.cc - then possibly it's because processed study doesn't have the corresponding experiment name? This could be if something is forced client-side...

If so, probably we need to make the variations_seed_processor.cc code more robust to it. Actually, from the sound of things, I could believe this is the same issue that jkrcal was looking at - since his code does activate a trial with a group name that might not exist in the config.

Comment 2 by jkrcal@chromium.org, Jun 30 2016

Mergedinto: 624807
Status: Duplicate (was: Assigned)

Comment 3 by klo...@chromium.org, Jun 30 2016 

Confirm that I have the mentioned flags turned on previously. So it should be same.
►

Sign in to add a comment