ICEmobile
  1. ICEmobile
  2. MOBI-193

Push stops in Android after chasing audio/video play link

    Details

    • Type: Bug Bug
    • Status: Reopened
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.0 Final
    • Fix Version/s: 1.4 Beta
    • Component/s: None
    • Labels:
      None
    • Environment:
      Android/ICEmobile

      Description

      When you chase a video/audio play link in Android, Ajax push stops functioning. This can be reproduced easily with mediacast by uploading an audio or camcorder file, playing it using the 'play' link, returning to the main page and then pushing to client from another device. The push does not show up. This problem exists for the stock browser and the container.

        Activity

        Steve Maryka created issue -
        Hide
        Steve Maryka added a comment -

        I have captured http traffic to the Android device with wireshark. Here is the exact scenario, using mediacast

        • Applications restarted, 2 clients connect - Android and desktop chrome
        • Audio captured and uploaded on Android
        • Push a photo from chrome - push is good
        • Select audio clip from carousel (NOTE: There is another problem here as this does not take to the audio page)
        • Select audio clip again - works this time
        • Play it using the play link
        • Hit back button when audio completes - Returns you to the audio page, but looks like there is network traffic at this point
        • Hit home button to go back to main page
        • Push another image from chrome - push does not work.

        The traffic captured shows:

        • Heartbeat: ice.pushid=1gzlpyc4s%3Ab4k&ice.pushid=1gzlpyc4s%3Ab4m&ice.pushid=1gzlpyc4s%3Afen&ice.pushid=viprlr92 <noop/>
        • Heartbeat: <notified-pushids>viprlr92</notified-pushids>
        • Faces-request: <partial-response><changes><update id="j_idt17:_t18_carousel">... <-- The carousel update (good push)
        • Hearbeat: <-- does not contain <noop/>
        • GET image for carousel
        • Faces-request: 2000<?xml version='1.0' encoding='UTF-8'?><partial-response><changes><update id="javax.faces.ViewRoot">... <-- this is the bad button press. it seems to return the existing page.
        • GET /mediacast/javax.faces.resource/jsf.js.jsf?ln=javax.faces&v=1131669477 HTTP/1.1 <-- fetching JS
        • Hearbeat: <-- does not contain <noop/>
        • Faces-request <partial-response><changes><update id="toolbar"> <-- Second button press. This one works and returns the Viewer page
        • GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf; <-- the audio file
        • GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf;
        • GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf;
        • GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf;
        • GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf;
        • Heartbeat <noop/>
        • Faces-request: <partial-response><changes><update id="toolbar"> <-- Home button pressed. Response includes home page.
        • Heartbeat <noop/> <-- should contain a notification
        • Heartbeat <noop/>

        So the heartbeat continues but the notification never comes.

        Show
        Steve Maryka added a comment - I have captured http traffic to the Android device with wireshark. Here is the exact scenario, using mediacast Applications restarted, 2 clients connect - Android and desktop chrome Audio captured and uploaded on Android Push a photo from chrome - push is good Select audio clip from carousel (NOTE: There is another problem here as this does not take to the audio page) Select audio clip again - works this time Play it using the play link Hit back button when audio completes - Returns you to the audio page, but looks like there is network traffic at this point Hit home button to go back to main page Push another image from chrome - push does not work. The traffic captured shows: Heartbeat: ice.pushid=1gzlpyc4s%3Ab4k&ice.pushid=1gzlpyc4s%3Ab4m&ice.pushid=1gzlpyc4s%3Afen&ice.pushid=viprlr92 <noop/> Heartbeat: <notified-pushids>viprlr92</notified-pushids> Faces-request: <partial-response><changes><update id="j_idt17:_t18_carousel">... <-- The carousel update (good push) Hearbeat: <-- does not contain <noop/> GET image for carousel Faces-request: 2000<?xml version='1.0' encoding='UTF-8'?><partial-response><changes><update id="javax.faces.ViewRoot">... <-- this is the bad button press. it seems to return the existing page. GET /mediacast/javax.faces.resource/jsf.js.jsf?ln=javax.faces&v=1131669477 HTTP/1.1 <-- fetching JS Hearbeat: <-- does not contain <noop/> Faces-request <partial-response><changes><update id="toolbar"> <-- Second button press. This one works and returns the Viewer page GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf; <-- the audio file GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf; GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf; GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf; GET /mediacast/javax.faces.resource/aef90e68-e074-4d41-a4d1-c862518c6f13.jsf; Heartbeat <noop/> Faces-request: <partial-response><changes><update id="toolbar"> <-- Home button pressed. Response includes home page. Heartbeat <noop/> <-- should contain a notification Heartbeat <noop/> So the heartbeat continues but the notification never comes.
        Hide
        Steve Maryka added a comment -

        The wireshark capture.

        Show
        Steve Maryka added a comment - The wireshark capture.
        Steve Maryka made changes -
        Field Original Value New Value
        Attachment android.lidpcap [ 14198 ]
        Steve Maryka made changes -
        Assignee Steve Maryka [ steve.maryka ] Ted Goddard [ ted.goddard ]
        Hide
        Ted Goddard added a comment -

        listen.icepush requests do not seem to continue after the link is played.

        Show
        Ted Goddard added a comment - listen.icepush requests do not seem to continue after the link is played.
        Hide
        Ted Goddard added a comment -

        It's not yet certain if the push connection is suspended during video playback or what events might be available to restart it when the user stops watching the video.

        Show
        Ted Goddard added a comment - It's not yet certain if the push connection is suspended during video playback or what events might be available to restart it when the user stops watching the video.
        Ted Goddard made changes -
        Assignee Ted Goddard [ ted.goddard ] Mircea Toma [ mircea.toma ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28554 Tue Mar 27 10:16:29 MDT 2012 mircea.toma MOBI-193 Reload page when the application resumes activity from a paused state to ensure that ICEFaces bridge is initialized and registers for push notifications with the ICEpush bridge.
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/client/android/container/src/org/icemobile/client/android/ICEmobileContainer.java
        Hide
        Mircea Toma added a comment -

        Reload page when the application resumes activity from a paused state to ensure that ICEFaces bridge is initialized and registers for push notifications with the ICEpush bridge.
        Previously only 'ice.push.connection.resumeConnection()' function was invoked which started the blocking connection but the page was not listening for push events.

        Show
        Mircea Toma added a comment - Reload page when the application resumes activity from a paused state to ensure that ICEFaces bridge is initialized and registers for push notifications with the ICEpush bridge. Previously only 'ice.push.connection.resumeConnection()' function was invoked which started the blocking connection but the page was not listening for push events.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Steve Maryka added a comment -

        A page reload seems heavy handed. The page is completely functional without a reload - except for push connection. A page load is very disruptive to the user experience.

        Show
        Steve Maryka added a comment - A page reload seems heavy handed. The page is completely functional without a reload - except for push connection. A page load is very disruptive to the user experience.
        Hide
        Mircea Toma added a comment -

        Obviously the page is not completely functional otherwise we wouldn't have this issue. We could initialize the ICEfaces bridge programmatically but I don't know if the container has access to the ICEfaces bridge configuration or viewID of the page.

        Show
        Mircea Toma added a comment - Obviously the page is not completely functional otherwise we wouldn't have this issue. We could initialize the ICEfaces bridge programmatically but I don't know if the container has access to the ICEfaces bridge configuration or viewID of the page.
        Hide
        Steve Maryka added a comment -

        Fixing it in the container still leaves the stock browser with an issue, so that is not a great option. This issue seems specific to Android, so why is it not a problem on other platforms?

        Show
        Steve Maryka added a comment - Fixing it in the container still leaves the stock browser with an issue, so that is not a great option. This issue seems specific to Android, so why is it not a problem on other platforms?
        Hide
        Mircea Toma added a comment -

        I understand that we have a problem in stock browser. Unfortunately , while debugging the app, I discovered that none of the page events are fired when returning to the page. So, 'onpageshow' or 'onload' will not fire hence the failure to fully setup the page.

        Show
        Mircea Toma added a comment - I understand that we have a problem in stock browser. Unfortunately , while debugging the app, I discovered that none of the page events are fired when returning to the page. So, 'onpageshow' or 'onload' will not fire hence the failure to fully setup the page.
        Hide
        Ted Goddard added a comment -

        It may be possible to use a history listener to restart the ICEfaces and ICEpush bridges.

        Show
        Ted Goddard added a comment - It may be possible to use a history listener to restart the ICEfaces and ICEpush bridges.
        Ted Goddard made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28611 Mon Apr 02 09:09:21 MDT 2012 mircea.toma MOBI-193 Revert previous change. Modified audio renderer so that when the 'Play' link is clicked a new window is opened to avoid shutting down the bridge belonging to the current page.
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/components/component/src/org/icefaces/mobi/component/audio/AudioRenderer.java
        Commit graph MODIFY /icemobile/trunk/icemobile/client/android/container/src/org/icemobile/client/android/ICEmobileContainer.java
        Hide
        Mircea Toma added a comment -

        Reverted previous fix.
        The new fix modifies the audio renderer so that when the 'Play' link is clicked a new window is opened. This way the bridge belonging to the current page is not shut down. This solution work also on stock browser.

        Show
        Mircea Toma added a comment - Reverted previous fix. The new fix modifies the audio renderer so that when the 'Play' link is clicked a new window is opened. This way the bridge belonging to the current page is not shut down. This solution work also on stock browser.
        Mircea Toma made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Ted Goddard added a comment -

        Similar change needs to be applied to VideoRenderer.

        Show
        Ted Goddard added a comment - Similar change needs to be applied to VideoRenderer.
        Ted Goddard made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Assignee Mircea Toma [ mircea.toma ] Judy Guglielmin [ judy.guglielmin ]
        Ted Goddard made changes -
        Fix Version/s EE 1.0 Final [ 10318 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #28618 Mon Apr 02 14:06:16 MDT 2012 judy.guglielmin MOBI-193
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/components/component/src/org/icefaces/mobi/component/video/VideoPlayerRenderer.java
        Hide
        Steve Maryka added a comment - - edited

        My current testing indicates that the stock browser on Android works fine with both Audio and Video links. Push keeps working after, but on Android container, push stops working after playing either audio or video links.

        Show
        Steve Maryka added a comment - - edited My current testing indicates that the stock browser on Android works fine with both Audio and Video links. Push keeps working after, but on Android container, push stops working after playing either audio or video links.
        Migration made changes -
        Fix Version/s 1.2 Beta [ 10344 ]
        Fix Version/s EE 1.0 Final [ 10318 ]
        Migration made changes -
        Assignee Judy Guglielmin [ judy.guglielmin ] User Philip.breau [ philip.breau ]
        Hide
        Steve Maryka added a comment -

        I am going to retest in detail to see where we stand on this issue.

        Show
        Steve Maryka added a comment - I am going to retest in detail to see where we stand on this issue.
        Steve Maryka made changes -
        Assignee User Philip.breau [ philip.breau ] Steve Maryka [ steve.maryka ]
        Fix Version/s 1.2 Final [ 10371 ]
        Fix Version/s 1.2 Beta [ 10344 ]
        Steve Maryka made changes -
        Fix Version/s 1.2 EE [ 10372 ]
        Fix Version/s 1.2 Final [ 10371 ]
        Steve Maryka made changes -
        Assignee Steve Maryka [ steve.maryka ] Jack Van Ooststroom [ jack.van.ooststroom ]
        Hide
        Steve Maryka added a comment -

        I confirmed that this is still broken for both Audio and Video links. The container pauses and resumes the blocking connection as expected, but the blocking connection does not resume when returning from the Audio/Video link.

        Show
        Steve Maryka added a comment - I confirmed that this is still broken for both Audio and Video links. The container pauses and resumes the blocking connection as expected, but the blocking connection does not resume when returning from the Audio/Video link.
        Ted Goddard made changes -
        Assignee Jack Van Ooststroom [ jack.van.ooststroom ] Ted Goddard [ ted.goddard ]
        Hide
        Ted Goddard added a comment -

        Testing on Nexus 7 tablet confirms that the blocking connection is not resumed after the video link is played.

        Show
        Ted Goddard added a comment - Testing on Nexus 7 tablet confirms that the blocking connection is not resumed after the video link is played.
        Hide
        Ted Goddard added a comment -

        The web console output indicates that ICEpush is attempting to connect with the server, but is receiving an error response. However, network monitoring shows no such HTTP requests.

        Show
        Ted Goddard added a comment - The web console output indicates that ICEpush is attempting to connect with the server, but is receiving an error response. However, network monitoring shows no such HTTP requests.
        Hide
        Ted Goddard added a comment -

        postAsynchronously() is not invoked by the icepush JavaScript as isEmpty(lastSentPushIds) returns true after the video link is played.

        Show
        Ted Goddard added a comment - postAsynchronously() is not invoked by the icepush JavaScript as isEmpty(lastSentPushIds) returns true after the video link is played.
        Hide
        Ted Goddard added a comment -

        The request for the video file from the container causes the pushids cookie to be cleared (notice the "ice.pushids="):

        GET /mobileshowcase/javax.faces.resource/945495df-801a-4f79-8fa9-9a017344edb5.jsf;jsessionid=6A51B4309586E2DFA54135357D33AE76 HTTP/1.1
        Host: tetra.ice:8080
        Connection: keep-alive
        Accept: text/html,application/xhtml+xml,application/xml;q=0.9,/;q=0.8
        X-Requested-With: org.icemobile.client.android
        User-Agent: Mozilla/5.0 (Linux; U; Android 4.2.1; en-us; Nexus 7 Build/JOP40D) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Safari/534.30
        Accept-Encoding: gzip,deflate
        Accept-Language: en-US
        Accept-Charset: utf-8, iso-8859-1, utf-16, *;q=0.7
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=6A51B4309586E2DFA54135357D33AE76; com.icesoft.user-agent=HyperBrowser/1.0; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=2hbzmz0j7; ice.connection.contextpath=%2Fmobileshowcase; ice.connection.running=f35ed:acquired; ice.connection.lease=1358292382056; ice.pushids=

        Show
        Ted Goddard added a comment - The request for the video file from the container causes the pushids cookie to be cleared (notice the "ice.pushids="): GET /mobileshowcase/javax.faces.resource/945495df-801a-4f79-8fa9-9a017344edb5.jsf;jsessionid=6A51B4309586E2DFA54135357D33AE76 HTTP/1.1 Host: tetra.ice:8080 Connection: keep-alive Accept: text/html,application/xhtml+xml,application/xml;q=0.9, / ;q=0.8 X-Requested-With: org.icemobile.client.android User-Agent: Mozilla/5.0 (Linux; U; Android 4.2.1; en-us; Nexus 7 Build/JOP40D) AppleWebKit/534.30 (KHTML, like Gecko) Version/4.0 Safari/534.30 Accept-Encoding: gzip,deflate Accept-Language: en-US Accept-Charset: utf-8, iso-8859-1, utf-16, *;q=0.7 Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=6A51B4309586E2DFA54135357D33AE76; com.icesoft.user-agent=HyperBrowser/1.0; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=2hbzmz0j7; ice.connection.contextpath=%2Fmobileshowcase; ice.connection.running=f35ed:acquired; ice.connection.lease=1358292382056; ice.pushids=
        Hide
        Ted Goddard added a comment -

        delistPushIDsWithBrowser() is being invoked when the video is played resulting in the list of pushIDs being empty.

        Show
        Ted Goddard added a comment - delistPushIDsWithBrowser() is being invoked when the video is played resulting in the list of pushIDs being empty.
        Hide
        Ted Goddard added a comment -

        When the video is displayed, onBeforeUnload is invoked

        onBeforeUnload(window, function()

        { delistPushIDsWithBrowser(pushIdentifiers); pushIdentifiers = []; }

        );

        However, the page is not reloaded when the video completes.

        Show
        Ted Goddard added a comment - When the video is displayed, onBeforeUnload is invoked onBeforeUnload(window, function() { delistPushIDsWithBrowser(pushIdentifiers); pushIdentifiers = []; } ); However, the page is not reloaded when the video completes.
        Ted Goddard made changes -
        Assignee Ted Goddard [ ted.goddard ] Steve Maryka [ steve.maryka ]
        Hide
        Ted Goddard added a comment -

        Is it possible for the container to reload the current page when the video or audio playback completes? I believe this would cause ICEpush to resume correctly.

        Show
        Ted Goddard added a comment - Is it possible for the container to reload the current page when the video or audio playback completes? I believe this would cause ICEpush to resume correctly.
        Hide
        Philip Breau added a comment -

        As this only seems to happen when you run video in full screen mode, perhaps we could use the media events to check if the page does need to be refreshed:

        function checkIfPushIdsEnlisted(elem)

        { var pushIdentifiers = lookUpPushIdsFromCookie(); elem.setAttribute('data-pushIds', pushIdentifiers); }

        function refreshIfPushLost(elem){
        var pushIdentifiers = lookUpPushIdsFromCookie();
        if( elem.getAttribute('data-pushIds') && !pushIdentifiers )

        { window.href.reload(); }

        }
        video.addEventListener("loadeddata", checkIfPushIdsEnlisted, false);
        video.addEventListener("ended", refreshIfPushLost, false);

        Show
        Philip Breau added a comment - As this only seems to happen when you run video in full screen mode, perhaps we could use the media events to check if the page does need to be refreshed: function checkIfPushIdsEnlisted(elem) { var pushIdentifiers = lookUpPushIdsFromCookie(); elem.setAttribute('data-pushIds', pushIdentifiers); } function refreshIfPushLost(elem){ var pushIdentifiers = lookUpPushIdsFromCookie(); if( elem.getAttribute('data-pushIds') && !pushIdentifiers ) { window.href.reload(); } } video.addEventListener("loadeddata", checkIfPushIdsEnlisted, false); video.addEventListener("ended", refreshIfPushLost, false);
        Hide
        Ted Goddard added a comment -

        In this case the video is just a link to a file – would there be a video object available to JavaScript?

        I also wonder if the page containing the link is actually "unloaded" at this point. The video playback may actually be a popup window (in that case the original unload even should never have been called).

        Show
        Ted Goddard added a comment - In this case the video is just a link to a file – would there be a video object available to JavaScript? I also wonder if the page containing the link is actually "unloaded" at this point. The video playback may actually be a popup window (in that case the original unload even should never have been called).
        Hide
        Philip Breau added a comment -

        No, using the video object wouldn't work for clicking the external link.

        onbeforeunload is called as soon as you click the link. And unfortunately onload is not called when returning to the page after the video finishes.

        Show
        Philip Breau added a comment - No, using the video object wouldn't work for clicking the external link. onbeforeunload is called as soon as you click the link. And unfortunately onload is not called when returning to the page after the video finishes.
        Hide
        Philip Breau added a comment -

        What about just not rendering the 'Play' external link on the android container. We already do the same for iOS. We can note it as a known issue and users can always add their own link manually.

        Show
        Philip Breau added a comment - What about just not rendering the 'Play' external link on the android container. We already do the same for iOS. We can note it as a known issue and users can always add their own link manually.
        Hide
        Ted Goddard added a comment -

        Using video conversion on labs the behavior on the iPad is improved, but playback is still not successful. The conversion script will need to be updated.

        Show
        Ted Goddard added a comment - Using video conversion on labs the behavior on the iPad is improved, but playback is still not successful. The conversion script will need to be updated.
        Hide
        Ted Goddard added a comment -

        It may be possible to allow multiple windows and avoid the onBeforeUnload (since the video link has a target="_blank" attribute):

        WebSettings settings = webView.getSettings();
        settings.setSupportMultipleWindows(true);

        http://stackoverflow.com/questions/8578332/webview-webchromeclient-method-oncreatewindow-not-called-for-target-blank

        Show
        Ted Goddard added a comment - It may be possible to allow multiple windows and avoid the onBeforeUnload (since the video link has a target="_blank" attribute): WebSettings settings = webView.getSettings(); settings.setSupportMultipleWindows(true); http://stackoverflow.com/questions/8578332/webview-webchromeclient-method-oncreatewindow-not-called-for-target-blank
        Hide
        Steve Maryka added a comment -

        The above suggestion did not work. WebChromeClient.onWindowCreated() is never called. Can't find any threads on the web that solves the problem.

        If we could modify the URL to the video resource to include something uniquely recognizable, we might be able to use WebViewClient.shouldOverrideUrlLoading() to do something different.

        Once the video starts playing, the container does an onPause which is calling pauseBlockingConnection, but it is already gone because of the page unload. Maybe pauseBlockingConnection could be smarter.

        Show
        Steve Maryka added a comment - The above suggestion did not work. WebChromeClient.onWindowCreated() is never called. Can't find any threads on the web that solves the problem. If we could modify the URL to the video resource to include something uniquely recognizable, we might be able to use WebViewClient.shouldOverrideUrlLoading() to do something different. Once the video starts playing, the container does an onPause which is calling pauseBlockingConnection, but it is already gone because of the page unload. Maybe pauseBlockingConnection could be smarter.
        Ted Goddard made changes -
        Fix Version/s 1.3 Beta [ 10373 ]
        Fix Version/s 1.2 EE [ 10372 ]
        Hide
        Ted Goddard added a comment -

        A few solutions are possible:

        • modify Android container so that onBeforeUnload is not called for full screen video playback
        • modify Android container so that page is reloaded when full screen video playback completes
        • modify ICEpush so that the cookie is not cleared in onBeforeUnload. For instance, on Android WebKit client-side storage could be used and the current list of PUSHIDs should be built up from the current PUSHIDs in each JavaScript memory space in each responding window
        Show
        Ted Goddard added a comment - A few solutions are possible: modify Android container so that onBeforeUnload is not called for full screen video playback modify Android container so that page is reloaded when full screen video playback completes modify ICEpush so that the cookie is not cleared in onBeforeUnload. For instance, on Android WebKit client-side storage could be used and the current list of PUSHIDs should be built up from the current PUSHIDs in each JavaScript memory space in each responding window
        Hide
        Steve Maryka added a comment -

        I have not been able to find any client side mechanism to reliably detect when VideoViewer is activated to play a video or audio clip. The WebChromeClient.onShowCustomView() is the mechanism that is supposed to be used, but apparently it does not work as of 4.0. We need to consider an ICEpush-based approach. The container is diligently pausing and resuming the blocking connection when the video plays.

        Show
        Steve Maryka added a comment - I have not been able to find any client side mechanism to reliably detect when VideoViewer is activated to play a video or audio clip. The WebChromeClient.onShowCustomView() is the mechanism that is supposed to be used, but apparently it does not work as of 4.0. We need to consider an ICEpush-based approach. The container is diligently pausing and resuming the blocking connection when the video plays.
        Steve Maryka made changes -
        Assignee Steve Maryka [ steve.maryka ] Ted Goddard [ ted.goddard ]
        Hide
        Ted Goddard added a comment -

        ice.setupPush('vgkhrkf9');
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkf9

        POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf
        ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkf9

        GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.connection.contextpath=%2Fmediacast; ice.connection.running=2a042:acquired; ice.connection.lease=1364248744765; ice.pushids=vxa54j65%20vxa54j68

        POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf
        ice.pushid=vxa54j65&ice.pushid=vxa54j68
        X-Connection: close
        X-Connection-reason: duplicate

        GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B
        User-Agent: stagefright/1.2 (Linux;Android 4.2.1)

        POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.connection.lease=1364248744765; ice.pushids=vxa54j65%20vxa54j68;
        ice.pushid=vxa54j65&ice.pushid=vxa54j68
        X-Connection: close
        X-Connection-reason: duplicate

        listen requests continue with ice.pushid=vxa54j65&ice.pushid=vxa54j68

        Show
        Ted Goddard added a comment - ice.setupPush('vgkhrkf9'); Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkf9 POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkf9 GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.connection.contextpath=%2Fmediacast; ice.connection.running=2a042:acquired; ice.connection.lease=1364248744765; ice.pushids=vxa54j65%20vxa54j68 POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf ice.pushid=vxa54j65&ice.pushid=vxa54j68 X-Connection: close X-Connection-reason: duplicate GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B User-Agent: stagefright/1.2 (Linux;Android 4.2.1) POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.connection.lease=1364248744765; ice.pushids=vxa54j65%20vxa54j68; ice.pushid=vxa54j65&ice.pushid=vxa54j68 X-Connection: close X-Connection-reason: duplicate listen requests continue with ice.pushid=vxa54j65&ice.pushid=vxa54j68
        Hide
        Ted Goddard added a comment -

        Adding a button to the page that prevents pushIDs from being cleared during onunload:

        ice.setupPush('vgkhrkfa');

        POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa;
        ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkfa

        GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa;

        GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B
        User-Agent: stagefright/1.2 (Linux;Android 4.2.1)

        POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa;
        ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkfa
        X-Connection: close
        X-Connection-reason: duplicate

        POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf
        Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa;
        ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkfa
        X-Connection: close
        X-Connection-reason: duplicate

        (I was originally recording this traffic to analyze the problem; on the last test push appears to continue to function, so it may be due to the device sleeping in between, or just inconsistent testing on my part.)

        Show
        Ted Goddard added a comment - Adding a button to the page that prevents pushIDs from being cleared during onunload: ice.setupPush('vgkhrkfa'); POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa; ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkfa GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa; GET /mediacast/javax.faces.resource/43785373-60f3-4727-964d-64a3b2a64bed.jsf;jsessionid=D3D49F70A9150E26876EA5D847DBF08B User-Agent: stagefright/1.2 (Linux;Android 4.2.1) POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa; ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkfa X-Connection: close X-Connection-reason: duplicate POST /mediacast/javax.faces.resource/listen.icepush.xml.jsf Cookie: com.icesoft.user-agent=HyperBrowser/1.0; JSESSIONID=D3D49F70A9150E26876EA5D847DBF08B; com.icesoft.user-agent=HyperBrowser/1.0; ice.push.browser=1heq42673; com.icesoft.user-agent=HyperBrowser/1.0; ice.pushids=vxa54j65%20vxa54j68%20vgkhrkfa; ice.pushid=vxa54j65&ice.pushid=vxa54j68&ice.pushid=vgkhrkfa X-Connection: close X-Connection-reason: duplicate (I was originally recording this traffic to analyze the problem; on the last test push appears to continue to function, so it may be due to the device sleeping in between, or just inconsistent testing on my part.)
        Hide
        Ted Goddard added a comment -

        Experimentation with an ICEpush feature that prevents the unload from clearing the cookies does seem to help, but unfortunately the container onPause() is invoked after the page is unloaded.

        Show
        Ted Goddard added a comment - Experimentation with an ICEpush feature that prevents the unload from clearing the cookies does seem to help, but unfortunately the container onPause() is invoked after the page is unloaded.
        Hide
        Ted Goddard added a comment -

        It is likely necessary to define a new mechanism for clearing unused pushIDs. Disabling cleanup during onUnload allows push to continue functioning after the video is played, but it also results in an accumulation of old pushIDs. However, there are other cases where unused pushIDs accumulate, so an improved cleanup mechanism should be investigated.

        Show
        Ted Goddard added a comment - It is likely necessary to define a new mechanism for clearing unused pushIDs. Disabling cleanup during onUnload allows push to continue functioning after the video is played, but it also results in an accumulation of old pushIDs. However, there are other cases where unused pushIDs accumulate, so an improved cleanup mechanism should be investigated.
        Hide
        Ted Goddard added a comment -

        Each browser window/tab has a set of pushIDs with active listeners. When a listen.icepush returns with a set of notifications, these need to be dispatched to the various browser windows via two mechanisms: local storage events and cookie polling. When a notification occurs (including a noop) each pushID in each window can have an associated timestamp so that the listening window can determine if a given pushID is still active. pushIDs with very old associated timestamps can be garbage collected.

        Show
        Ted Goddard added a comment - Each browser window/tab has a set of pushIDs with active listeners. When a listen.icepush returns with a set of notifications, these need to be dispatched to the various browser windows via two mechanisms: local storage events and cookie polling. When a notification occurs (including a noop) each pushID in each window can have an associated timestamp so that the listening window can determine if a given pushID is still active. pushIDs with very old associated timestamps can be garbage collected.
        Ted Goddard made changes -
        Fix Version/s 1.4 Beta [ 10874 ]
        Fix Version/s 1.3 Beta [ 10373 ]
        Hide
        Ted Goddard added a comment -

        This is an important fix so should be considered for 1.3EE.

        Show
        Ted Goddard added a comment - This is an important fix so should be considered for 1.3EE.

          People

          • Assignee:
            Ted Goddard
            Reporter:
            Steve Maryka
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated: