ICEmobile
  1. ICEmobile
  2. MOBI-266

<mobi:microphone> causing dom diff failure on desktop

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0 Beta
    • Fix Version/s: 1.1 Beta
    • Component/s: Faces
    • Labels:
      None
    • Environment:
      Tomcat 7, Desktop FF and Chrome

      Description

      Rendering the <mobi:microphone> on a desktop is causing a dom diff failure. Currently seen in trunk and on labs. Not seen in 1.0 demo on .org.

      Steps to reproduce:
      1. open the mobileshowcase with desktop chrome or FF
      2. click on the microphone link
      3. notice that the page does not update, and the following error is seen in the console: SEVERE: Diff propagated to root but no ID set [html: null]

      Note:
      - the <mobi:camcorder> and <mobi:camera> do not exhibit the same problem
      - the <mobi:microphone> did not cause a dom diff failure when tested on the iOS 5.0.1 (iPhone 4)

        Activity

        Philip Breau created issue -
        Hide
        Philip Breau added a comment -

        dom diff logging

        Show
        Philip Breau added a comment - dom diff logging
        Philip Breau made changes -
        Field Original Value New Value
        Attachment dom-diff-logging.txt [ 14722 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #30124 Thu Jul 19 13:34:40 MDT 2012 ted.goddard close span tag for microphone on stock browser (MOBI-266)
        Files Changed
        Commit graph MODIFY /icemobile/trunk/icemobile/jsf/components/component/src/org/icefaces/mobi/component/microphone/MicrophoneRenderer.java
        Hide
        Philip Breau added a comment -

        testing results:
        rev 30045 reproduced
        rev 30021 reproduced
        rev 29903 reproduced
        rev 29830 reproduced
        rev 29375 reproduced
        rev 29161 reproduced
        rev 28688 could not reproduce

        so, the bug must have been introduced between icemobile revs between April 12 and May 23

        Show
        Philip Breau added a comment - testing results: rev 30045 reproduced rev 30021 reproduced rev 29903 reproduced rev 29830 reproduced rev 29375 reproduced rev 29161 reproduced rev 28688 could not reproduce so, the bug must have been introduced between icemobile revs between April 12 and May 23
        Hide
        Ted Goddard added a comment -

        The <span> created by the MicrophoneRenderer was not being closed, so the resulting output (and remainder of the page) would be malformed. The tag is now being closed.

        Show
        Ted Goddard added a comment - The <span> created by the MicrophoneRenderer was not being closed, so the resulting output (and remainder of the page) would be malformed. The tag is now being closed.
        Ted Goddard made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 1.1 Beta [ 10320 ]
        Resolution Fixed [ 1 ]
        Ted Goddard made changes -
        Assignee Steve Maryka [ steve.maryka ] Ted Goddard [ ted.goddard ]
        Hide
        Philip Breau added a comment -

        verified fixed in 30124

        Show
        Philip Breau added a comment - verified fixed in 30124
        Philip Breau made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Ted Goddard
            Reporter:
            Philip Breau
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: