New issue
Advanced search Search tips

Issue 898158 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Dec 7
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-12-10
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Remove EncodingStreams runtime flag

Project Member Reported by ricea@chromium.org, Oct 23

Issue description

TextEncoderStream and TextDecoderStream are enabled by default and shipping, but they still are protected by the "EncodingStreams" flag. This should be removed once M71 has shipped.
 
Status: Started (was: Assigned)
Status: Assigned (was: Started)
Project Member

Comment 3 by bugdroid1@chromium.org, Dec 7

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

commit 2fbb267e716151d87e9bb0a394935f6944515700
Author: Adam Rice <ricea@chromium.org>
Date: Fri Dec 07 05:10:02 2018

Remove EncodingStreams runtime flag

TextEncoderStream and TextDecoderStream have shipped. The flag is no
longer needed. Remove it.

BUG= 898158 

Change-Id: Ie08d787509f075715b084c7d5e55ad7b2a0e2fb4
Reviewed-on: https://chromium-review.googlesource.com/c/1364934
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#614613}
[modify] https://crrev.com/2fbb267e716151d87e9bb0a394935f6944515700/third_party/blink/renderer/modules/encoding/text_decoder_stream.idl
[modify] https://crrev.com/2fbb267e716151d87e9bb0a394935f6944515700/third_party/blink/renderer/modules/encoding/text_encoder_stream.idl
[modify] https://crrev.com/2fbb267e716151d87e9bb0a394935f6944515700/third_party/blink/renderer/platform/runtime_enabled_features.json5

Status: Fixed (was: Assigned)
The NextAction date has arrived: 2018-12-10

Sign in to add a comment