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

Issue 872539 link

Starred by 1 user
Status: Fixed
Owner:
sky@chromium.org
Closed: Aug 9
Cc:
█ mgiuca@chromium.org
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

views_mus_unittests failing on chromium.memory/Linux ChromiumOS MSan Tests

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Aug 9 
Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of mgiuca@chromium.org

views_mus_unittests failing on chromium.memory/Linux ChromiumOS MSan Tests

Builders failed on: 
- Linux ChromiumOS MSan Tests: 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20ChromiumOS%20MSan%20Tests

Comment 1 by mgiuca@chromium.org, Aug 9 

First failing build: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20ChromiumOS%20MSan%20Tests/8164

Regression range: r581559 .. r581614.

Failure:

[ RUN      ] RemoteViewProviderTest.EmbedAgain
../../ui/views/mus/remote_view/remote_view_provider_unittest.cc:166: Failure
Expected: (new_embedder) != (embedder), actual: 0x712000002280 vs 0x712000002280
Stack trace:
    #0 0x000002548de7 in StackTraceGetter::CurrentStackTrace(int, int) ./../../third_party/googletest/custom/gtest/internal/custom/stack_trace_getter.cc:22:27
    #1 0x000002581489 in testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop(int) ./../../third_party/googletest/src/googletest/src/gtest.cc:810:35
    #2 0x00000257f6e2 in testing::internal::AssertHelper::operator=(testing::Message const&) const ./../../third_party/googletest/src/googletest/src/gtest.cc:386:25
[  FAILED  ] RemoteViewProviderTest.EmbedAgain (14 ms)

(The call to SimulateEmbed() should return a different object each time, but it returns the same object.)

FindIt blamed r581568 with 85% confidence:
https://findit-for-me.appspot.com/waterfall/failure?url=https://build.chromium.org/p/chromium.memory/builders/Linux%20ChromiumOS%20MSan%20Tests/builds/8164

A quick look over the regression range, I would agree; it's the only CL touching mus and window related code.

Comment 2 by mgiuca@chromium.org, Aug 9

Owner: sky@chromium.org
Status: Fixed (was: Available)
This CL was already reverted by sky@: r581718
►

Sign in to add a comment