joseph/Monocle — Issue #79: null pointer exception causes all browsers to report needing force-columns
Author: spacemoose2020
Created: 2012-01-13 21:06:32+00:00
State: open

First of all, thanks for the great work here. Fiendishly brilliant!

On line 95 of env.js (essentially, if the frame doesn't need re-loading), frameCallback is invoked without the testFrame argument. In my case, this is causing an NPE on line 342, and force-columns falls into the runNextTest try..catch handler, which causes API["force-columns"] to equal the non_object_property_load error. This is still non-null, so in a lot of tests, columns.js assumes it needs to do the dreaded minWidth 200%.

Simple fix is for line env.js / 95 :

return frameLoadCallback(testFrame);

Have to assume other tests might be hitting this as well?