Differential D4646

Manager: Fix emitting deviceAdded twice when NM restarts
ClosedPublic
Actions

Authored by drosca on Feb 17 2017, 11:37 AM.

Details

Reviewers
jgrulich
Group Reviewers
Frameworks
Commits
R282:a183e1f346a4: Manager: Fix emitting deviceAdded twice when NM restarts
Test Plan

Restart NetworkManager -> deviceAdded only once for each device

Diff Detail

Repository
R282 NetworkManagerQt
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
drosca updated this revision to Diff 11444.Feb 17 2017, 11:37 AM
drosca retitled this revision from to Manager: Fix emitting deviceAdded twice when NM restarts.
drosca updated this object.
drosca edited the test plan for this revision. (Show Details)
drosca added reviewers: Frameworks, jgrulich.
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 17 2017, 11:37 AM
jgrulich accepted this revision.Feb 17 2017, 11:51 AM
jgrulich edited edge metadata.

Looks good. Thanks.

This revision is now accepted and ready to land.Feb 17 2017, 11:51 AM
Closed by commit R282:a183e1f346a4: Manager: Fix emitting deviceAdded twice when NM restarts (authored by drosca). · Explain WhyFeb 17 2017, 12:16 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 11444

View Options

autotests/managertest.cpp

Loading...
View Options

src/fakenetwork/fakenetwork.h

Loading...
View Options

src/fakenetwork/fakenetwork.cpp

Loading...
View Options

src/manager.cpp

Loading...