ICEpush
  1. ICEpush
  2. PUSH-343

Push notifications stop when third window is opened

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.1, EE-3.3.0.GA_P04
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      ICEpush

      Description

      Push notifications stop when a third window is opened. The notifications stop in one or two of the windows. When and which windows have their notifications stopped seems to be random.

        Activity

        Mircea Toma created issue -
        Mircea Toma made changes -
        Field Original Value New Value
        Assignee Mircea Toma [ mircea.toma ]
        Mircea Toma made changes -
        Affects Version/s 4.0 [ 11383 ]
        Mircea Toma made changes -
        Fix Version/s 4.1 [ 11570 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43620 Thu Nov 27 10:56:14 MST 2014 mircea.toma PUSH-343 Fixed LocalStorageNotificationBroadcaster unreliability by reading localStorage new values off the event object instead of reading it from the storage (since other thread can update the value again in the mean time). Factored out PushIDExpiryMonitor to monitor for unused pushIDs. The monitor runs off a thread, every 10 seconds. Improved logging.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/pushid.expiry.js
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/inter.window.notification.js
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/application.js
        Hide
        Mircea Toma added a comment -

        Fixed LocalStorageNotificationBroadcaster unreliability by reading localStorage new values off the event object instead of reading it from the storage (since other thread can update the value again in the mean time). Rolled back the change that ran local storage updates spread out at regular intervals (not required anymore). Factored out PushIDExpiryMonitor to monitor for unused pushIDs. The monitor runs off a thread, every 10 seconds. Improved logging.

        Show
        Mircea Toma added a comment - Fixed LocalStorageNotificationBroadcaster unreliability by reading localStorage new values off the event object instead of reading it from the storage (since other thread can update the value again in the mean time). Rolled back the change that ran local storage updates spread out at regular intervals (not required anymore). Factored out PushIDExpiryMonitor to monitor for unused pushIDs. The monitor runs off a thread, every 10 seconds. Improved logging.
        Mircea Toma made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Mircea Toma added a comment -

        Tested functionality with Safari, Firefox, Chrome, IE10 and IE11. push-test with 10 windows opened ran without hick-ups in Safari, Firefox and Chrome. IE10 becomes unstable after opening the 5th window, IE11 after the 8th.

        Show
        Mircea Toma added a comment - Tested functionality with Safari, Firefox, Chrome, IE10 and IE11. push-test with 10 windows opened ran without hick-ups in Safari, Firefox and Chrome. IE10 becomes unstable after opening the 5th window, IE11 after the 8th.
        Hide
        Liana Munroe added a comment -

        Tested with ICEfaces 4 trunk r43625.
        Test used - showcase icecore:push demo in 4 individual browser windows opened (not 4 tabs).
        FF 33 - OK
        Chrome 38 - OK
        IE 11 - OK (there is still the double submit issue remaining, ICE-10384 is open.)
        IE 10 - updates become unstable after opening the 4th window. This was tested on 2 different machines. Results are comparable. Updates stop for a moment then restart.

        Test used - QA push-test app in 6 individual browser windows opened (not 6 tabs).
        FF 33 - OK
        Chrome 38 - OK
        IE 11 - OK
        IE 10 - updates become unstable after opening the 4th, 5th and 6th windows. New windows are slow to start the timer, perviously opened windows stop updates for several seconds then occasionally resume.

        Show
        Liana Munroe added a comment - Tested with ICEfaces 4 trunk r43625. Test used - showcase icecore:push demo in 4 individual browser windows opened (not 4 tabs). FF 33 - OK Chrome 38 - OK IE 11 - OK (there is still the double submit issue remaining, ICE-10384 is open.) IE 10 - updates become unstable after opening the 4th window. This was tested on 2 different machines. Results are comparable. Updates stop for a moment then restart. Test used - QA push-test app in 6 individual browser windows opened (not 6 tabs). FF 33 - OK Chrome 38 - OK IE 11 - OK IE 10 - updates become unstable after opening the 4th, 5th and 6th windows. New windows are slow to start the timer, perviously opened windows stop updates for several seconds then occasionally resume.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #43874 Tue Dec 30 15:36:57 MST 2014 mircea.toma PUSH-343 Replace console usage with logger.
        Files Changed
        Commit graph MODIFY /icepush/trunk/icepush/core/src/main/javascript/pushid.expiry.js
        Ken Fyten made changes -
        Resolution Fixed [ 1 ]
        Status Resolved [ 5 ] Reopened [ 4 ]
        Ken Fyten made changes -
        Fix Version/s EE-3.3.0.GA_P04 [ 12271 ]
        Ken Fyten made changes -
        Status Reopened [ 4 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Mircea Toma
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: