|
|
Chromium Code Reviews|
Created:
6 years, 9 months ago by Ken Rockot(use gerrit already) Modified:
6 years, 8 months ago CC:
chromium-reviews, chromium-apps-reviews_chromium.org, extensions-reviews_chromium.org Base URL:
svn://svn.chromium.org/chrome/trunk/src Visibility:
Public. |
DescriptionAdd template for chrome.hid documentation.
Also update channel to stable.
BUG=348085
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266393
Patch Set 1 #Patch Set 2 : poke hid.idl #Patch Set 3 : update docs with usage info #Patch Set 4 : no stable bump #
Messages
Total messages: 22 (0 generated)
+mkearney@: I can't find any evidence that this API ever received a proper documentation review. If it has, please ignore
Hid is going to stable as part of this ?
On 2014/02/28 19:10:21, kalman wrote: > Hid is going to stable as part of this ? Yes. We're just about ready to launch, contingent upon a final security review.
how long ago was it approved?
On 2014/02/28 21:23:38, kalman wrote: > how long ago was it approved? Don't know the complete history of this effort, but there's a launch bug at http://crbug.com/305300. It was targeted for M34 but some relevant changes missed the branch point. In any case it's ready for M35.
On 2014/02/28 22:44:07, Ken Rockot wrote: > On 2014/02/28 21:23:38, kalman wrote: > > how long ago was it approved? > > Don't know the complete history of this effort, but there's a launch bug at > http://crbug.com/305300. It was targeted for M34 but some relevant changes > missed the branch point. In any case it's ready for M35. The launch bug still has AppExt-No.
On 2014/02/28 22:49:50, kalman wrote: > On 2014/02/28 22:44:07, Ken Rockot wrote: > > On 2014/02/28 21:23:38, kalman wrote: > > > how long ago was it approved? > > > > Don't know the complete history of this effort, but there's a launch bug at > > http://crbug.com/305300. It was targeted for M34 but some relevant changes > > missed the branch point. In any case it's ready for M35. > > The launch bug still has AppExt-No. I think this is ready now!
can we just do the documentation part for now?
On 2014/04/25 19:46:50, kalman wrote: > can we just do the documentation part for now? Sure, done. What unblocks the feature update?
lgtm ask mike
On 2014/04/25 19:49:36, kalman wrote: > lgtm > > ask mike API has LGTM from mike
The CQ bit was checked by rockot@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rockot@chromium.org/184483005/50001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: tryserver.chromium on linux_chromium_clang_dbg tryserver.chromium on linux_chromium_rel
The CQ bit was checked by rockot@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rockot@chromium.org/184483005/50001
The CQ bit was unchecked by commit-bot@chromium.org
Try jobs failed on following builders: tryserver.chromium on win_chromium_rel
The CQ bit was checked by rockot@chromium.org
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/rockot@chromium.org/184483005/50001
Message was sent while issue was closed.
Change committed as 266393 |
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
