[Chrome Bookmark Runner] Use QJsonObject directly
ClosedPublic
Actions

Authored by broulik on Dec 12 2018, 9:20 AM.

Details

Reviewers

mart

Group Reviewers

Plasma

Commits

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

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Restricted Application added a subscriber: plasma-devel. · View Herald Transcript

broulik requested review of this revision.Dec 12 2018, 9:20 AM

This revision is now accepted and ready to land.Dec 13 2018, 12:05 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			runners/bookmarks/browsers/chrome.h (6 lines)
M			runners/bookmarks/browsers/chrome.cpp (34 lines)

Diff	ID	Base	Description	Created	Lint	Unit
Base			Base
Diff 1	47425			Dec 12 2018, 9:18 AM	★	★
Diff 2	47511	2907d7f	R120:434765e9fb9759a1a435c019be0044dfa817562c	Dec 13 2018, 12:52 PM	★	★