Issue metadata
Sign in to add a comment
|
WebViewLayoutTests failing on 2 builders since May 4 |
||||||||||||||||||||
Issue descriptionRun CTS failing on 2 builders Builders failed on: - Android WebView L (dbg): https://build.chromium.org/p/chromium.android/builders/Android%20WebView%20L%20%28dbg%29 - Android WebView M (dbg): https://build.chromium.org/p/chromium.android/builders/Android%20WebView%20M%20%28dbg%29
,
May 9 2017
some js interface change I think
,
May 9 2017
The CTS failure started here, after the system webview shell test failure: https://build.chromium.org/p/chromium.android/builders/Android%20WebView%20L%20%28dbg%29/builds/4716 which most likely is caused by this CL: https://codereview.chromium.org/2801813005 add +avi I will disable these two tests for now here: https://codereview.chromium.org/2874533002
,
May 9 2017
,
May 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/ce2aa1a17a985a989bada5ecf204f9e2232c09ad commit ce2aa1a17a985a989bada5ecf204f9e2232c09ad Author: yolandyan <yolandyan@chromium.org> Date: Tue May 09 23:54:14 2017 Disable failing tests on WebView bots BUG= 719974 TBR=sgurun@chromium.org,timvolodine@chromium.org Review-Url: https://codereview.chromium.org/2874533002 Cr-Commit-Position: refs/heads/master@{#470440} [modify] https://crrev.com/ce2aa1a17a985a989bada5ecf204f9e2232c09ad/android_webview/tools/cts_config/expected_failure_on_bot.json [modify] https://crrev.com/ce2aa1a17a985a989bada5ecf204f9e2232c09ad/android_webview/tools/system_webview_shell/layout_tests/src/org/chromium/webview_shell/test/WebViewLayoutTest.java
,
May 10 2017
,
May 10 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e50ed3b9e1a96c042dddb7e14d0a7ef40e19ce3d commit e50ed3b9e1a96c042dddb7e14d0a7ef40e19ce3d Author: yolandyan <yolandyan@chromium.org> Date: Wed May 10 18:43:46 2017 Change disabled CTS test BUG= 719974 TBR=sgurun@chromium.org Review-Url: https://codereview.chromium.org/2875743002 Cr-Commit-Position: refs/heads/master@{#470656} [modify] https://crrev.com/e50ed3b9e1a96c042dddb7e14d0a7ef40e19ce3d/android_webview/tools/cts_config/expected_failure_on_bot.json
,
May 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f53cbba636ebdaff456dc42d4356c353e2fdcb38 commit f53cbba636ebdaff456dc42d4356c353e2fdcb38 Author: timvolodine <timvolodine@chromium.org> Date: Thu May 11 15:40:04 2017 Re-enable WebView stable interfaces layout test. Update not-webview-exposed.txt (allowPaymentRequest) and re-enable the testWebViewIncludedStableInterfaces test. BUG= 719974 NOTRY=true TBR=yolandyan@chromium.org,avayvod@chromium.org Review-Url: https://codereview.chromium.org/2877783002 Cr-Commit-Position: refs/heads/master@{#470966} [modify] https://crrev.com/f53cbba636ebdaff456dc42d4356c353e2fdcb38/android_webview/tools/system_webview_shell/layout_tests/src/org/chromium/webview_shell/test/WebViewLayoutTest.java [modify] https://crrev.com/f53cbba636ebdaff456dc42d4356c353e2fdcb38/android_webview/tools/system_webview_shell/test/data/webexposed/not-webview-exposed.txt
,
May 15 2017
bots green, this one is fixed
,
Aug 10 2017
Hi Tim, it seemed that you only fixed part of the issue, the test case testOnJsBeforeUnloadIsCalled is still failed, but it has been disabled in our cts bot, now it also failed in oc-rm1, I assigned the b/64152248 to you.
,
Aug 18 2017
Hmm, yes looks like this got confused somehow, there are two issues in this bug.. I've spun off the CTS issue into a separate bug crbug.com/757007
,
Aug 18 2017
,
Aug 18 2017
,
Sep 15 2017
I think this is related to http://b/64152248. We have fixed the CTS test on master, OMR1, O, and I *think* nougat. Is it reasonable to enable this test again?
,
Sep 30
Right, this bug is about LayoutTests that have been fixed. So marking this one as fixed. Will continue with the CTS issue on crbug.com/757007 (which has been spun off for this reason) |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by hartma...@chromium.org
, May 9 2017