New issue
Advanced search Search tips

Issue 679032 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: ----



Sign in to add a comment

ios_web_unittests (iPad Air 2 iOS 10.0) failing on chromium.mac/ios-simulator

Project Member Reported by dewittj@chromium.org, Jan 6 2017

Issue description

ios_web_unittests (iPad Air 2 iOS 10.0) failing on chromium.mac/ios-simulator

Type: build-failure

Builders failed on: 
- ios-simulator: 
  https://build.chromium.org/p/chromium.mac/builders/ios-simulator


Tests:
[ RUN      ] CRWWebControllerPageDialogOpenPolicyTest.AllowPrompt
../../ios/web/web_state/ui/crw_web_controller_unittest.mm:608: Failure
Value of: ExecuteJavaScript(@"prompt('Yes?', 'No')")
  Actual: <null>
Expected: @"Maybe"
Which is: Maybe
[  FAILED  ] CRWWebControllerPageDialogOpenPolicyTest.AllowPrompt (280 ms)

[ RUN      ] CRWWebControllerPageDialogOpenPolicyTest.AllowConfirmWithTrue
../../ios/web/web_state/ui/crw_web_controller_unittest.mm:553: Failure
Value of: ExecuteJavaScript(@"confirm('test')")
  Actual: 0
Expected: @__objc_yes
Which is: 1
[  FAILED  ] CRWWebControllerPageDialogOpenPolicyTest.AllowConfirmWithTrue (290 ms)


 
Components: -Blink>HTML>Dialog Test>iOS Mobile>WebView>Glue
Status: Started (was: Available)
 Issue 679059  has been merged into this issue.
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 6 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1d41c27d99e1f50349bee096fa92efe91635c9f1

commit 1d41c27d99e1f50349bee096fa92efe91635c9f1
Author: eugenebut <eugenebut@chromium.org>
Date: Fri Jan 06 23:26:37 2017

[ios] Fixed memory management bug in TestJavaScriptDialogPresenter.

Copy strings before passing them to scoped_nsobjects.

BUG= 679032 

Review-Url: https://codereview.chromium.org/2612423003
Cr-Commit-Position: refs/heads/master@{#442093}

[modify] https://crrev.com/1d41c27d99e1f50349bee096fa92efe91635c9f1/ios/web/public/test/fakes/test_java_script_dialog_presenter.mm

Status: Fixed (was: Started)

Sign in to add a comment