New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 700594 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2017
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Task

Blocking:
issue 704309



Sign in to add a comment

Modify APDU and U2fMessage classes to use std::unique_ptr

Project Member Reported by piperc@chromium.org, Mar 11 2017

Issue description

This bug tracks the modification of device/u2f/ classes from using shared_refptr to unique_ptrs when they are uniquely owned.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 22 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6de00828602a051db2bb66c17e5867b5559d2ef3

commit 6de00828602a051db2bb66c17e5867b5559d2ef3
Author: piperc <piperc@chromium.org>
Date: Wed Mar 22 16:37:32 2017

Modify U2F apdu classes to use unique pointers

Apdu classes will have unique ownership, so changing the classes
to std::unique_ptr instead of shared_refptr.

BUG= 700594 

Review-Url: https://codereview.chromium.org/2743623006
Cr-Commit-Position: refs/heads/master@{#458774}

[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_apdu_command.cc
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_apdu_command.h
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_apdu_fuzzer.cc
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_apdu_response.cc
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_apdu_response.h
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_apdu_unittest.cc
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_device.cc
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_device.h
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_hid_device.cc
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_hid_device.h
[modify] https://crrev.com/6de00828602a051db2bb66c17e5867b5559d2ef3/device/u2f/u2f_hid_device_unittest.cc

Comment 2 by piperc@chromium.org, Mar 22 2017

Blocking: 704309

Comment 4 by piperc@chromium.org, Mar 22 2017

Status: Fixed (was: Started)

Sign in to add a comment