[rekonq] Review Request: refactor UA code and menu (to be in the same shape as Konqueror)

[Johannes Tröscher]

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

Ship it!


looks fine! :)


src/useragent/useragentmanager.h
<http://git.reviewboard.kde.org/r/103230/#comment7177>

    maybe we should come to an agreement how to name members as some of the devs always use m_memberName and others _memberName


- Johannes Tröscher


On Nov. 24, 2011, 5:09 p.m., Andrea Diamantini wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103230/
> -----------------------------------------------------------
> 
> (Updated Nov. 24, 2011, 5:09 p.m.)
> 
> 
> Review request for rekonq.
> 
> 
> Description
> -------
> 
> Title says it all.
> Two commits in this patch:
> 1) refactor UA code, adding a userAgentManager to better split out ua code.
> 2) refactor UA menu, to be in the same shape as konqueror
> 
> 
> Diffs
> -----
> 
>   src/CMakeLists.txt ced397c 
>   src/application.h acc0335 
>   src/application.cpp f37720f 
>   src/mainwindow.h 9dfe4a1 
>   src/mainwindow.cpp c53fcb6 
>   src/useragent/useragentinfo.h 417aeea 
>   src/useragent/useragentinfo.cpp 0829a7e 
>   src/useragent/useragentmanager.h PRE-CREATION 
>   src/useragent/useragentmanager.cpp PRE-CREATION 
> 
> Diff: http://git.reviewboard.kde.org/r/103230/diff/diff
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> new UA menu
>   http://git.reviewboard.kde.org/r/103230/s/344/
> 
> 
> Thanks,
> 
> Andrea Diamantini
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/rekonq/attachments/20111125/b44cf3ff/attachment.html>