Review Request: BookmarksRunner uses better way to find the default browser

Matthias Fuchs mat69 at gmx.net
Thu Dec 2 20:20:32 CET 2010


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

Review request for Plasma and Aaron Seigo.


Summary
-------

If no specific browser has been set as default, uses the application that is set to open html-files.

Ideally there would be a method in kdelibs to retrieve the default browser though I suppose that is a different task than this bug.


This addresses bug 224406.
    https://bugs.kde.org/show_bug.cgi?id=224406


Diffs
-----

  /trunk/KDE/kdebase/apps/dolphin/src/tests/dolphinviewtest_allviewmodes.cpp 1202673 
  /trunk/KDE/kdebase/apps/dolphin/src/tests/testbase.cpp 1202673 
  /trunk/KDE/kdebase/runtime/nepomuk/services/backupsync/gui/nepomukbackup.desktop 1202673 
  /trunk/KDE/kdebase/runtime/plasma/containments/newspaper/appletoverlay.cpp 1202673 
  /trunk/KDE/kdebase/runtime/plasma/containments/newspaper/appletsview.cpp 1202673 
  /trunk/KDE/kdebase/runtime/plasma/declarativeimports/core/theme.cpp 1202673 
  /trunk/KDE/kdebase/runtime/plasma/declarativeimports/core/theme_p.h 1202673 
  /trunk/KDE/kdebase/workspace/plasma/desktop/shell/activitymanager/filterbar.cpp 1202673 
  /trunk/KDE/kdebase/workspace/plasma/generic/containmentactions/switchactivity/switch.cpp 1202673 
  /trunk/KDE/kdebase/workspace/plasma/generic/containmentactions/switchwindow/switch.cpp 1202673 
  /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/activities/activities.operations 1202673 
  /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/activities/activityengine.cpp 1202673 
  /trunk/KDE/kdebase/workspace/plasma/generic/dataengines/activities/activityjob.cpp 1202673 
  /trunk/KDE/kdebase/workspace/plasma/generic/runners/bookmarks/bookmarksrunner.cpp 1202673 

Diff: http://svn.reviewboard.kde.org/r/6032/diff


Testing
-------


Thanks,

Matthias

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20101202/b25e11b5/attachment.htm 


More information about the Plasma-devel mailing list