251835 - one line of redundant code in nsXPInstallManager.cpp

Status: RESOLVED FIXED

(Core Graveyard :: Installer: XPInstall Engine, defect)

Description

[Constantine A. Murenin]

Variable isPageLoading was introduced in version 1.124 (bug 238684), but is no
longer needed in version 1.123 (bug 242111). See the upcoming patch.

Comment 1

[Constantine A. Murenin]

Attachment: the patch

Comment 2

[Constantine A. Murenin]

(In reply to comment #0)
> Variable isPageLoading was introduced in version 1.124 (bug 238684), but is no
> longer needed in version 1.123 (bug 242111). See the upcoming patch.

"1.123" should read "1.125". :-)

Comment 3

[Johnny Stenback (:jst)]

Comment on attachment 153487 [details] [diff] [review]
the patch

r+sr=jst. Email me if you need help with getting this checked in.

Comment 4

[Christian :Biesinger (don't email me, ping me on IRC)]

Checking in nsXPInstallManager.cpp;
/cvsroot/mozilla/xpinstall/src/nsXPInstallManager.cpp,v  <--  nsXPInstallManager.cpp
new revision: 1.126; previous revision: 1.125
done