Issue metadata
Sign in to add a comment
|
Test failures due to UKM entries not recording on official desktop continuous Win and Win64 builder. |
||||||||||||||||||||||
Issue descriptionPopupTrackerBrowserTest's which are failing : 1.PopupTrackerBrowserTest.ControlClick_HasTracker 2.PopupTrackerBrowserTest.ShiftClick_HasTracker 3.PopupTrackerBrowserTest.WhitelistedPopup_HasTracker 4.PopupTrackerBrowserTest.WindowOpenPopup_HasTracker_GestureClose Link to builders : https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/win%20trunk/builds/86200 https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/win64%20trunk/builds/35433 Link to Error log : https://logs.chromium.org/v/?s=chrome%2Fbb%2Fofficial.desktop.continuous%2Fwin64_trunk%2F35433%2F%2B%2Frecipes%2Fsteps%2Fbrowser_tests%2F0%2Flogs%2FPopupTrackerBrowserTest.ControlClick_HasTracker%2F0
,
Jun 15 2018
I don't think we've made any recent changes on the UKM side that should affect this. The errors are that an expected event was not recorder, which imply that the code here is not being run: https://codesearch.chromium.org/chromium/src/chrome/browser/ui/blocked_content/popup_tracker.cc?rcl=deda551f496ebb23f44d75fb241fa22ef41088f3&l=66
,
Jun 15 2018
Sorry, I meant that this was a general problem, not just in popup tracker. E.g. if you look at the builder failures: https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/win64%20trunk/builds/35433 Multiple test suites are failing and it seems like they all fail some UKM check. That's why I suspected a lower-level problem.
,
Jun 15 2018
+holte back to cc, see response in #3
,
Jun 15 2018
Ah, yes, that is more compelling. Looking into this a bit more.
,
Jun 15 2018
Ah, it was probably this: https://chromium.googlesource.com/chromium/src/+/1e874ff96c92a68a26dd484ffdc858817935754d
,
Jun 15 2018
,
Jun 15 2018
Thanks for investigating! |
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by csharrison@chromium.org
, Jun 15 2018Status: Assigned (was: Untriaged)