New issue
Advanced search Search tips

Issue 601505 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Crash in ash::DisplayManager in linux-chromeos

Project Member Reported by steve...@chromium.org, Apr 7 2016

Issue description

Running linux-chromeos with DCHECK enabled:
1. Enable mirror mode (ctrl-f4 or in the Settings UI)
2. Rotate the displays (ctrl-shift-f3, it does not work in the current Settings UI in mirror mode)
3. Disable mirror more (ctrl-f4 or in the Settings UI)

Crash:

[4247:4247:0407/095835:FATAL:display_util.cc(399)] Check failed: id1 != id2 (2200000001 vs. 2200000001)
#0 0x7f5a1a56541e base::debug::StackTrace::StackTrace()
#1 0x7f5a1a581c0a logging::LogMessage::~LogMessage()
#2 0x7f5a159eff98 ash::CompareDisplayIds()
#3 0x7f5a159ed10b std::__unguarded_linear_insert<>()
#4 0x7f5a159ed0b8 std::__insertion_sort<>()
#5 0x7f5a159e6b41 ash::DisplayManager::UpdateDisplaysWith()
#6 0x7f5a159e8401 ash::DisplayManager::ReconfigureDisplays()
#7 0x7f5a159ee201 ash::DisplayConfigurationController::SetMirrorModeImpl()

 
Project Member

Comment 1 by bugdroid1@chromium.org, Apr 7 2016

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

commit eb29f79de1daad57bb823173fe575be77fd40a1a
Author: oshima <oshima@chromium.org>
Date: Thu Apr 07 21:28:22 2016

Clear the software mirroring display list when adding it to the updated list.

BUG= 601505 
TEST=DisplayManagerTest.SoftwareMirroringAndRotate
R=stevenjb@chromium.org

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

Cr-Commit-Position: refs/heads/master@{#385871}

[modify] https://crrev.com/eb29f79de1daad57bb823173fe575be77fd40a1a/ash/display/display_manager.cc
[modify] https://crrev.com/eb29f79de1daad57bb823173fe575be77fd40a1a/ash/display/display_manager_unittest.cc

Status: Fixed (was: Assigned)
Labels: VerifyIn-51
Status: Verified (was: Fixed)
Bulk verified

Sign in to add a comment