1648141 - IPCBlobInputStream should be renamed to something more generic than "IPCBlob"

Status: RESOLVED FIXED

(Core :: DOM: File, task)

Description

[Andrea Marchesini [:baku]]

IPCBlobInputStream is a powerful inputStream implementation able to reduce the amount of data sent when inputStreams are shared between processes.
Initially, it was meant to be used by DOM Blobs only, but now, ServiceWorkers, DocumentChannel, and some necko components use it as well.

IPCBlobInputStream needs to be renamed. I suggest: RemoteLazyInputStream.

Comment 1

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 1 - mozIRemoteLazyInputStream interface, r?smaug

Comment 2

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 2 - RemoteLazyInputStreamStorage, r?smaug

Depends on D80924

Comment 3

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 3 - PRemoteLazyInputStream, r?smaug

Depends on D80925

Comment 4

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 4 - IPDL actors, r?smaug

Depends on D80926

Comment 5

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 5 - thread, r?smaug

Depends on D80927

Comment 6

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 6 - remoteLazyInputStream, r?smaug

Depends on D80928

Comment 7

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 7 - RemoteLazyInputStreamUtils, r?smaug

Depends on D80929

Comment 8

[Andrea Marchesini [:baku]]

Attachment: Bug 1648141 - IPCBlobInputStream to RemoteLazyInputStream - part 8 - mozilla namespace, r?smaug

Depends on D80930

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/749d894dde52
IPCBlobInputStream to RemoteLazyInputStream - part 1 - mozIRemoteLazyInputStream interface, r=smaug
https://hg.mozilla.org/integration/autoland/rev/24fdb83db3cd
IPCBlobInputStream to RemoteLazyInputStream - part 2 - RemoteLazyInputStreamStorage, r=smaug
https://hg.mozilla.org/integration/autoland/rev/c1a3d5fa0c61
IPCBlobInputStream to RemoteLazyInputStream - part 3 - PRemoteLazyInputStream, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/e5b8292e7095
IPCBlobInputStream to RemoteLazyInputStream - part 4 - IPDL actors, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/d824d2f67f27
IPCBlobInputStream to RemoteLazyInputStream - part 5 - thread, r=smaug
https://hg.mozilla.org/integration/autoland/rev/11dfce46ec14
IPCBlobInputStream to RemoteLazyInputStream - part 6 - remoteLazyInputStream, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/b9bb847cee47
IPCBlobInputStream to RemoteLazyInputStream - part 7 - RemoteLazyInputStreamUtils, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/e9b4ca0ee700
IPCBlobInputStream to RemoteLazyInputStream - part 8 - mozilla namespace, r=smaug,necko-reviewers,dragana

Comment 10

[Razvan Maries]

Backed out for build bustages.

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedTaskRun=RtL6xSX0QPSueaSJPM5Qsg.0&resultStatus=testfailed%2Cbusted%2Cexception&revision=e9b4ca0ee7002e3413086237421d2c68d6b17af5

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307840612&repo=autoland&lineNumber=34033

Backout: https://hg.mozilla.org/integration/autoland/rev/21c88acfbb4dc0e7c3f1cb670bc33982c8ad9fbd

Comment 11

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f0a9b0a0df2d
IPCBlobInputStream to RemoteLazyInputStream - part 1 - mozIRemoteLazyInputStream interface, r=smaug
https://hg.mozilla.org/integration/autoland/rev/0c753cdd5c53
IPCBlobInputStream to RemoteLazyInputStream - part 2 - RemoteLazyInputStreamStorage, r=smaug
https://hg.mozilla.org/integration/autoland/rev/05e418cb5770
IPCBlobInputStream to RemoteLazyInputStream - part 3 - PRemoteLazyInputStream, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/ca622eac3cdc
IPCBlobInputStream to RemoteLazyInputStream - part 4 - IPDL actors, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/bf10fbdfcc0f
IPCBlobInputStream to RemoteLazyInputStream - part 5 - thread, r=smaug
https://hg.mozilla.org/integration/autoland/rev/a791379137ac
IPCBlobInputStream to RemoteLazyInputStream - part 6 - remoteLazyInputStream, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/dd2100135c4a
IPCBlobInputStream to RemoteLazyInputStream - part 7 - RemoteLazyInputStreamUtils, r=smaug,necko-reviewers,dragana
https://hg.mozilla.org/integration/autoland/rev/2e11bff9e6a9
IPCBlobInputStream to RemoteLazyInputStream - part 8 - mozilla namespace, r=smaug,necko-reviewers,dragana

Comment 12

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/f0a9b0a0df2d
https://hg.mozilla.org/mozilla-central/rev/0c753cdd5c53
https://hg.mozilla.org/mozilla-central/rev/05e418cb5770
https://hg.mozilla.org/mozilla-central/rev/ca622eac3cdc
https://hg.mozilla.org/mozilla-central/rev/bf10fbdfcc0f
https://hg.mozilla.org/mozilla-central/rev/a791379137ac
https://hg.mozilla.org/mozilla-central/rev/dd2100135c4a
https://hg.mozilla.org/mozilla-central/rev/2e11bff9e6a9