Half of MSE Conformance tests and all EME conformance tests fail on Chrome 61.0.3162.3 |
|||||||||||||||
Issue descriptionChrome Version: 61.0.3162.3 OS: Android O This is a regression from Chrome release 58.0.3029.82 and Beta build. Thom, is there any by design feature change you can think of may cause such massive amount of failure? These are the tests pass in beta build 60.0.3112.72 and fail on dev 61.0.3162.3 https://yt-dash-mse-test.commondatastorage.googleapis.com/unit-tests/tip.html?timestamp=1501017199406 MSE Core: Test 21 MSE(Opus): Test: 36~41 MSE(AAC): Test: 50~55 MSE(VP9): Test: 59, 64~65, 69, 73~75 MSE (H264): Test: 79, 83~85, 87~95 MSE (WAA): Test: 97~99 MSE (Media) Test: 100~102
,
Jul 25 2017
I don't know of a recent change that would cause this level of failure.
,
Jul 25 2017
I suspect this is a product change/breakage. Let me know if I can help.
,
Jul 25 2017
Matt, Can you take a look? All most half of MSE and all EME conformance tests failed. This is very unusual. I have tried beta build on the same device, all tests pass as expected. This is a recent regression.
,
Jul 25 2017
,
Jul 26 2017
I don't have O on a device that I can readily try. I'm also unaware of any major MSE change in that range; perhaps something in AVDA changed (sandersd@ is out). Dale, any ideas? Yining, can you try bisecting please to narrow down the regression range? Also, is this regression *only* on Android O?
,
Jul 26 2017
,
Jul 26 2017
=>liberato who's making changes in these areas
,
Jul 26 2017
We've already cut M61, so this is RBB for the 61 beta.
,
Jul 26 2017
Pls apply appropriate OS label.
,
Jul 26 2017
I just tried a few repro attempts on non-Android-O (attempting only a subset of the reported failing tests: MSE(Opus): Test: 36~41, and MSE (Media) Test: 100~102: Linux Chrome: Unable to repro on either 59.0.3071.115 (Official Build) (64-bit) or ToT (M62) Debug. Android M Nexus 7 (2013): Unable to repro on either 59.0.3071.125 (Official Build) (32-bit) or 61.0.3162.3 (Official Build) dev (32-bit) I haven't tried to repro on Android O. liberato@, please take a look. Also, +xhwang@ w.r.t. EME conformance failures. yiningc@, can you please provide a quick example of such an EME conformance failure too?
,
Jul 26 2017
Did EME tests succeed on previous Chrome versions and start to fail on M61?
,
Jul 26 2017
I have tried on Android K, M, and O this issue only repro on Android O. On the other two android OS, both MSE and EME conformance tests pass appropriately on 61.0.3162.3 build. But on Andoid O, there are massive MSE and EME failures the Chrome 61.0.3162.3. the regression on Android O happened somewhere between 61.0.3136.4 and 61.0.3162.3.
,
Jul 26 2017
will take a look.
,
Jul 26 2017
MSE core (21) fails yiningc@'s pixel using chrome dev 3162.3 . my locally build ToT and also locally built chromium 61.0.3162.0 both pass. will try building .3 exactly.
,
Jul 26 2017
,
Jul 26 2017
Hmm, if this is O specifically it may not be ReleaseBlock-Beta since it's unreleased, but I'm not sure what our release schedule is. +cmasso who is in charge of Android M61.
,
Jul 26 2017
61.0.3162.3 (93e40b30f8) built locally runs fine. so, it's something about the official dev build.
,
Jul 26 2017
--no-experiments: chrome dev build still fails. is_official_build: local build still works. trying build is_chrome_branded.
,
Jul 26 2017
is_chrome_branded local build works. i still have a lot of variations in chrome://about in the dev build that aren't in my local build. i'll try that next.
,
Jul 26 2017
network folks: in the failed case, i'm seeing this error in the official dev build: Assert failed: XHR length is (32072) which should be (828645) any idea what would be different between an official dev build and my locally built (is_chrome_branded, is_official_build)? variations: setting the channel to either canary or dev didn't cause my local build to fail.
,
Jul 27 2017
dalecurtis@ suggested unzipping the apks. min/target sdks are different, 16 / 23 locally and 24 / 26 in the dev build (why?). i set mine to 24 / 26 locally, but still works fine. anyway, i downloaded 61.0.3163.13 dev from the play store onto my n5x with OPM1.170726, and it works fine. i upgraded chrome dev on the pixel to the same version, and it fails. it has OPR1.170420. so, i'm going to install that on the n5x and see what happens.
,
Jul 27 2017
i ended up installing OPM on both devices. nothing changed. i tried canary 62.0.3166.3 from the play store, and it works on the pixel. so, to summarize: A. 61 dev from play store works on N5x, not on pixel xl. both have OPM. B. 61 dev built locally (official, chrome branded) works on both N5x and Pixel XL. C. 62 canary from play store works on Pixel. i can't explain all of those at once. A seems like it's related to the device. B seems like it's something about how it's built / packaged. C seems like it's something we fixed in chrome. hubbe@ suggested data saver, which i enabled manually. didn't break the working cases on the pixel.
,
Jul 27 2017
Hmm, will reach out to clank-team@ to see if anyone has any ideas.
,
Jul 27 2017
i'm really wondering if it's not related to finch in some way, and my tests above were insufficient to rule it out. as in, looking at A,B,C as ruling things out: - it can't be the OS, since A - it can't be a bad CL in chrome, since B - it can't be the hardware, since B - it can't be packaging / signing since C what's left? it's chrome dev on this particular device. that sounds like finch to me.
,
Jul 27 2017
i installed an internal build of chrome on the pixel. i had to uninstall chrome-dev to do it, and it worked. i then upgraded in-place to the play store version, and it still works. luckily, i captured all the variations before and after: good: AdsDisableActiveView-Experiment bad: AdsDisableActiveView-Control good: AppBannerTriggering-CheckOnPageLoad bad: AppBannerTriggering-Control good: BackgroundTracing-AggressiveLowMemorySignalStats bad: BackgroundTracing-default good: ChromeHome-Enabled_Button bad: ChromeHome-Default good: DisablePreferCompositingToLCDTextOnLowEndAndroid-Control bad: DataReductionProxyUseQuic-Enabled12 good: DynamicExpectCT-DynamicExpectCTEnabled bad: DisablePreferCompositingToLCDTextOnLowEndAndroid-DisablePreferCompositingToLCDTextOnLowEndAndroid good: NTPArticleSuggestions-Enabled-FCS-NewSchedule bad: NTPArticleSuggestions-Enabled-FCS good: NetDelayableH2AndQuicRequests-Control2 bad: NetDelayableH2AndQuicRequests-Yielding good: NewPhotoPicker-Enabled bad: NewPhotoPicker-Control good: OmniboxBundledExperimentV1-Dev_Android_HQPUrlsIndexing_Control bad: OmniboxBundledExperimentV1-Dev_Android_HighHQPUrlsIndexedAtStartup good: OmniboxPlaceholderExperiment-Unused bad: OmniboxPlaceholderExperiment-Default good: PasswordGeneration-Disabled bad: PasswordGeneration-Enabled good: PersistentHistograms-EnabledInMemory7 bad: PersistentHistograms-EnabledOnDisk7 good: PreviewsClientLoFi-Enabled_2G_Replace_Server_Placeholders bad: PreviewsClientLoFi-Control_2G_Replace_Server_Placeholders good: QUIC-EnabledPathMtuDiscoveryLow bad: QUIC-EnabledMultipleVersions39and37 good: ResourceLoadScheduler-Disabled bad: ResourceLoadScheduler-Enabled_bg_limit_5 good: SocketReadIfReady-Control bad: SocketReadIfReady-Enabled good: SyncUSSAutocomplete-Control bad: SyncUSSAutocomplete-Enabled good: TranslateRankerModel-OverrideControl2 bad: TranslateRankerModel-CKEnableTranslateRankerDisableUlp good: UKM-Control2 bad: UKM-Enabled2 good: UMA-Uniformity-Trial-10-Percent-group_02 bad: UMA-Uniformity-Trial-10-Percent-group_01 good: UMA-Uniformity-Trial-20-Percent-group_01 bad: UMA-Uniformity-Trial-20-Percent-default good: VsyncAlignedInput-Enabled bad: VsyncAlignedInput-Control
,
Jul 27 2017
I'd focus on forcing opt in for the QUIC/Net trials on your good build and see what happens
,
Jul 28 2017
i've not had luck with this. on my good build, setting this does nothing: --force-fieldtrials=QUIC/EnabledMultipleVersions39and37/SocketReadIfReady/Enabled/NetDelayableH2AndQuicRequests/Yielding/DataReductionProxyUseQuic/Enabled12/UKM/Enabled2 --enable-features=Ukm<UKM --force-fieldtrial-params=QUIC.EnabledMultipleVersions39and37:enable_quic/true/quic_version/QUIC_VERSION_39%2CQUIC_VERSION_37,UKM.Enabled2:RecordSessionId/true setting the entire list of "bad" field trials also doesn't seem to help, though it's possible that i've simply done something wrong with such a big list. since i re-installed the internal build, i can't get chrome-dev to fail, either. i've tried uninstalling and re-installing chrome-dev many, many times. it's now 61.0.3163.13. the omnibox is sometimes at the bottom, so it's definitely getting new field trials.
,
Jul 28 2017
> network folks: in the failed case, i'm seeing this error in the official dev build: > Assert failed: XHR length is (32072) which should be (828645) I am not sure if that's the fatal error here. Please follow instructions at https://sites.google.com/a/chromium.org/dev/for-testers/providing-network-details to obtain a chrome://net-export log. It should tell us what is happening in the //net stack.
,
Jul 31 2017
cmasso: TL;DR: without a repro, and even then only observed on O, may i remove ReleaseBlock-Beta? still no luck. i've tried re-building .3, with some of the finch-releated features hardwired in the code, since not all of the --force-fieldtrials seemed to actually show up in the variations. i also tried clearing app data a bunch of times to get new variations. no repro. i'm not sure what else to do with this without a repro. would be nice to know if clearing app data is sufficient to fix it once it's broken, since we don't really know for sure that a finch experiment is involved. just seems liklely to me.
,
Jul 31 2017
,
Jul 31 2017
Seems fine to remove RBB label if yining@ can no longer repro. Were we able to capture a net log?
,
Jul 31 2017
net log: no, unfortunately. the repro was gone by that point. i'll return the pixel to yiningc@, and if still no repro, then i'll -RBB.
,
Jul 31 2017
I couldn't repro it on the latest chrome dev (61.0.3163.13), canary (62.0.3172.0) and beta (60.0.3112.78) build anymore.
,
Jul 31 2017
thanks, yiningc@. marking WAI, removing ReleaseBlock-Beta. please re-open if this comes back. |
|||||||||||||||
►
Sign in to add a comment |
|||||||||||||||
Comment 1 by yini...@chromium.org
, Jul 25 2017Components: -Internals>Media Internals>Media>Source
Status: Assigned (was: Untriaged)