New issue
Advanced search Search tips

Issue 851927 link

Starred by 4 users

Issue metadata

Status: WontFix
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression bug in Chrome Beta PNACL Chrome FileSystem API no longer works

Reported by bgavi...@gmail.com, Jun 12 2018

Issue description

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

Steps to reproduce the problem:
1.  In Chrome stable version go to this url https://kitchenplanner.kaboodle.com.au/is/apps/idealspaces/?country=aus&anonymousAccess=true#projects/yBHd29zd3t8/design
2. User Name bgavin71@gmail.ocm
Password chromebug
3. Notice how there is oak wood texture on the floor when using chrome stable, but when using chrome beta there is no texture on the flow.

What is the expected behavior?
The 3d scene should look the same when switching to a newer version of chrome.

What went wrong?
This change broke our app that uses PNACL  https://chromium.googlesource.com/chromium/src/+/23efe422cf0cda9a08f542738e1b73b9b170df08

The following error message appears in the console in Chrome Beta

PPB_URLRequestInfo.SetProperty: Attempted to set a value for PP_URLRequestProperty 3, but either this property type is invalid or its parameter was inappropriate (e.g., the wrong type of PP_Var).

Unlike the comment in the changeset this feature is used by our software on some pretty big websites.

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 68.0.3440.17  Channel: beta
OS Version: 10.0
Flash Version:
 
Components: Internals>Services>Network Internals>Plugins>Pepper
Labels: -Type-Bug Type-Bug-Regression
Cc: mek@chromium.org

Comment 3 by mek@chromium.org, Jun 12 2018

Not sure what you mean with "PNACL Chrome FileSystem API"? The only thing that should be broken by my change is the StreamToFile flag when doing a URLRequest. Which according to our statistics is a feature that is used in a vanishingly tiny number of page loads. And if I'm understanding things correctly (I'm unfortunately not a PPAPI expert) you should still be able to get very similar behavior by streaming the data from a url request to a file in the sandboxed filesystem manually.

Comment 4 by bgavi...@gmail.com, Jun 12 2018

Unfortunately it sounds like this is not going to be fixed.   

Not having StreamToFile supported breaks my app that makes use of the documented function.   I will have to rewrite my code, QA it and deploy the change to all of our websites.   

In the future is there a way to know what Chrome API features are not popular and are going to be removed?


Comment 5 by mek@chromium.org, Jun 12 2018

I'm really sorry about the lack of communication around this change. We should of course have tried to reach out to developers to announce this deprecation/removal. But yeah, it's unlikely to change at this point, especially since ultimately all of PNaCl is going to get deprecated/removed anyway (i.e. https://developer.chrome.com/native-client/migration#p-nacl-deprecation-announcements, although that says Q1 2018, which obviously didn't happen).

(and even though the table on that migration page claims there is no open web replacement for StreamToFile, that isn't technically true. XHR to a blob does exactly the same as what StreamToFile used to do).

I'm not sure if there is some general way to be informed about PPAPI/PNACL changes, although I also don't think there are many changes in that area, given it's all deprecated/going away anyway (but again, I don't usually work in this area, so I might be wrong). In general though chromestatus.com is where we track removal and addition of new web platform features in chrome. But I didn't create an entry for this change there, even though I probably should have.
Labels: Needs-Triage-M68

Comment 7 by dxie@chromium.org, Jun 12 2018

Components: -Internals>Services>Network
I don't think this is Servicification related. removing label.
Cc: phanindra.mandapaka@chromium.org
Labels: M-68 Triaged-ET
Owner: mek@chromium.org
Status: Assigned (was: Unconfirmed)
Assigning to @mek@chromium.org  for further action on this and not adding any blocker as this seems to be an intended change.

Thanks.!
Labels: -Hotlist-Interop
> Does this work in other browsers? Yes
No other browser supports Native Client, this is not an interoperability issue.

Comment 10 by mek@chromium.org, Jun 25 2018

Status: WontFix (was: Assigned)
Marking this WontFix. I realize this is unfortunate, but given that PNaCl is deprecated anyway, and the fact that workarounds exists I don't see much reason to change course and try to bring back the StreamToFile API that was intentionally removed. Usage numbers are extremely low as well (https://www.chromestatus.com/metrics/feature/timeline/popularity/2396).

Sign in to add a comment