search

google-code-export / gwt-platform

Automatically exported from code.google.com/p/gwt-platform
1 stars 0 forks source link

Logic bug in PopupViewImpl.setAutoHideOnNavigationEventEnabled() #452

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.Visual inspection of code: 
https://github.com/ArcBees/GWTP/blob/master/gwtp-core/gwtp-mvp-client/src/main/j
ava/com/gwtplatform/mvp/client/PopupViewImpl.java (line 92)

What is the expected output? What do you see instead?
There appears to be a bug in the logic of the 
setAutoHideOnNavigationEventEnabled() method.
Specifically, the 'autoHideHandler' variable is not being cleared.
Suggest adding 'autoHideHandler = null;' at line 93.

What version of the product are you using? On what operating system?
1.0, Linux

Original issue reported on code.google.com by peter.le...@freshcode.biz on 2 Jul 2013 at 1:46