Closed
Bug 653794
Opened 14 years ago
Closed 14 years ago
improve management of plugin MIME types, MIME descriptions, and extensions
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jaas, Assigned: jaas)
Details
Attachments
(2 files, 2 obsolete files)
22.39 KB,
patch
|
benjamin
:
review+
|
Details | Diff | Splinter Review |
23.07 KB,
patch
|
Details | Diff | Splinter Review |
We should improve management of plugin MIME types, MIME descriptions, and extensions. The array management is error prone.
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.
Attachment #529177 -
Attachment is patch: true
Attachment #529177 -
Attachment mime type: text/x-patch → text/plain
Attachment #529177 -
Flags: review?(benjamin)
It also has some fixes for MIME information potentially getting out of sync (type vs. descriptions vs. extensions).
Attachment #529173 -
Attachment is obsolete: true
Updated•14 years ago
|
Attachment #529177 -
Flags: review?(benjamin) → review+
pushed to mozilla-central http://hg.mozilla.org/mozilla-central/rev/6824e96ca32a
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Comment 7•14 years ago
|
||
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 ?
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•14 years ago
|
||
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.
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•14 years ago
|
||
(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 ?
(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.
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•