New issue
Advanced search Search tips

Issue 628911 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Bug



Sign in to add a comment

Use FrozenArray instead of T[] array in ServiceWorkerMessageEvent and ExtendableMessageEvent.

Project Member Reported by jinho.b...@samsung.com, Jul 17 2016

Issue description

The T[] array type has ben removed from WebIDL. Instead we should use
FrozenArray in ServiceWorkerMessageEvent and ExtendableMessageEvent.

https://github.com/slightlyoff/ServiceWorker/issues/810
 
Labels: M-54
Project Member

Comment 2 by bugdroid1@chromium.org, Jul 19 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/541047a6af9e1a9ac0e4c20dcfc0cf952de45c23

commit 541047a6af9e1a9ac0e4c20dcfc0cf952de45c23
Author: jinho.bang <jinho.bang@samsung.com>
Date: Tue Jul 19 07:53:48 2016

ServiceWorker: Use FrozenArray for *MessageEvent.ports.

The T[] array type has ben removed from WebIDL. Instead we should use
FrozenArray in ServiceWorkerMessageEvent and ExtendableMessageEvent.

Related spec issue:
  https://github.com/slightlyoff/ServiceWorker/issues/810

BUG= 628911 

Review-Url: https://codereview.chromium.org/2155943002
Cr-Commit-Position: refs/heads/master@{#406230}

[modify] https://crrev.com/541047a6af9e1a9ac0e4c20dcfc0cf952de45c23/third_party/WebKit/Source/modules/serviceworkers/ExtendableMessageEvent.idl
[modify] https://crrev.com/541047a6af9e1a9ac0e4c20dcfc0cf952de45c23/third_party/WebKit/Source/modules/serviceworkers/ServiceWorkerMessageEvent.idl

Status: Fixed (was: Started)

Sign in to add a comment