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

Issue 718308 link

Starred by 1 user
Status: Assigned
Owner:
█ sonnysasaka@chromium.org
Last visit > 30 days ago
Cc:
r...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug

Blocking:
issue 707630



Sign in to add a comment

bluetooth: Pass base::Time* by value instead of const&

Project Member Reported by ortuno@chromium.org, May 4 2017 
SetAdvertisingInterval[1] currently takes base::TimeDelta by const&. The documentation of base::Time[2] states that we should pass it by value.

[1] https://cs.chromium.org/chromium/src/device/bluetooth/bluetooth_adapter.h?type=cs&q=SetAdvertisingInterval&sq=package:chromium&l=454
[2] https://cs.chromium.org/chromium/src/base/time/time.h?sq=package:chromium&dr=C&l=24
►

Sign in to add a comment