D17527: [Chrome Bookmark Runner] Use QJsonObject directly

Kai Uwe Broulik noreply at phabricator.kde.org
Wed Dec 12 09:20:10 GMT 2018


broulik created this revision.
broulik added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
broulik requested review of this revision.

REVISION SUMMARY
  Instead of converting to `QVariantMap` first.
  Also, port code touched as part of this to range for

TEST PLAN
  Saves some cycles on a query, 0.1ms here,
  bookmarks are still found

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D17527

AFFECTED FILES
  runners/bookmarks/browsers/chrome.cpp
  runners/bookmarks/browsers/chrome.h

To: broulik, #plasma
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20181212/c2950d79/attachment.html>


More information about the Plasma-devel mailing list