Issue metadata
Sign in to add a comment
|
PaymentRequestMiscellaneousEGTest fails with slim-navigation-manager |
||||||||||||||||||||||
Issue descriptionFailing test with --enable-features=SlimNavigationManager: PaymentRequestMiscellaneousEGTest/testMultipleRequests
,
Aug 20
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e6c24877edcb59c340d0e18f84e4edda13bd99bc commit e6c24877edcb59c340d0e18f84e4edda13bd99bc Author: Danyao Wang <danyao@chromium.org> Date: Mon Aug 20 13:23:23 2018 [Nav Experiment] Skip window ID injection for placeholder URLs. gCrWeb is not injected for placeholder URLs, so CRWJSWindowIDManager ends up repeatedly injecting window ID for NTPs, which interferes with injected JavaScript for the next web page. This fixes PaymentRequestMiscellaneousEGTest/testMultipleRequests. It was failing because the payment request messages were ignored due to mismatched window ID. Bug: 874637 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I9a4f95d8f2f767c27a0c8dd953bb0d9914b6c0ed Reviewed-on: https://chromium-review.googlesource.com/1179221 Reviewed-by: Eugene But <eugenebut@chromium.org> Commit-Queue: Danyao Wang <danyao@chromium.org> Cr-Commit-Position: refs/heads/master@{#584421} [modify] https://crrev.com/e6c24877edcb59c340d0e18f84e4edda13bd99bc/ios/web/web_state/ui/crw_web_controller.mm
,
Aug 27
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by danyao@chromium.org
, Aug 15