Add BluetoothDevice.uuids attribute removal date in the deprecation string

third_party/WebKit/Source/core/frame/Deprecation.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/frame/Deprecation.h"
 
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/frame/FrameConsole.h"
 #include "core/frame/FrameHost.h"
@@ skipping 444 matching lines @@
       return willBeRemoved("PaymentAddress.careOf", M56, "5728579069411328");
 
     case UseCounter::VRDeprecatedFieldOfView:
       return replacedBy("VREyeParameters.fieldOfView",
                         "projection matrices provided by VRFrameData");
 
     case UseCounter::VRDeprecatedGetPose:
       return replacedBy("VRDisplay.getPose()", "VRDisplay.getFrameData()");
 
     case UseCounter::DeprecatedBluetoothDeviceUUIDsAttribute:
-      return "BluetoothDevice.uuids is deprecated. Use getPrimaryServices() to "
-             "retrieve all available UUIDs.";
+      return String::format(

[dcheng, 2016/10/13 04:43:33]

Can this use replacedBy("BluetoothDevice.uuids", M57, "getPrimaryServices()")?

[juncai, 2016/10/13 18:22:07]

The replacedBy() takes two parameters instead of three:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/core/frame/Dep...
So the removal date can not be added.

[dcheng, 2016/10/13 20:13:37]

Oops, sorry, I meant willBeRemoved(): does that make sense here?

[juncai, 2016/10/13 21:01:08]

As the comment at:
https://codereview.chromium.org/2408773003/
mentioned, we don't have a chromestatus link, so we can't use willBeRemoved()
either.

[Jeffrey Yasskin, 2016/10/13 21:04:34]

Specifically, Web Bluetooth has a link at
https://www.chromestatus.com/features/5264933985976320, but the removal of
'uuids' doesn't. I don't think this removal needs a chromestatus link because
Web Bluetooth isn't shipped yet (it's in an origin trial), but it does make
using the standard helper functions more difficult. The reason to deprecate
instead of removing outright, despite not having shipped, is that sites are
using the API under the origin trial, and it's kind not to break them abruptly
if we don't have to.

+          "BluetoothDevice.uuids is deprecated and will be removed in %s. Use "
+          "getPrimaryServices() to retrieve all available UUIDs.",
+          milestoneString(M57));
 
     // Features that aren't deprecated don't have a deprecation message.
     default:
       return String();
   }
 }
 
 }  // namespace blink