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

Issue 894683 link

Starred by 1 user

Issue metadata

Status: Fixed
Closed: Oct 16
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----

Sign in to add a comment

ash_unittests failing on multiple builders

Project Member Reported by, Oct 12

Issue description

Snippet of failure:

[ RUN      ] SplitViewControllerTest.AdjustTransientChildBounds
  ==9304==ERROR: AddressSanitizer: heap-use-after-free on address 0x61400008ed60 at pc 0x0000094d0bea bp 0x7fff7d15bbf0 sp 0x7fff7d15bbe8
  READ of size 8 at 0x61400008ed60 thread T0
      #0 0x94d0be9 in begin buildtools/third_party/libc++/trunk/include/vector:1479:30
      #1 0x94d0be9 in HasObserver base/observer_list.h:302
      #2 0x94d0be9 in aura::Window::HasObserver(aura::WindowObserver const*) const ui/aura/
      #3 0x94ee113 in aura::WindowObserver::OnUnobservingWindow(aura::Window*) ui/aura/
      #4 0x94c7667 in aura::Window::RemoveObserver(aura::WindowObserver*) ui/aura/
      #5 0x76fc645 in RemoveAll base/scoped_observer.h:45:20
      #6 0x76fc645 in ~ScopedObserver base/scoped_observer.h:26
      #7 0x76fc645 in ash::SplitViewDivider::~SplitViewDivider() ash/wm/splitview/
      #8 0x76fc90d in ash::SplitViewDivider::~SplitViewDivider() ash/wm/splitview/
      #9 0x76e641f in operator() buildtools/third_party/libc++/trunk/include/memory:2321:5
      #10 0x76e641f in reset buildtools/third_party/libc++/trunk/include/memory:2634
      #11 0x76e641f in ash::SplitViewController::EndSplitView(ash::SplitViewController::EndReason) ash/wm/splitview/
      #12 0x76ef5fa in ash::SplitViewController::OnSnappedWindowDetached(aura::Window*) ash/wm/splitview/
      #13 0x76f1a43 in ash::SplitViewController::OnWindowDestroyed(aura::Window*) ash/wm/splitview/
      #14 0x94c6354 in aura::Window::~Window() ui/aura/
      #15 0x94c7b8d in aura::Window::~Window() ui/aura/
      #16 0x9a5769f in views::NativeWidgetAura::~NativeWidgetAura() ui/views/widget/
      #17 0x9a5793d in views::NativeWidgetAura::~NativeWidgetAura() ui/views/widget/
      #18 0x99ff867 in views::Widget::~Widget() ui/views/widget/
      #19 0x9a005ed in views::Widget::~Widget() ui/views/widget/
      #20 0x1bb90b4 in operator() buildtools/third_party/libc++/trunk/include/memory:2321:5
      #21 0x1bb90b4 in reset buildtools/third_party/libc++/trunk/include/memory:2634
      #22 0x1bb90b4 in ~unique_ptr buildtools/third_party/libc++/trunk/include/memory:2588
      #23 0x1bb90b4 in ash::SplitViewControllerTest_AdjustTransientChildBounds_Test::TestBody() ash/wm/splitview/
      #24 0x4db8612 in HandleExceptionsInMethodIfSupported<testing::Test, void> third_party/googletest/src/googletest/src/
      #25 0x4db8612 in testing::Test::Run() third_party/googletest/src/googletest/src/
      #26 0x4dba544 in testing::TestInfo::Run() third_party/googletest/src/googletest/src/
      #27 0x4dbb916 in testing::TestCase::Run() third_party/googletest/src/googletest/src/
      #28 0x4de3366 in testing::internal::UnitTestImpl::RunAllTests() third_party/googletest/src/googletest/src/
      #29 0x4de25b5 in HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool> third_party/googletest/src/googletest/src/
      #30 0x4de25b5 in testing::UnitTest::Run() third_party/googletest/src/googletest/src/
      #31 0x7c55f8a in RUN_ALL_TESTS third_party/googletest/src/googletest/include/gtest/gtest.h:2333:46
      #32 0x7c55f8a in base::TestSuite::Run() base/test/
      #33 0x7c5c677 in Run base/callback.h:99:12
      #34 0x7c5c677 in base::(anonymous namespace)::LaunchUnitTestsInternal(base::OnceCallback<int ()>, unsigned long, int, bool, base::OnceCallback<void ()>) base/test/launcher/
      #35 0x7c5c120 in base::LaunchUnitTests(int, char**, base::OnceCallback<int ()>) base/test/launcher/
      #36 0x182f467 in main ash/test/
      #37 0x7f55099bdf44 in __libc_start_main (/lib/x86_64-linux-gnu/

Will try reverting to fix.
Project Member

Comment 2 by, Oct 12

The following revision refers to this bug:

commit e41458e037d2d6eea8297dfee4006848bb6f123a
Author: Patti <>
Date: Fri Oct 12 02:14:27 2018

Revert "splitscreen: Adjust bubble dialog bounds if splitscreen is active."

This reverts commit c3acb7ba6a2b6565656736f3050bfc593baf4781.

Reason for revert: This change may be causing ash_unittests to fail on multiple builders - see 

Original change's description:
> splitscreen: Adjust bubble dialog bounds if splitscreen is active.
> When a dialog is added to a window that's currently showig in
> splitscreen, we should adjust the dialog bounds to show the dialog
> aligned within its own side of the split, rather than across the divider
> bar.
> Note the divider is still placed on top of the snapped windows and its
> transient children if they are placed in default container.
> Bug:  878268 
> Change-Id: Ibdd71f1d42aac419ef73ae9f7a2f78af6bf36880
> Reviewed-on:
> Commit-Queue: Xiaoqian Dai <>
> Reviewed-by: Xiyuan Xia <>
> Cr-Commit-Position: refs/heads/master@{#598827},

Change-Id: I74d78ce0184d2028811872ac633da33ede16f296
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  878268 ,  894683 
Reviewed-by: Patti <>
Commit-Queue: Patti <>
Cr-Commit-Position: refs/heads/master@{#599069}

Components: UI>Shell>WindowManager>Splitscreen
Status: Untriaged (was: Available)
Labels: -Sheriff-Chromium
Status: Assigned (was: Untriaged)
The failure hasn't occurred in days. Removing from sheriff queue.

It looks like r599069 fixed the problem, therefore assigning to the author of the reverted CL.
Status: Fixed (was: Assigned)
Yes, it should have been fixed. 

Sign in to add a comment