Bindings and client interface for the IsAppInstalled API.

Add IsAppInstalled bindings and interface.

This is part 1 of a multipart change set to add IsAppInstalled bindings to Blink.

A later part will add the content/ side of the patch and layout tests
once the implementation is landed.

Intent to Implement: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0xXsJYdkaWg

Explainer: https://github.com/DHNishi/IsNativeAppInstalled/blob/master/explainer.md

BUG=587623
Committed: https://crrev.com/6259e2ce653d05821a1a42784bde6a1ba946e564
Cr-Commit-Position: refs/heads/master@{#379891}

[Daniel Nishi, 2016-02-26 21:10:43]

Description was changed from

==========
Blink IDL and bindings for the IsAppInstalled API.

BUG=
==========

to

==========
Add IsAppInstalled bindings and interface.

This is part 1 of a multipart change set to add IsAppInstalled bindings to
Blink.

A later part will add the content/ side of the patch and layout tests
once the implementation is landed.


Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0xXsJYdkaWg

Explainer:
https://github.com/DHNishi/IsNativeAppInstalled/blob/master/explainer.md

BUG=587623
==========

[Daniel Nishi, 2016-02-26 22:50:19]

Patchset #2 (id:20001) has been deleted

[Daniel Nishi, 2016-02-26 22:50:29]

dhnishi@chromium.org changed reviewers:
+ dcheng@chromium.org, haraken@chromium.org

[Daniel Nishi, 2016-02-26 22:50:29]

haraken@chromium.org: Please review changes in 
third_party/WebKit/Source/modules/*, 
third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in,
third_party/WebKit/Source/web/WebLocalFrameImpl.cpp,
and the layout test.

dcheng@chromium.org: Please review changes in  third_party/WebKit/public/*.

Thanks!

[haraken, 2016-02-27 15:53:12]

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp
(right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:22:
WillBeHeapSupplement<LocalFrame>::provideTo(frame, supplementName(),
adoptPtrWillBeNoop(controller));

OwnPtr<InstalledAppController> controllers = adoptPtr(new ...);
provideTo(..., controllers.release());

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:27:
return
static_cast<InstalledAppController*>(WillBeHeapSupplement<LocalFrame>::from(frame,
supplementName()));

Can we add an assert to check that the returned InstalledAppController is never
null?

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:46:
delete callback;

It's really nasty to call delete manually... Can we avoid it by wrapping the
callback with OwnPtr or something?

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp
(right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:30:
return 0;

nullptr

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:39:
supplement = new NavigatorInstalledApp(navigator.frame());

Use adoptPtr & release.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:71: new
CallbackPromiseAdapter<RelatedAppArray, void>(resolver));

Can't we use adoptPtr(new ...)? We should avoid creating a new object with a raw
pointer.

[dcheng, 2016-02-27 18:30:37]

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp
(right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:58:
HeapVector<Member<RelatedApplication>> applications;
What's the benefit to using a heap pointer here? Why not just have
Vector<RelatedApplication>? RelatedApplication seems like a simple struct.

Similarly, what's the point of passing in a WebVector<WebRelatedApplication*>?
It seems like it should just be WebVector<WebRelatedApplication> (it's also
unclear who owns the pointer values in the WebVector).

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebFrameClient.h (right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebFrameClient.h:363: virtual WebInstalledApp*
installedAppClient() { return 0; }
Nit: nullptr

[Daniel Nishi, 2016-02-29 18:38:39]

Thanks! Please take another look when you get a chance.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp
(right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:22:
WillBeHeapSupplement<LocalFrame>::provideTo(frame, supplementName(),
adoptPtrWillBeNoop(controller));
On 2016/02/27 15:53:11, haraken wrote:
> 
> OwnPtr<InstalledAppController> controllers = adoptPtr(new ...);
> provideTo(..., controllers.release());

.release() for an OwnPtr is an OwnPtr, whereas provideTo expects a raw ptr.

Does this pattern work? I found it used for other provideTo frame places in the
code base.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:27:
return
static_cast<InstalledAppController*>(WillBeHeapSupplement<LocalFrame>::from(frame,
supplementName()));
On 2016/02/27 15:53:11, haraken wrote:
> 
> Can we add an assert to check that the returned InstalledAppController is
never
> null?

Done.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:46:
delete callback;
On 2016/02/27 15:53:11, haraken wrote:
> 
> It's really nasty to call delete manually... Can we avoid it by wrapping the
> callback with OwnPtr or something?
> 

Done.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp
(right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:30:
return 0;
On 2016/02/27 15:53:12, haraken wrote:
> 
> nullptr

Done.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:39:
supplement = new NavigatorInstalledApp(navigator.frame());
On 2016/02/27 15:53:12, haraken wrote:
> 
> Use adoptPtr & release.

The rest of Blink uses regular pointers for Navigator<S>::from. I think I could
toss adoptPtrWillBeNoop, but I'm not sure how to do it with adoptPTr and
release.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:58:
HeapVector<Member<RelatedApplication>> applications;
On 2016/02/27 18:30:37, dcheng wrote:
> What's the benefit to using a heap pointer here? Why not just have
> Vector<RelatedApplication>? RelatedApplication seems like a simple struct.
> 
> Similarly, what's the point of passing in a WebVector<WebRelatedApplication*>?
> It seems like it should just be WebVector<WebRelatedApplication> (it's also
> unclear who owns the pointer values in the WebVector).

I think when using CallbackPromiseAdapter in this way, it has to be a
HeapVector, rather then Vector. Otherwise, there is a static assertion regarding
DISALLOW_NEW_EXCEPT_PLACEMENT_NEW. If I'm missing something about how it works,
please let me know.

I've also changed it over to being a WebVector<WebRelatedApplication> instead of
a pointer.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:71: new
CallbackPromiseAdapter<RelatedAppArray, void>(resolver));
On 2016/02/27 15:53:11, haraken wrote:
> 
> Can't we use adoptPtr(new ...)? We should avoid creating a new object with a
raw
> pointer.
> 

I've changed it to use adoptWebPtr -- this bit crosses over into Chromium.

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebFrameClient.h (right):

https://codereview.chromium.org/1735033004/diff/40001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebFrameClient.h:363: virtual WebInstalledApp*
installedAppClient() { return 0; }
On 2016/02/27 18:30:37, dcheng wrote:
> Nit: nullptr

Done.

[haraken, 2016-03-01 02:15:26]

LGTM

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:47:
callback.release()->onError();

callback->onError()

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl:5: [

Add a link to the spec.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/RelatedApplication.idl
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/RelatedApplication.idl:5: [

Add a link to the spec.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
File
third_party/WebKit/public/platform/modules/installedapp/WebRelatedApplication.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/platform/modules/installedapp/WebRelatedApplication.h:1:
// Copyright 2015 The Chromium Authors. All rights reserved.

2016

[dcheng, 2016-03-01 20:03:04]

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/InstalledAppController.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.h:21: 
Nit: no newline before public: (here and elsewhere)

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:59: for
(const auto webApplication : *webInfo)
const auto&, this copies WebRelatedApplication otherwise.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/RelatedApplication.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/RelatedApplication.h:23:
RelatedApplication(const String& platform, const String& url, const String& id)
Constructor should be private.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in (right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in:105: InstalledApp
status=experimental until this ships?

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/modules/installedapp/WebInstalledApp.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/platform/modules/installedapp/WebInstalledApp.h:15:
using AppInstalledCallbacks =
WebCallbacks<WebPassOwnPtr<WebVector<WebRelatedApplication>>, void>;
Do we need to wrap the WebVector in a WebPassOwnPtr here? It's hard to tell if
this is needed without seeing the //content implementation.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebFrameClient.h (right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebFrameClient.h:363: virtual WebInstalledApp*
installedAppClient() { return nullptr; }
This should be named WebInstalledAppClient.

[Daniel Nishi, 2016-03-01 20:45:38]

Patchset #4 (id:80001) has been deleted

[Daniel Nishi, 2016-03-01 20:46:20]

Thanks again to you two.

dcheng: Please take another look when you get a chance. Thanks again for
reviewing.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.cpp:47:
callback.release()->onError();
On 2016/03/01 02:15:25, haraken wrote:
> 
> callback->onError()

WebPassOwnPtr doesn't seem to let us use the arrow operator -- the returned
PassOwnPtr from the release does, though.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/InstalledAppController.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/InstalledAppController.h:21: 
On 2016/03/01 20:03:03, dcheng wrote:
> Nit: no newline before public: (here and elsewhere)

Done.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.cpp:59: for
(const auto webApplication : *webInfo)
On 2016/03/01 20:03:03, dcheng wrote:
> const auto&, this copies WebRelatedApplication otherwise.

Done.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/NavigatorInstalledApp.idl:5: [
On 2016/03/01 02:15:26, haraken wrote:
> 
> Add a link to the spec.

Done.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/RelatedApplication.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/RelatedApplication.h:23:
RelatedApplication(const String& platform, const String& url, const String& id)
On 2016/03/01 20:03:03, dcheng wrote:
> Constructor should be private.

Done.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/modules/installedapp/RelatedApplication.idl
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/modules/installedapp/RelatedApplication.idl:5: [
On 2016/03/01 02:15:26, haraken wrote:
> 
> Add a link to the spec.

Done.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
File third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in (right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/Sour...
third_party/WebKit/Source/platform/RuntimeEnabledFeatures.in:105: InstalledApp
On 2016/03/01 20:03:03, dcheng wrote:
> status=experimental until this ships?

Done.

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/modules/installedapp/WebInstalledApp.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/platform/modules/installedapp/WebInstalledApp.h:15:
using AppInstalledCallbacks =
WebCallbacks<WebPassOwnPtr<WebVector<WebRelatedApplication>>, void>;
On 2016/03/01 20:03:03, dcheng wrote:
> Do we need to wrap the WebVector in a WebPassOwnPtr here? It's hard to tell if
> this is needed without seeing the //content implementation.

After playing around with the //content implementation, it doesn't have to be.

I've switched this to a const WebVector<WebRelatedApplication>&

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebFrameClient.h (right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebFrameClient.h:363: virtual WebInstalledApp*
installedAppClient() { return nullptr; }
On 2016/03/01 20:03:03, dcheng wrote:
> This should be named WebInstalledAppClient.

Done.

[dcheng, 2016-03-01 22:03:46]

lgtm

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
File third_party/WebKit/public/platform/modules/installedapp/WebInstalledApp.h
(right):

https://codereview.chromium.org/1735033004/diff/60001/third_party/WebKit/publ...
third_party/WebKit/public/platform/modules/installedapp/WebInstalledApp.h:15:
using AppInstalledCallbacks =
WebCallbacks<WebPassOwnPtr<WebVector<WebRelatedApplication>>, void>;
On 2016/03/01 at 20:46:20, Daniel Nishi wrote:
> On 2016/03/01 20:03:03, dcheng wrote:
> > Do we need to wrap the WebVector in a WebPassOwnPtr here? It's hard to tell
if
> > this is needed without seeing the //content implementation.
> 
> After playing around with the //content implementation, it doesn't have to be.
> 
> I've switched this to a const WebVector<WebRelatedApplication>&

OK, I leave it up to your judgement which one works best. Some code does this on
purpose to avoid copying the vector. Either way is OK with me as long as it
makes sense.

[Daniel Nishi, 2016-03-08 18:03:50]

The CQ bit was checked by dhnishi@chromium.org

[Daniel Nishi, 2016-03-08 18:03:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/1735033004/#ps200001
(title: "Fix layout tests for real.")

[Daniel Nishi, 2016-03-08 18:04:01]

The CQ bit was unchecked by dhnishi@chromium.org

[Daniel Nishi, 2016-03-08 18:04:21]

The CQ bit was checked by dhnishi@chromium.org

[commit-bot: I haz the power, 2016-03-08 18:04:47]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1735033004/200001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1735033004/200001

[commit-bot: I haz the power, 2016-03-08 19:49:42]

Description was changed from

==========
Add IsAppInstalled bindings and interface.

This is part 1 of a multipart change set to add IsAppInstalled bindings to
Blink.

A later part will add the content/ side of the patch and layout tests
once the implementation is landed.


Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0xXsJYdkaWg

Explainer:
https://github.com/DHNishi/IsNativeAppInstalled/blob/master/explainer.md

BUG=587623
==========

to

==========
Add IsAppInstalled bindings and interface.

This is part 1 of a multipart change set to add IsAppInstalled bindings to
Blink.

A later part will add the content/ side of the patch and layout tests
once the implementation is landed.


Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0xXsJYdkaWg

Explainer:
https://github.com/DHNishi/IsNativeAppInstalled/blob/master/explainer.md

BUG=587623
==========

[commit-bot: I haz the power, 2016-03-08 19:49:43]

Committed patchset #9 (id:200001)

[commit-bot: I haz the power, 2016-03-08 19:50:27]

Description was changed from

==========
Add IsAppInstalled bindings and interface.

This is part 1 of a multipart change set to add IsAppInstalled bindings to
Blink.

A later part will add the content/ side of the patch and layout tests
once the implementation is landed.


Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0xXsJYdkaWg

Explainer:
https://github.com/DHNishi/IsNativeAppInstalled/blob/master/explainer.md

BUG=587623
==========

to

==========
Add IsAppInstalled bindings and interface.

This is part 1 of a multipart change set to add IsAppInstalled bindings to
Blink.

A later part will add the content/ side of the patch and layout tests
once the implementation is landed.

Intent to Implement:
https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/0xXsJYdkaWg

Explainer:
https://github.com/DHNishi/IsNativeAppInstalled/blob/master/explainer.md

BUG=587623

Committed: https://crrev.com/6259e2ce653d05821a1a42784bde6a1ba946e564
Cr-Commit-Position: refs/heads/master@{#379891}
==========

[commit-bot: I haz the power, 2016-03-08 19:50:28]

Patchset 9 (id:??) landed as
https://crrev.com/6259e2ce653d05821a1a42784bde6a1ba946e564
Cr-Commit-Position: refs/heads/master@{#379891}

[haraken, 2016-07-06 09:08:01]

It looks like that the content-side implementation of installedAppClient is not
yet landed. Are you actively working on it?

(Note: My intention is not to ask you to revert this CL. I'm removing
willDetachFromFrameHost from the code base and wondering how to handle 
InstalledAppController::willDetachFrameHost.)

[Daniel Nishi, 2016-07-06 16:33:27]

On 2016/07/06 09:08:01, haraken wrote:
> It looks like that the content-side implementation of installedAppClient is
not
> yet landed. Are you actively working on it?
> 
> (Note: My intention is not to ask you to revert this CL. I'm removing
> willDetachFromFrameHost from the code base and wondering how to handle 
> InstalledAppController::willDetachFrameHost.)

Thanks for reaching out, Haraken!

I'm not on the team anymore, but benwells has ownership of isAppInstalled now.

[haraken, 2016-07-07 00:08:53]

haraken@chromium.org changed reviewers:
+ benwells@chromium.org

[haraken, 2016-07-07 00:08:54]

+benwells

Are you actively working on this feature? (See #22 for more contexts)

[benwells, 2016-07-17 22:08:01]

On 2016/07/07 00:08:54, haraken wrote:
> +benwells
> 
> Are you actively working on this feature? (See #22 for more contexts)

Yes, it's in my list of things to do and still relevant from a product
perspective. My next step was to raise with the wicg and get a feel for the
reactions there.