New issue
Advanced search Search tips

Issue 821536 link

Starred by 7 users

Issue metadata

Status: Duplicate
Merged: issue 820163
Owner: ----
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Chrome 65 is losing / unable to use the XHR response if response size > ~64K

Reported by mjofarr...@gmail.com, Mar 13 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/65.0.3325.146 Safari/537.36

Steps to reproduce the problem:
1. Make an XHR request that has a large response (>64K)
2. The DOM cannot update with the response

What is the expected behavior?
DOM should be able to be updated with the response.

What went wrong?
Large async XHR requests are not able to update the DOM with the response. Either the response is lost / not returning, or it cannot be reacted to with the callback. Breaking change in Chrome 65.

Did this work before? Yes Chrome 64

Does this work in other browsers? Yes

Chrome version: 65.0.3325.146  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 29.0 r0

Other reports: https://groups.google.com/a/chromium.org/forum/#!topic/chromium-discuss/HQFUbvsLgSI

https://www.essentialobjects.com/forum/postst10828_Chrome-65-callback.aspx
 

Comment 1 by mattm@chromium.org, Mar 13 2018

Components: Blink>Network>XHR
Labels: Needs-Triage-M65 Needs-Bisect

Comment 3 by f...@opera.com, Mar 14 2018

Components: Blink>XML
There has been other reports that XML parsing has failed for certain types of documents ( issue 820163  for instance.) So under the assumption that the payload here is indeed an XML document, this could have the same underlying cause. My reading of the above links is that you get two Text nodes where you previously got one - is that correct? Could you provide an example response (or a way of reproducing the issue)?

(FWIW, if you're getting multiple siblings Text nodes, you could work around it by calling Node.normalize() on the XML document/fragment root before using the document/fragment.)
On the second link (https://www.essentialobjects.com/forum/postst10828_Chrome-65-callback.aspx), one of the comments gives a test example, with code, replicating this issue here: http://webshop.langauto.hu/callback.aspx 

It uses a callback component (eo:callbackpanel) to return a random string in the response. If 100000 is entered as the length of the string, it returns and loads to the page. If you enter 300000, it does not load to the page (but the poster says there is a response from the server). This previuosly worked in 64, and works in other browsers.

Comment 5 by f...@opera.com, Mar 14 2018

Components: -Blink>Network>XHR
Status: Untriaged (was: Unconfirmed)
Thanks! Easily verifiable there, and in this case it's the/a CDATA that ends up being split. Doesn't look like XHR is to blame.

Comment 6 by f...@opera.com, Mar 14 2018

Labels: OS-Android OS-Linux OS-Mac
Cc: viswa.karala@chromium.org
Labels: -Pri-2 -Needs-Bisect Target-67 Triaged-ET Target-66 M-65 RegressedIn-65 FoundIn-66 FoundIn-67 Target-65 FoundIn-65 ReleaseBlock-Stable hasbisect Pri-1
Owner: joelhockey@chromium.org
Status: Assigned (was: Untriaged)
Able to reproduce the issue on reported version 65.0.3325.146 and latest chrome 67.0.3370.0 using Mac 10.12.6, Ubuntu 14.04 and Windows-10, as the break is on 65.0.3325.0 branch builds, hence providing manual change log from omahaproxy

Bisect Info:
================
Good build: 65.0.3325.68
Bad build: 65.0.3325.69

https://chromium.googlesource.com/chromium/src/+/9dc050f2480deaf9882ca95a6f214cc552ef9339

Change-Id: Id2fa1d96e55be1e0483c135c20c20b90a068f4c3
Reviewed-on: https://chromium-review.googlesource.com/897220

@Joel Hockey: Please confirm the issue and help in re-assigning if it is not related to your change.
Adding ReleaseBlock-Stable for M-65 as it is seems a recent break, feel free to remove it if not applicable.

Thanks!
Owner: ----
I'm sorry, I'm no longer working in chromium, so I wont be able to look into this.  I suspect that this bug may be related to the XML changes described in  issue 820163 .  I give some suggestions there about possible solutions, including rolling back the change I made.
Status: Available (was: Assigned)
Cc: schenney@chromium.org
+schenney@ , could you ptal comment #8 and mark this as dupe if needed?
It's probably a duplicate but I know nothing about this kind of thing, so would have to defer the decision.
Mergedinto: 820163
Status: Duplicate (was: Available)
Pls verify this fix on canary version 67.0.3372.1+. Thank you.
Labels: TE-Verified-M67 TE-Verified-67.0.3375.0
Able to reproduce the issue on chrome reported version 65.0.3325.146
Verified the fix on Windows-10, Ubuntu 14.04, Mac 10.13.1 on Chrome version #67.0.3375.0 as per the comment#0
Attaching screen shot for reference.
Observed "Able to see DOM response"
Hence, the fix is working as expected.
Adding the verified label.

Thanks!
821536.png
785 KB View Download
Is the plan to apply this fix to v65 too? Any idea of timescales?

Comment 16 by f...@opera.com, Mar 19 2018

The fix has been applied to the M-65 branch, so it should be making its way out in due time.
Yes, with high probability the fix will go to users this week.

Sign in to add a comment