1410346 - Merge U2F.cpp and U2FManager.cpp

Status: RESOLVED FIXED

(Core :: DOM: Device Interfaces, enhancement, P2)

Description

[Tim Taubert [:ttaubert] (inactive)]

We should merge the two files, we don't need to have two state machines, the first working with callbacks from the original API and using promises to communicate with the U2FManager that implements the IPC part.

Comment 1

[Phabricator Automation]

Comment on attachment 8932057 [details]
Bug 1410346 - Merge U2F.cpp and U2FManager.cpp r?jcj

J.C. Jones [:jcj] has approved the revision.

https://phabricator.services.mozilla.com/D288#6993

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0750af27bcee
Merge U2F.cpp and U2FManager.cpp r=jcj

Comment 3

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7aa92cc6daba
Backed out changeset 0750af27bcee

Comment 4

[Tim Taubert [:ttaubert] (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=cf6274cbdf3b8f8d4aacaf690e6cfd6c229b84eb

Comment 5

[Tim Taubert [:ttaubert] (inactive)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=33f0bab29fa03cc8412c2ffd65b604b8e56c05b6

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/da0b490b4aea
Merge U2F.cpp and U2FManager.cpp r=jcj

Comment 7

[Dorel Luca [:dluca]]

https://hg.mozilla.org/mozilla-central/rev/da0b490b4aea