1158537 - Remove unneeded FileImplTemporaryFileBlob::{mLength,mContentType} members

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[(no longer active)]

The FileImplBase base class has protected mLength and mContentType
members that this class can use.  These are currently just masking those
base members.

Comment 1

[(no longer active)]

Attachment: Remove unneeded FileImplTemporaryFileBlob::{mLength,mContentType} members

Comment 2

[Andrea Marchesini [:baku]]

Comment on attachment 8597639 [details] [diff] [review]
Remove unneeded FileImplTemporaryFileBlob::{mLength,mContentType} members

Review of attachment 8597639 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/base/File.h
@@ +611,2 @@
>      , mStartPos(aStart)
> +    , mFileDescOwner(aOther->mFileDescOwner) {}

can you put {} in a new line?

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e8e9d876c016

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e8e9d876c016