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

Issue 852767 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: ----



Sign in to add a comment

sync_integration_tests are failing on Windows-10-15063 - WebContents title incorrect

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jun 14 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of fdoray@chromium.org

Builders failed on: 
- Win10 Tests x64 (dbg): 
  https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20Tests%20x64%20%28dbg%29

Sample failures:

../../chrome/browser/sync/test/integration/sync_test.cc(1147): error: Expected equality of these values:
  "Notification sent"
  base::UTF16ToASCII( browser()->tab_strip_model()->GetActiveWebContents()-> GetTitle())
    Which is: ""
Stack trace:
Backtrace:
	StackTraceGetter::CurrentStackTrace [0x00007FF7CCCEC560+80]
	testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop [0x00007FF7CCD0735A+90]
	testing::internal::AssertHelper::operator= [0x00007FF7CCD06E5A+90]
	SyncTest::TriggerNotification [0x00007FF7CF589A95+1157]

[4996:4336:0613/233148.854:ERROR:get_updates_processor.cc(243)] PostClientToServerMessage() failed during GetUpdates
../../chrome/browser/sync/test/integration/sync_test.cc(1173): error: Expected equality of these values:
  "Migration: 200"
  base::UTF16ToASCII( browser()->tab_strip_model()->GetActiveWebContents()-> GetTitle())
    Which is: ""
Stack trace:
Backtrace:
	StackTraceGetter::CurrentStackTrace [0x00007FF7CCCEC560+80]
	testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop [0x00007FF7CCD0735A+90]
	testing::internal::AssertHelper::operator= [0x00007FF7CCD06E5A+90]
	SyncTest::TriggerMigrationDoneError [0x00007FF7CF58A0EE+1006]

../../chrome/browser/sync/test/integration/sync_test.cc(1111): error: Expected equality of these values:
  "Notifications disabled"
  base::UTF16ToASCII( browser()->tab_strip_model()->GetActiveWebContents()-> GetTitle())
    Which is: ""
Stack trace:
Backtrace:
	StackTraceGetter::CurrentStackTrace [0x00007FF7CCCEC560+80]
	testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop [0x00007FF7CCD0735A+90]
	testing::internal::AssertHelper::operator= [0x00007FF7CCD06E5A+90]
	SyncTest::DisableNotificationsImpl [0x00007FF7CF58909D+781]
	SyncTest::DisableNotifications [0x00007FF7CF5891C3+19]

 

Comment 1 by fdoray@google.com, Jun 14 2018

Components: Services>Sync

Comment 2 by mastiz@chromium.org, Jun 14 2018

Could be due to https://chromium-review.googlesource.com/1097405, apologies if that's the case.

I'll investigate for 10 mins and revert otherwise, feel free to meanwhile trigger a speculative revert if it's annoying enough.

Comment 3 by mastiz@chromium.org, Jun 14 2018

Actually, the patch above seems quite unrelated, and the tests seem to be flaky for a very long time.

Comment 4 by mastiz@chromium.org, Jun 14 2018

Many of the tests above seem to be related to the legacy python sync server, used for migration tests. Perhaps it's not working well on this particular platform, not sure.

Anyway, it doesn't seem a recent regression.

Comment 5 by hzl@chromium.org, Jun 14 2018

I believe the test is just flaky. Now it is passing again.

Comment 6 by hzl@chromium.org, Jun 14 2018

Status: WontFix (was: Available)
The test has been passing for some time now. Let me close this one and reopen if necessary.

Sign in to add a comment