D15490: [Bookmarks Runner] Remove unused database parameter from query

2018-09-13 Thread Stefan Brüns
This revision was automatically updated to reflect the committed changes. Closed by commit R120:c7e6b584de6d: [Bookmarks Runner] Remove unused database parameter from query (authored by bruns). REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D15490?vs=4

D15490: [Bookmarks Runner] Remove unused database parameter from query

2018-09-13 Thread David Edmundson
davidedmundson accepted this revision. This revision is now accepted and ready to land. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D15490 To: bruns, #frameworks, davidedmundson Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterbe

D15490: [Bookmarks Runner] Remove unused database parameter from query

2018-09-13 Thread Stefan Brüns
bruns added a dependency: D15489: [Bookmarks Runner] Initialize Chrome favicon query only once per profile. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D15490 To: bruns, #frameworks Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreu

D15490: [Bookmarks Runner] Remove unused database parameter from query

2018-09-13 Thread Stefan Brüns
bruns added a dependent revision: D15491: [Bookmarks Runner] Remove BuildQuery Functor class. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D15490 To: bruns, #frameworks Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abett

D15490: [Bookmarks Runner] Remove unused database parameter from query

2018-09-13 Thread Stefan Brüns
bruns created this revision. bruns added a reviewer: Frameworks. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. bruns requested review of this revision. REVISION SUMMARY The query is no longer initialized for each query, just once, thus there no longer is a need for t