GitHub user sarangan12 opened a pull request:

    https://github.com/apache/cordova-plugin-geolocation/pull/70

    CB-10574: MobileSpec can't get results for WP8.1 Builds

    The MobileSpec can't get results for WP8.1 Builds. 
    
    **Why?**
    Because, one of the tests in the geolocation plugin is failing and causes 
the app to crash. This test gets the "0x80004004 - JavaScript runtime error: 
Operation aborted" error. So, the app crashes. The issue around this test must 
be fixed for the CI to finish successfully for Windows Phone 8.1. 
    
    **What is the scenario for the crash?**
    The scenario could be seen manually also. When you run the mobilespec test 
for geolocatio API, try the following steps:
    
    1. Start Watching 
    2. Stop Watching
    3. Start Watching
    4. Start Watching 
    
    The app will crash in step 4. But it will not crash on step 3.  
    
    **Why does the app crash on step 4?**
    
    Per code, the loc object (instance of 
Windows.Devices.Geolocation.Geolocator object) is being reused in both the 
steps - which causes the crash. 
    
    **What is the fix?**
    
    The current fix is to create  a new instance of the object instead of 
resuing the same object. I have validated the fix by running the geolocation 
tests and all of them are passing. 


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sarangan12/cordova-plugin-geolocation CB-10574

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cordova-plugin-geolocation/pull/70.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #70
    
----
commit fbe3bdaef60155cefe2cf789cc62cb27e5a8a3af
Author: Sarangan Rajamanickam <saraj...@microsoft.com>
Date:   2016-02-12T00:19:04Z

    CB-10574: MobileSpec can't get results for WP8.1 Builds
    
    MobileSpec can't get results for WP8.1 Builds

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@cordova.apache.org
For additional commands, e-mail: dev-h...@cordova.apache.org

Reply via email to