The default bug view has changed. See this FAQ.

Remove expose image / maemo specific npapi support

RESOLVED FIXED

Status

()

Core
Plug-ins
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: dougt, Assigned: dougt)

Tracking

Trunk
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(2 attachments)

(Assignee)

Description

6 years ago
We added a nokia specific NAPI called Image Expose.  However, we never really shipped a product with flash support -- the only plugin that used this API.  The supporting code is complicated and it basically is dead.  I asked at our mobile engineering meeting to remove it and there were 3 yes's and no no's.
(Assignee)

Updated

6 years ago
Assignee: nobody → doug.turner
(Assignee)

Comment 1

6 years ago
Created attachment 533979 [details] [diff] [review]
patch v.1
Attachment #533979 - Flags: review?(blassey.bugs)
(Assignee)

Updated

6 years ago
Attachment #533979 - Flags: review?(joshmoz)

Comment 2

6 years ago
Comment on attachment 533979 [details] [diff] [review]
patch v.1

Thanks!
Attachment #533979 - Flags: review?(joshmoz) → review+
Comment on attachment 533979 [details] [diff] [review]
patch v.1

Review of attachment 533979 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #533979 - Flags: review?(blassey.bugs) → review+
(Assignee)

Comment 4

6 years ago
try run:  http://tbpl.mozilla.org/?tree=Try&rev=f125c5536cc5
Keywords: checkin-needed

Comment 5

6 years ago
Created attachment 534228 [details] [diff] [review]
updated to current trunk

Comment 6

6 years ago
pushed to mozilla-central

http://hg.mozilla.org/mozilla-central/rev/c560ce170534
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
hmm.. sounds like bug 624165 is the same or similar
sounds like build with patch from bug  624165 looks more stable, and compilable...

Comment 9

6 years ago
(In reply to comment #8)
> sounds like build with patch from bug  624165 looks more stable, and
> compilable...

Why do you say that?
oh, I though busted Bq build is reason of this patch, nm
You need to log in before you can comment on or make changes to this bug.