[ktp-common-internals] [debugger] Split logic and UI

Review Request #128847 - Created Sept. 6, 2016 and submitted

Information
Alexandr Akulich
ktp-common-internals
master
Reviewers
telepathy

The main goal of this change is to split logic and UI parts

This is the first step in direction to debugger, which:
1) works with any Telepathy process with DebugInterface support;
2) detects new processess "on fly";
3) has no hardcoded services;
4) shows one process just once, independently of number of dbus services, registered by the process.

The change also opens a way to a QML-based UI at some point in future.

Questionable thing is the "TelepathyProcess" class name.
TelepathyService does not fit, because:
1) Single process can expose a number of services (e.g. MissionControl),
2) The debug interface is applicable to any telepathy application, including clients, so word "Service" (which is not associated with clients) would mislead.

I uploaded a draft of "second step" to my scratch repo:
https://quickgit.kde.org/?p=scratch%2Fakulichalexandr%2Fktp-common-internals.git&a=commitdiff&h=7e07b65f330d85527c9a6b014154527f7e3e7c01&hp=db202a7143be88db37e056913a88992fe7ce507d

I will make a ReviewRequest with the second part on this (split) commit landed.

Works as previously.

Issues

  • 0
  • 2
  • 0
  • 2
Description From Last Updated
Alexandr Akulich
Alexandr Akulich
Anthony Fieroni
Alexandr Akulich
Alexandr Akulich
Alexandr Akulich
Aleix Pol Gonzalez
Alexandr Akulich
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted with commit 33af28091656d4b39412c1a0064b83f28e3064ac by Alexandr Akulich to branch master.
Loading...