2001-09-29 00:14:13 +04:00
|
|
|
/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
|
2012-05-21 15:12:37 +04:00
|
|
|
/* This Source Code Form is subject to the terms of the Mozilla Public
|
|
|
|
* License, v. 2.0. If a copy of the MPL was not distributed with this
|
|
|
|
* file, You can obtain one at http://mozilla.org/MPL/2.0/. */
|
2001-07-31 23:05:34 +04:00
|
|
|
|
2002-12-11 03:01:36 +03:00
|
|
|
|
Bug 1496581 - Split nsISeekableStream in 2 classes: nsISeekableStream and nsITellableStream, f=mayhemer, r=froydnj
In the current code there are 3 main issues:
1. nsFileStream is not really thread-safe. There is nothing to protect the
internal members and we see crashes.
2. nsPipeInputStream doesn't implement ::Seek() method and that caused issues
in devtools when a nsHttpChannel sends POST data using a pipe. In order to fix
this, bug 1494176 added a check in nsHttpChannel: if the stream doesn't
implement ::Seek(), let's clone it. This was an hack around nsPipeInputStream,
and it's bad.
3. When nsHttpChannel sends POST data using a file stream, nsFileStream does
I/O on main-thread because of the issue 2. Plus, ::Seek() is called on the
main-thread causing issue 1.
Note that nsPipeInputStream implements only ::Tell(), of the nsISeekableStream
methods. It doesn't implement ::Seek() and it doesn't implement ::SetEOF().
With this patch I want to fix point 2 and point 3 (and consequentially issue 1
- but we need a separate fix for it - follow up). The patch does:
1. it splits nsISeekableStream in 2 interfaces: nsITellableStream and
nsISeekableStream.
2. nsPipeInputStream implements only nsITellableStream. Doing this, we don't
need the ::Seek() check for point 2 in nsHttpChannel: a simple QI check is
enough.
3. Because we don't call ::Seek() in nsHttpChannel, nsFileStream doesn't do I/O
on the main-thread, and we don't crash doing so.
2018-10-18 14:35:35 +03:00
|
|
|
#include "nsITellableStream.idl"
|
|
|
|
|
2002-12-11 03:01:36 +03:00
|
|
|
/*
|
|
|
|
* nsISeekableStream
|
|
|
|
*
|
|
|
|
* Note that a stream might not implement all methods (e.g., a readonly stream
|
|
|
|
* won't implement setEOF)
|
|
|
|
*/
|
|
|
|
|
2001-07-31 23:05:34 +04:00
|
|
|
#include "nsISupports.idl"
|
|
|
|
|
2004-04-14 03:08:59 +04:00
|
|
|
[scriptable, uuid(8429d350-1040-4661-8b71-f2a6ba455980)]
|
Bug 1496581 - Split nsISeekableStream in 2 classes: nsISeekableStream and nsITellableStream, f=mayhemer, r=froydnj
In the current code there are 3 main issues:
1. nsFileStream is not really thread-safe. There is nothing to protect the
internal members and we see crashes.
2. nsPipeInputStream doesn't implement ::Seek() method and that caused issues
in devtools when a nsHttpChannel sends POST data using a pipe. In order to fix
this, bug 1494176 added a check in nsHttpChannel: if the stream doesn't
implement ::Seek(), let's clone it. This was an hack around nsPipeInputStream,
and it's bad.
3. When nsHttpChannel sends POST data using a file stream, nsFileStream does
I/O on main-thread because of the issue 2. Plus, ::Seek() is called on the
main-thread causing issue 1.
Note that nsPipeInputStream implements only ::Tell(), of the nsISeekableStream
methods. It doesn't implement ::Seek() and it doesn't implement ::SetEOF().
With this patch I want to fix point 2 and point 3 (and consequentially issue 1
- but we need a separate fix for it - follow up). The patch does:
1. it splits nsISeekableStream in 2 interfaces: nsITellableStream and
nsISeekableStream.
2. nsPipeInputStream implements only nsITellableStream. Doing this, we don't
need the ::Seek() check for point 2 in nsHttpChannel: a simple QI check is
enough.
3. Because we don't call ::Seek() in nsHttpChannel, nsFileStream doesn't do I/O
on the main-thread, and we don't crash doing so.
2018-10-18 14:35:35 +03:00
|
|
|
interface nsISeekableStream : nsITellableStream
|
2001-07-31 23:05:34 +04:00
|
|
|
{
|
2002-12-11 03:01:36 +03:00
|
|
|
/*
|
|
|
|
* Sets the stream pointer to the value of the 'offset' parameter
|
|
|
|
*/
|
2012-08-22 19:56:38 +04:00
|
|
|
const int32_t NS_SEEK_SET = 0;
|
2002-12-11 03:01:36 +03:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Sets the stream pointer to its current location plus the value
|
|
|
|
* of the offset parameter.
|
|
|
|
*/
|
2012-08-22 19:56:38 +04:00
|
|
|
const int32_t NS_SEEK_CUR = 1;
|
2002-12-11 03:01:36 +03:00
|
|
|
|
|
|
|
/*
|
|
|
|
* Sets the stream pointer to the size of the stream plus the value
|
|
|
|
* of the offset parameter.
|
|
|
|
*/
|
2012-08-22 19:56:38 +04:00
|
|
|
const int32_t NS_SEEK_END = 2;
|
2001-07-31 23:05:34 +04:00
|
|
|
|
2002-12-11 03:01:36 +03:00
|
|
|
/**
|
|
|
|
* seek
|
|
|
|
*
|
|
|
|
* This method moves the stream offset of the steam implementing this
|
|
|
|
* interface.
|
|
|
|
*
|
|
|
|
* @param whence specifies how to interpret the 'offset' parameter in
|
|
|
|
* setting the stream offset associated with the implementing
|
|
|
|
* stream.
|
|
|
|
*
|
|
|
|
* @param offset specifies a value, in bytes, that is used in conjunction
|
|
|
|
* with the 'whence' parameter to set the stream offset of the
|
|
|
|
* implementing stream. A negative value causes seeking in
|
|
|
|
* the reverse direction.
|
2006-01-02 05:30:32 +03:00
|
|
|
*
|
|
|
|
* @throws NS_BASE_STREAM_CLOSED if called on a closed stream.
|
2002-12-11 03:01:36 +03:00
|
|
|
*/
|
2004-04-13 18:37:53 +04:00
|
|
|
void seek(in long whence, in long long offset);
|
2002-12-11 03:01:36 +03:00
|
|
|
|
|
|
|
/**
|
|
|
|
* setEOF
|
|
|
|
*
|
|
|
|
* This method truncates the stream at the current offset.
|
2006-01-02 05:30:32 +03:00
|
|
|
*
|
|
|
|
* @throws NS_BASE_STREAM_CLOSED if called on a closed stream.
|
2002-12-11 03:01:36 +03:00
|
|
|
*/
|
2001-07-31 23:05:34 +04:00
|
|
|
void setEOF();
|
|
|
|
};
|