JTAF-ExtWebDriver icon indicating copy to clipboard operation
JTAF-ExtWebDriver copied to clipboard

NPE @DefaultExtWebDriver.java line #295

Open mibrahim opened this issue 10 years ago • 1 comments

if (currentWindowIds == null && currentWindowIds.size() > 0)

Fixed to

if (currentWindowIds != null && currentWindowIds.size() > 0)

mibrahim avatar Jul 19 '14 15:07 mibrahim

Actually, seems not logical at all. I changed to the code to: if (currentWindowIds == null) { throw new NullPointerException( "WebDriver returned a null set of WindowIds to storeCurrentWindowIds()"); }

Removed the unreachable throw new StaleWindowIdListException .

Someone needs to double verify the original intention of the code !!

mibrahim avatar Jul 19 '14 16:07 mibrahim