Vicamo Yang [:vicamo][:vyang] Reporter
	Description • 11 years ago

Moving CellBroadcastMessage to WebIDL doesn't block bug 888591, so breaking down the task into two parts.

Moving CellBroadcastEvent won't be necessary because we don't need that for bug 864484.

	Ken Chang[:kenkai]
	Comment 1 • 10 years ago

Put this bug into backlog.

	Bobby Holley (:bholley)
	Comment 2 • 10 years ago

Same as bug 994461 comment 3.

	Bevis Tseng[:bevis][:btseng](Exited) Assignee
	Comment 3 • 10 years ago

assign to myself to follow up.

	Hsin-Yi Tsai (she/her) [:hsinyi]
	Comment 4 • 10 years ago

(In reply to Bobby Holley (:bholley) from comment #2)
> Same as bug 994461 comment 3.

(In reply to Bevis Tseng [:bevistseng] (btseng@mozilla.com) from comment #3)
> assign to myself to follow up.

Hi Bobby,
We've seen this as [priority]. Thanks Bevis to follow this up. 

However, please kindly understand that 2.1 feature scope is locked down and our team doesn't have enough bandwidth to these late requests. It's hardly to be done by the end of August.

	Bobby Holley (:bholley)
	Comment 5 • 10 years ago

(In reply to Hsin-Yi Tsai [:hsinyi] from comment #4)
> However, please kindly understand that 2.1 feature scope is locked down and
> our team doesn't have enough bandwidth to these late requests. It's hardly
> to be done by the end of August.

That's fine. The issue is mostly that these old-style APIs are a lingering security threat, and we need to apply some pressure to get traction from the wide spectrum of teams that own the various APIs to fix their stuff. I know you folks are on a tight schedule, so doing it in September is fine too, so long as somebody commits to getting it done.

For what it's worth, I expect that this conversion will be a very straightforward process, and shouldn't take more than 45 minutes for someone familiar with the code. Please let me know if you run into any issues with the WebIDL machinery - I'm happy to provide guidance and fix any bugs there if needed.

Thanks for prioritizing it!

	Bevis Tseng[:bevis][:btseng](Exited) Assignee
	Comment 7 • 10 years ago

While implementing bug 864484, 1 necessary modification is to define CellBroadcastMessage in C++ implementation.
To reduce the effort for implementation and review of CellBroadcastMessage implementation,
I'd like combine this bug to Patch Part 2 of bug 864484 to finish this task at once.