Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

iPad Reset Freeze #185

Closed
KatieWoe opened this issue Feb 28, 2020 · 6 comments
Closed

iPad Reset Freeze #185

KatieWoe opened this issue Feb 28, 2020 · 6 comments
Labels

Comments

@KatieWoe
Copy link
Contributor

For ES6 Testing.
Seen on iPad 13 and 12, but not on Win 10.
When you hit the reset all button the sim freezes. Looking at the console you see an error that reads:

Error: Assertion failed: describedby association already registed
@zepumph
Copy link
Member

zepumph commented Feb 28, 2020

Thanks!

@zepumph
Copy link
Member

zepumph commented Feb 29, 2020

I can use this in correlation with phetsims/chipper#845 to help debug the case

Index: friction/js/friction/view/FrictionScreenView.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- friction/js/friction/view/FrictionScreenView.js	(revision 3bfbb998f8890d32a92d5cdb824848abff8618c3)
+++ friction/js/friction/view/FrictionScreenView.js	(date 1582919813685)
@@ -12,6 +12,7 @@
 import ScreenView from '../../../../joist/js/ScreenView.js';
 import inherit from '../../../../phet-core/js/inherit.js';
 import ResetAllButton from '../../../../scenery-phet/js/buttons/ResetAllButton.js';
+import DebugLoggerNode from '../../../../scenery-phet/js/DebugLoggerNode.js';
 import ThermometerNode from '../../../../scenery-phet/js/ThermometerNode.js';
 import SoundClip from '../../../../tambo/js/sound-generators/SoundClip.js';
 import SoundLevelEnum from '../../../../tambo/js/SoundLevelEnum.js';
@@ -69,6 +70,9 @@
     screenSummaryContent: frictionScreenSummaryNode
   } );
 
+  phet.debugLoggerNode = new DebugLoggerNode();
+  this.addChild( phet.debugLoggerNode );
+
   // @private
   this.model = model;
 
Index: scenery/js/accessibility/Accessibility.js
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- scenery/js/accessibility/Accessibility.js	(revision 07da7de8a96ffda07bc8c45453c94f0efe9d97e8)
+++ scenery/js/accessibility/Accessibility.js	(date 1582919884495)
@@ -1737,6 +1737,8 @@
         assert && AccessibilityUtils.validateAssociationObject( associationObject );
         assert && assert( !_.includes( this._ariaDescribedbyAssociations, associationObject ), 'describedby association already registed' );
 
+
+        !_.includes( this._ariaDescribedbyAssociations, associationObject ) && phet.debugLoggerNode.log( new Error().stack );
         this._ariaDescribedbyAssociations.push( associationObject ); // Keep track of this association.
 
         // Flag that this node is is being described by the other node, so that if the other node changes it can tell

@zepumph
Copy link
Member

zepumph commented Oct 8, 2021

I cannot reproduce this anymore. When I use an iPad all is good with the reset all button. QA, if you cannot reproduce this during the soon-to-come dev test, please close.

@zepumph zepumph removed their assignment Oct 8, 2021
@zepumph
Copy link
Member

zepumph commented Oct 8, 2021

Scratch that. I improved my error handling and I see things clearer now.

@zepumph zepumph self-assigned this Oct 8, 2021
@zepumph
Copy link
Member

zepumph commented Oct 8, 2021

Ok. I did a bit more poking around, and my previous thoughts were a false positive. QA, if you can't reproduce, please close.

@KatieWoe
Copy link
Contributor Author

Looks ok from what I can see.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants