653794 - improve management of plugin MIME types, MIME descriptions, and extensions

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[Josh Aas]

Attachment: fix v1.0

We should improve management of plugin MIME types, MIME descriptions, and extensions. The array management is error prone.

Comment 1

[Josh Aas]

Attachment: fix v1.1

Comment 2

[Josh Aas]

Attachment: fix v1.2

Comment 3

[Josh Aas]

Comment on attachment 529177 [details] [diff] [review]
fix v1.2

This makes our MIME mgmt code more efficient, consistent, and a lot easier to read/understand.

Comment 4

[Josh Aas]

It also has some fixes for MIME information potentially getting out of sync (type vs. descriptions vs. extensions).

Comment 5

[Josh Aas]

Attachment: fix v1.3

Build fixes.

Comment 6

[Josh Aas]

pushed to mozilla-central

http://hg.mozilla.org/mozilla-central/rev/6824e96ca32a

Comment 7

[Jim Jeffery not reading bug-mail 1/2/11]

I don't have a hard regression range, but the latest m-c win32 hourly with this patch hangs on startup.  Disabling OOPP allows the browser to start. 

Possible this patch has broken plugin-container on windows ?

Comment 8

[Josh Aas]

Anything is possible but tests pass and I ran it locally with no problems. Keep us updated with any new information. Maybe post some more details about your system in an attached text file, including installed browser plugins (about:plugins) and your OS version.

Comment 9

[Jim Jeffery not reading bug-mail 1/2/11]

Well at the moment I'm at work, and won't get to find a true regress-range till sometime tomorrow.

system is Win7 x64 running on AMD Phenom II Quad 8 gig ram
plugins are all up to date with the latest Flash version.

Using the win32 m-c hourly build with your cset from here:
http://ftp.mozilla.org/pub/mozilla.org/firefox/tinderbox-builds/mozilla-central-win32/1305037440/

The hourly before this one, or perhaps two before this one is OK, so its either this patch or the patch-set that re-landed all the stuff backed out yesterday:
http://hg.mozilla.org/mozilla-central/rev/8d378453a8ac

I'm pretty sure that cset is OK, but as I said - I'm not at my machine right-now.

Comment 10

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Neither the 32 bit zip build from http://hg.mozilla.org/mozilla-central/rev/6824e96ca32a nor the 32 bit zip build from the prior push http://hg.mozilla.org/mozilla-central/rev/83ca7e971857 is hanging on startup for me with a clean profile on Win7x64.

Comment 11

[Jim Jeffery not reading bug-mail 1/2/11]

(In reply to comment #10)
> Neither the 32 bit zip build from
> http://hg.mozilla.org/mozilla-central/rev/6824e96ca32a nor the 32 bit zip
> build from the prior push
> http://hg.mozilla.org/mozilla-central/rev/83ca7e971857 is hanging on startup
> for me with a clean profile on Win7x64.

Even if you open a page with flash on it ?

Comment 12

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

(In reply to comment #11)
> Even if you open a page with flash on it ?

Watched a youtube video. Pretty sure it was Flash, lemme try again.