D29807: WIP: Change Chrome API design

2020-05-17 Thread Alexander Lohnau
alex updated this revision to Diff 83029. alex marked an inline comment as done. alex added a comment. Create test cases REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29807?vs=83028&id=83029 BRANCH api_inconsistencies (branched from master) RE

D29807: WIP: Change Chrome API design

2020-05-17 Thread Alexander Lohnau
alex updated this revision to Diff 83028. alex added a comment. Make constructor private REPOSITORY R120 Plasma Workspace CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D29807?vs=83019&id=83028 BRANCH api_inconsistencies (branched from master) REVISION DETAIL https://phabrica

D29807: WIP: Change Chrome API design

2020-05-17 Thread Anthony Fieroni
anthonyfieroni added reviewers: broulik, Plasma. anthonyfieroni added a comment. Commented code should be removed. INLINE COMMENTS > chromeprofile.h:11 > +public: > +ProfileBookmarks(const QString &path, const QString &name); > +static QList findAllProfiles(const QString > &applicati

D29807: WIP: Change Chrome API design

2020-05-17 Thread Alexander Lohnau
alex created this revision. Herald added a project: Plasma. Herald added a subscriber: plasma-devel. alex requested review of this revision. REVISION SUMMARY - The FindProfile class was only for Chrome implemented => removed - Having ProfileBookmarks and Profile class is unnecessary, especiall