Details

      Description

      Android has supported relative path for window URL since 1.8. Due to PR#3057, it is not supported any more on 3_0_X and master. PR#3057 was trying to fix a regression (TIMOB-10783) which was caused by PR#2027.
      Steps to reproduce:
      Run the attached project.
      Actual result:
      An error shows in the console:

      E/TiAssetHelper(14013): Error while reading asset "Resources/dir/relative2.js":
      E/TiAssetHelper(14013): java.io.FileNotFoundException: Resources/dir/relative2.js
      

      Expected result:
      A blue window opens first, then a red window opens, and finally both windows close. The log shows in the console:

      I/TiAPI   ( 8074):  ************************* relative2.js: win closed
      I/TiAPI   ( 8074):  ************************* relative.js: win closed
      I/TiAPI   ( 8074):  ************************* app.js: win closed
      

      PS: This regression causes the failure of several Anvil tests: ui/ui->absoluteAndRelativeWinURLs, includes/includes->includeFromUrlWindow and includes/includes->secondContextRequire.

        Issue Links

          Activity

          Hide
          Ping Wang added a comment -

          Steps for FR:
          1. Run the attached project "timob11525" and follow the instruction in the ticket.
          2. Run the test cases in TIMOB-10783, TIMOB-11408, TIMOB-11307, TIMOB-5434, TIMOB-10091 and TIMOB-5748.
          3. Run Anvil on both v8 and rhino. ui/ui->absoluteAndRelativeWinURLs, includes/includes->includeFromUrlWindow and includes/includes->secondContextRequire should pass without any new regression.

          Show
          Ping Wang added a comment - Steps for FR: 1. Run the attached project "timob11525" and follow the instruction in the ticket. 2. Run the test cases in TIMOB-10783 , TIMOB-11408 , TIMOB-11307 , TIMOB-5434 , TIMOB-10091 and TIMOB-5748 . 3. Run Anvil on both v8 and rhino. ui/ui->absoluteAndRelativeWinURLs, includes/includes->includeFromUrlWindow and includes/includes->secondContextRequire should pass without any new regression.
          Show
          Ping Wang added a comment - PR https://github.com/appcelerator/titanium_mobile/pull/3321
          Show
          Ping Wang added a comment - 3_0_X PR: https://github.com/appcelerator/titanium_mobile/pull/3356

            People

            • Assignee:
              Ping Wang
              Reporter:
              Ping Wang
            • Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development