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
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
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 11445

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...