659956 - Remove unused variable from nsOfflineCacheUpdate.cpp

Status: RESOLVED FIXED

Description

[Ed Morley [:emorley]]

Similar to bug 619590, but for nsOfflineCacheUpdate.cpp

> 511     nsCOMPtr<nsICachingChannel> oldCachingChannel =
> 512         do_QueryInterface(aOldChannel);

|oldCachingChannel| is not used anywhere else (http://mxr.mozilla.org/mozilla-central/search?string=oldCachingChannel) so can be removed.

Comment 1

[Ed Morley [:emorley]]

Created attachment 535356 [details] [diff] [review]
Remove unused variable from nsOfflineCacheUpdate.cpp

Two line removal.

Thanks :-)

Comment 2

[Boris Zbarsky [:bz] (work week until 5/26) (if a patch has no decent message, automatic r-)]

Comment on attachment 535356 [details] [diff] [review]
Remove unused variable from nsOfflineCacheUpdate.cpp

r=me

Comment 3

[Ed Morley [:emorley]]

Created attachment 535393 [details] [diff] [review]
Remove unused variable from nsOfflineCacheUpdate.cpp

Only change is adding the patch author/username, after Tortoisehg apparently ate my hgrc.

Carrying forwards r+.

Comment 4

[Daniel Holbert [:dholbert] (reduced availability - travel & post-PTO backlog)]

http://hg.mozilla.org/projects/cedar/rev/2280f0d23ce2

Comment 5

[Mounir Lamouri (:mounir)]

Pushed:
http://hg.mozilla.org/mozilla-central/rev/2280f0d23ce2