Issue 610418 link

Starred by 1 user

Issue metadata

Status:	Fixed
Owner:	peter@chromium.org
Closed:	Jan 2017
Cc:	xhw...@chromium.org
Components:	Blink>Internals
EstimatedDays:	----
NextAction:	----
OS:	----
Pri:	2
Type:	Bug

Blocking:
issue 610421

Notifications should use blink style bindings for mojo

Project Member Reported by esprehn@chromium.org, May 9 2016

Issue description

https://code.google.com/p/chromium/codesearch#chromium/src/third_party/WebKit/Source/modules/notifications/Notification.h&q=file:webkit%20%22.mojom.h%22%20-file:gen%20lang:cpp&sq=package:chromium&dr=C&l=49

this should use the -blink.h bindings, not the std::string/std::array bindings.

Comment 1 by esprehn@chromium.org, May 9 2016

Blocking: 610421

Comment 2 by tkent@chromium.org, Jun 23 2016

Components: -Blink>Architecture Blink>Internals

Renaming Blink>Architecture to Blink>Internals

Comment 3 by jialiul@chromium.org, Jul 25 2016

Components: -Internals>Permissions

Comment 4 by dcheng@chromium.org, Jan 12 2017

Owner: peter@chromium.org
Status: Fixed (was: Untriaged)

I believe this was fixed by https://chromium.googlesource.com/chromium/src/+/bd303897bfe40025be26eaaf729568429373ef79

Comment 5 by peter@chromium.org, Jan 12 2017

Yup - thanks for closing!

►

Issue 610418 link

Issue metadata

Notifications should use blink style bindings for mojo

Issue description

Comment 1 by esprehn@chromium.org, May 9 2016

Comment 1 Deleted

Comment 2 by tkent@chromium.org, Jun 23 2016

Comment 2 Deleted

Comment 3 by jialiul@chromium.org, Jul 25 2016

Comment 3 Deleted

Comment 4 by dcheng@chromium.org, Jan 12 2017

Comment 4 Deleted

Comment 5 by peter@chromium.org, Jan 12 2017

Comment 5 Deleted

Sign in to add a comment