{"action":"created","issue":{"url":"https://api.github.com/repos/appcelerator/titanium_mobile/issues/9889","repository_url":"https://api.github.com/repos/appcelerator/titanium_mobile","labels_url":"https://api.github.com/repos/appcelerator/titanium_mobile/issues/9889/labels{/name}","comments_url":"https://api.github.com/repos/appcelerator/titanium_mobile/issues/9889/comments","events_url":"https://api.github.com/repos/appcelerator/titanium_mobile/issues/9889/events","html_url":"https://github.com/appcelerator/titanium_mobile/pull/9889","id":300762287,"number":9889,"title":"[TIMOB-25786] Android: WebView eval JS timeout error","user":{"login":"sgtcoolguy","id":3252,"avatar_url":"https://avatars0.githubusercontent.com/u/3252?v=4","gravatar_id":"","url":"https://api.github.com/users/sgtcoolguy","html_url":"https://github.com/sgtcoolguy","followers_url":"https://api.github.com/users/sgtcoolguy/followers","following_url":"https://api.github.com/users/sgtcoolguy/following{/other_user}","gists_url":"https://api.github.com/users/sgtcoolguy/gists{/gist_id}","starred_url":"https://api.github.com/users/sgtcoolguy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sgtcoolguy/subscriptions","organizations_url":"https://api.github.com/users/sgtcoolguy/orgs","repos_url":"https://api.github.com/users/sgtcoolguy/repos","events_url":"https://api.github.com/users/sgtcoolguy/events{/privacy}","received_events_url":"https://api.github.com/users/sgtcoolguy/received_events","type":"User","site_admin":false},"labels":[{"id":319882185,"url":"https://api.github.com/repos/appcelerator/titanium_mobile/labels/android","name":"android","color":"006b75","default":false},{"id":319887342,"url":"https://api.github.com/repos/appcelerator/titanium_mobile/labels/improvement","name":"improvement","color":"64bf3d","default":false},{"id":671068703,"url":"https://api.github.com/repos/appcelerator/titanium_mobile/labels/parity","name":"parity","color":"e99695","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-02-27T19:29:54Z","updated_at":"2018-02-27T19:32:34Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/appcelerator/titanium_mobile/pulls/9889","html_url":"https://github.com/appcelerator/titanium_mobile/pull/9889","diff_url":"https://github.com/appcelerator/titanium_mobile/pull/9889.diff","patch_url":"https://github.com/appcelerator/titanium_mobile/pull/9889.patch"},"body":"**JIRA:** https://jira.appcelerator.org/browse/TIMOB-25786\r\n\r\n**Optional Description:**\r\n- Add an async variant of Ti.UI.WebView.evalJS, add unit test for it. This matches Windows API.\r\n- Add warning log for calling sync version when run-on-main-thread is true.\r\n"},"comment":{"url":"https://api.github.com/repos/appcelerator/titanium_mobile/issues/comments/368998342","html_url":"https://github.com/appcelerator/titanium_mobile/pull/9889#issuecomment-368998342","issue_url":"https://api.github.com/repos/appcelerator/titanium_mobile/issues/9889","id":368998342,"user":{"login":"build","id":125029,"avatar_url":"https://avatars1.githubusercontent.com/u/125029?v=4","gravatar_id":"","url":"https://api.github.com/users/build","html_url":"https://github.com/build","followers_url":"https://api.github.com/users/build/followers","following_url":"https://api.github.com/users/build/following{/other_user}","gists_url":"https://api.github.com/users/build/gists{/gist_id}","starred_url":"https://api.github.com/users/build/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/build/subscriptions","organizations_url":"https://api.github.com/users/build/orgs","repos_url":"https://api.github.com/users/build/repos","events_url":"https://api.github.com/users/build/events{/privacy}","received_events_url":"https://api.github.com/users/build/received_events","type":"User","site_admin":false},"created_at":"2018-02-27T19:32:34Z","updated_at":"2018-02-27T19:32:34Z","author_association":"COLLABORATOR","body":"\n<!--\n  1 failure:  :disappointed_rel...\n  0 warning: \n  1 messages\n  \n-->\n\n<table>\n  <thead>\n    <tr>\n      <th width=\"50\"></th>\n      <th width=\"100%\" data-danger-table=\"true\">Fails</th>\n    </tr>\n  </thead>\n  <tbody><tr>\n      <td>:no_entry_sign:</td>\n      <td>\n\n  :disappointed_relieved: `npm test` failed. See below for details.\n  </td>\n    </tr>\n  </tbody>\n</table>\n\n\n\n<table>\n  <thead>\n    <tr>\n      <th width=\"50\"></th>\n      <th width=\"100%\" data-danger-table=\"true\">Messages</th>\n    </tr>\n  </thead>\n  <tbody><tr>\n      <td>:book:</td>\n      <td>\n\n  ```\n> titanium-mobile@7.2.0 test /Users/build/jenkins/workspace/sdk_titanium_mobile_PR-9889-AWQ3M5BGI7E5HAVI7L3AGZAUHM5JMMAFGKJWBIPBCOKWJY7QYR6A\n> grunt\n\nRunning \"appcJs:src\" (appcJs) task\n\nRunning \"eslint:src\" (eslint) task\n\n/Users/build/jenkins/workspace/sdk_titanium_mobile_PR-9889-AWQ3M5BGI7E5HAVI7L3AGZAUHM5JMMAFGKJWBIPBCOKWJY7QYR6A/android/cli/commands/_buildModule.js\n  554:18  warning  Found non-literal argument to RegExp Constructor  security/detect-non-literal-regexp\n\n/Users/build/jenkins/workspace/sdk_titanium_mobile_PR-9889-AWQ3M5BGI7E5HAVI7L3AGZAUHM5JMMAFGKJWBIPBCOKWJY7QYR6A/tests/Resources/ti.android.notificationmanager.addontest.js\n  13:3  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n\n/Users/build/jenkins/workspace/sdk_titanium_mobile_PR-9889-AWQ3M5BGI7E5HAVI7L3AGZAUHM5JMMAFGKJWBIPBCOKWJY7QYR6A/tests/Resources/ti.ui.picker.addontest.js\n   85:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  122:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  123:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  171:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  172:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  175:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  176:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  245:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  246:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  252:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n  253:5  warning  Expected an assignment or function call and instead saw an expression  no-unused-expressions\n\n/Users/build/jenkins/workspace/sdk_titanium_mobile_PR-9889-AWQ3M5BGI7E5HAVI7L3AGZAUHM5JMMAFGKJWBIPBCOKWJY7QYR6A/tests/Resources/ti.ui.tableview.addontest.js\n  11:5  warning  'should' is assigned a value but never used  no-unused-vars\n\n✖ 14 problems (0 errors, 14 warnings)\n\n\nRunning \"continue:on\" task\n\nRunning \"retire:js\" (retire) task\nIgnoring {\"paths\":[\"growl\"],\"descriptors\":[]}\nDownloading https://raw.github.com/RetireJS/retire.js/master/repository/jsrepository.json ...\n(node:70622) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated.\nChecking: dangerfile.js\nChecking: Gruntfile.js\nChecking: apidoc/docgen.js\nChecking: apidoc/lib/addon_generator.js\nChecking: apidoc/lib/changes_generator.js\nChecking: apidoc/lib/common.js\nChecking: apidoc/lib/html_generator.js\nChecking: apidoc/lib/jsca_generator.js\nChecking: apidoc/lib/jsduck_generator.js\nChecking: apidoc/lib/json_generator.js\nChecking: apidoc/lib/modulehtml_generator.js\nChecking: apidoc/lib/parity_generator.js\nChecking: apidoc/lib/solr_generator.js\nChecking: apidoc/validate.js\nChecking: build/android.js\nChecking: build/androidsdk.js\nChecking: build/ant.js\nChecking: build/docs.js\nChecking: build/git.js\nChecking: build/ios.js\nChecking: build/packager.js\nChecking: build/scons-build.js\nChecking: build/scons-clean.js\nChecking: build/scons-cleanbuild.js\nChecking: build/scons-install.js\nChecking: build/scons-modules-integrity.js\nChecking: build/scons-package.js\nChecking: build/scons-ssri.js\nChecking: build/scons-test.js\nChecking: build/scons-update-node-deps.js\nChecking: build/scons.js\nChecking: build/titanium-sdk/lib/titanium.js\nChecking: build/utils.js\nChecking: build/windows.js\nChecking: cli/commands/build.js\nChecking: cli/commands/clean.js\nChecking: cli/commands/create.js\nChecking: cli/commands/project.js\nChecking: cli/hooks/hyperloop-fix.js\nChecking: cli/lib/creator.js\nChecking: cli/lib/creators/app.js\nChecking: cli/lib/creators/applewatch.js\nChecking: cli/lib/creators/module.js\nChecking: cli/lib/info.js\nChecking: android/cli/commands/_build.js\nChecking: android/cli/commands/_buildModule.js\nChecking: android/cli/hooks/aar-transform.js\nChecking: android/cli/hooks/package.js\nChecking: android/cli/hooks/run.js\nChecking: android/cli/lib/AndroidManifest.js\nChecking: android/cli/lib/base-builder.js\nChecking: android/cli/lib/detect.js\nChecking: android/cli/lib/info.js\nChecking: android/cli/tests/run.js\nChecking: android/cli/tests/test-androidmanifest.js\nChecking: iphone/cli/commands/_build.js\nChecking: iphone/cli/commands/_buildModule.js\nChecking: iphone/cli/hooks/frameworks.js\nChecking: iphone/cli/hooks/hyperloop.js\nChecking: iphone/cli/hooks/install.js\nChecking: iphone/cli/hooks/package.js\nChecking: iphone/cli/hooks/run.js\nChecking: iphone/cli/lib/info.js\nChecking: tests/Resources/ti.android.notificationmanager.addontest.js\nChecking: tests/Resources/ti.api.addontest.js\nChecking: tests/Resources/ti.example.addontest.js\nChecking: tests/Resources/ti.network.httpclient.addontest.js\nChecking: tests/Resources/ti.ui.picker.addontest.js\nChecking: tests/Resources/ti.ui.tableview.addontest.js\nChecking: tests/Resources/ti.ui.webview.addontest.js\nNo vulnerabilities found.\n\nRunning \"retire:node\" (retire) task\nIgnoring {\"paths\":[\"growl\"],\"descriptors\":[]}\nDownloading https://raw.github.com/RetireJS/retire.js/master/repository/npmrepository.json ...\n(node:70622) [DEP0013] DeprecationWarning: Calling an asynchronous function without callback is deprecated.\nChecking: .\nMissing version for babel-eslint. Need to run npm install ?\nMissing version for eslint-plugin-jsx-a11y. Need to run npm install ?\nMissing version for eslint-plugin-react. Need to run npm install ?\nMissing version for babel-eslint. Need to run npm install ?\nMissing version for eslint-plugin-jsx-a11y. Need to run npm install ?\nMissing version for eslint-plugin-react. Need to run npm install ?\nNo vulnerabilities found.\n\nRunning \"continue:off\" task\n\nRunning \"checkFormat:ios\" (checkFormat) task\n\nRunning \"checkFormat:android\" (checkFormat) task\nFatal error: Error: Formatting incorrect on \"android/modules/ui/src/java/ti/modules/titanium/ui/WebViewProxy.java\", proposed changes: <?xml version='1.0'?>\n<replacements xml:space='preserve' incomplete_format='false'>\n<replacement offset='4695' length='1'>&#10;\t\t\t\t\t</replacement>\n<replacement offset='4943' length='1'>&#10;\t\t\t\t\t\t\t\t\t\t\t\t </replacement>\n<replacement offset='5181' length='0'>&#10;\t\t\t\t</replacement>\n<replacement offset='5185' length='1'>&#10;\t\t\t\t</replacement>\n</replacements>\n\u0007\nnpm ERR! Test failed.  See above for more details.\n\n```\n  </td>\n    </tr>\n  </tbody>\n</table>\n\n\n<p align=\"right\">\n  Generated by :no_entry_sign: <a href=\"http://github.com/danger/danger-js/\">dangerJS</a>\n</p>\n"}}
{"actor":{"display_login":"build"}}