Closed Bug 653201 Opened 9 years ago Closed 9 years ago

Remove expose image / maemo specific npapi support

Categories

(Core :: Plug-ins, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dougt, Assigned: dougt)

References

()

Details

Attachments

(2 files)

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: nobody → doug.turner
Attached patch patch v.1Splinter Review
Attachment #533979 - Flags: review?(blassey.bugs)
Attachment #533979 - Flags: review?(joshmoz)
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+
pushed to mozilla-central

http://hg.mozilla.org/mozilla-central/rev/c560ce170534
Status: NEW → RESOLVED
Closed: 9 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...
(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.