Review Request: KIO::TransferJob member function for disabling internal redirection handling...

[Dawit Alemayehu]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2726/
-----------------------------------------------------------

(Updated 2010-01-25 21:32:30.611754)


Review request for kdelibs and David Faure.


Changes
-------

* Moved the new functions for disabling internal redirection handling to KIO::SimpleJob.
* Renamed the new get/set functions to isRedirectionHandlingEnabled & setRedirectionHandlingEnabled respectively based on feedback from David.
* Added support for disabling internal redirection handling to almost all of the classes (except KIO::MultiGetJob) that support redirection and inherit from KIO::SimpleJob.


Summary
-------

Currently there is no way to prevent KIO from automatically handling redirection requests internally. Unfortunately, this causes a lot of issues for the class used to provide KIO integration to QtWebKit, KIO::AccessManager, because QtWebKit handles redirections internally too! In other words, there is no way to stop one of these two modules from doing the redirection. Since all the hacks and workarounds used so far have their own issues that end up causing bugs, for example you cannot currently login into blogger.com using konqueror+kwebkitpart, I have created a patch to address this issue properly.

The attached patch adds two member functions to KIO::TransfterJob that allows a programmer to explicitly disable internal handling of redirection requests. The default behavior remains exactly as it is now, i.e. redirections are automatically handled internally. 


Since this change is essentially a bug fix for a brand new module in kdelibs, I would like to apply this patch before to the 4.4 goes out.


Diffs (updated)
-----

  /branches/KDE/4.4/kdelibs/kio/kio/accessmanager.cpp 1080138 
  /branches/KDE/4.4/kdelibs/kio/kio/job.cpp 1080137 
  /branches/KDE/4.4/kdelibs/kio/kio/job_p.h 1080137 
  /branches/KDE/4.4/kdelibs/kio/kio/jobclasses.h 1080137 

Diff: http://reviewboard.kde.org/r/2726/diff


Testing
-------


Thanks,

Dawit