New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 823230 link

Starred by 2 users
Status: Fixed
Owner:
schenney@chromium.org
Closed: Mar 2018
Cc:
█ abdulsyed@chromium.org
█ manoranj...@chromium.org
pbomm...@chromium.org
schenney@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[Missing Test]: Chrome 65 is losing / unable to use the XHR response if response size > ~64K

Project Member Reported by viswa.karala@chromium.org, Mar 19 2018

Comment 1 by gov...@chromium.org, Mar 19 2018

Cc: pbomm...@chromium.org abdulsyed@chromium.org manoranj...@chromium.org
Labels: -M-65 M-66
This is not a blocker for M65, pls have missing tests added for M66 if possible.

Comment 2 by schenney@chromium.org, Mar 20 2018

Cc: schenney@chromium.org
 Issue 823633  has been merged into this issue.
Project Member

Comment 3 by bugdroid1@chromium.org, Mar 21 2018 

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

commit 4819d080e4c615633a070572e806721ed5114b0b
Author: Stephen Chenney <schenney@chromium.org>
Date: Wed Mar 21 17:17:55 2018

Add a test for xml parsing of long multi-byte encoded files

Verified that this fails with the broken libxml that caused
crbug/820163 and its friends.

R=dcheng@chromium.org
BUG= 820163 , 823230 

Change-Id: I53d631871d4343aaa67c874a90eacbf4d58dc454
Reviewed-on: https://chromium-review.googlesource.com/969408
Commit-Queue: Stephen Chenney <schenney@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544748}
[add] https://crrev.com/4819d080e4c615633a070572e806721ed5114b0b/third_party/WebKit/LayoutTests/xmlviewer/long-multi-byte-char-expected.html
[add] https://crrev.com/4819d080e4c615633a070572e806721ed5114b0b/third_party/WebKit/LayoutTests/xmlviewer/long-multi-byte-char.xml
[add] https://crrev.com/4819d080e4c615633a070572e806721ed5114b0b/third_party/WebKit/LayoutTests/xmlviewer/resources/long-multi-byte-char.xslt

Comment 4 by schenney@chromium.org, Mar 22 2018

Status: Fixed (was: Assigned)
Project Member

Comment 5 by blocker...@chrome-infra-auth.iam.gserviceaccount.com, Mar 22 2018

Labels: Merge-TBD
[Auto-generated comment by a script] We noticed that this issue is targeted for M-66; it appears the fix may have landed after branch point, meaning a merge might be required. Please confirm if a merge is required here - if so add Merge-Request-66 label, otherwise remove Merge-TBD label. Thanks.

Comment 6 by gov...@chromium.org, Mar 22 2018 

Pls request a merge to M66 if needed and remove "Merge-TBD" label. Thank you.

Comment 7 by schenney@chromium.org, Mar 22 2018

Labels: -Merge-TBD -ReleaseBlock-Stable
This is a test. We only need it on tip of tree.
►

Sign in to add a comment