#7 Error on attempt to access content of null cleanHistoryFrame

[Stephen Keller]

I'm using MSIE with Clean AJAX version 4.3 and ran into a bug on line 47 of
the HistoryTool.js.

The code attempts to access window.cleanHistoryFrame.contentWindow.document
before checking whether window.cleanHistoryFrame is not null. I moved line
57 into the immediately following if(...) block and eliminated an error in
MSIE. It's not clear why the cleanHistoryFrame is null in IE but not in
Firefox under the same circumstances, and I'm still investigating that
issue. Nevertheless, the attempt to access the content of the
cleanHistoryFrame object needs to move inside the conditional either way.

Original code (line 56 on):
var iterator = new DomIterator(doc);
var cache = window.cleanHistoryFrame.contentWindow.document;
if(window.cleanHistoryFrame != null) {
// Form cache
// ...

New code (line 56 on):
var iterator = new DomIterator(doc);
if(window.cleanHistoryFrame != null) {
var cache = window.cleanHistoryFrame.contentWindow.document;
// Form cache
// ...

See attached file for full context.

Stph

[Stephen Keller]

Modified HistoryTool.js

[Carlos Eduardo Goncalves]

Bug received.