If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Extensions currently not included in WinCE XULRunner Build

RESOLVED WONTFIX

Status

()

Toolkit
Add-ons Manager
RESOLVED WONTFIX
9 years ago
9 years ago

People

(Reporter: wolfe, Unassigned)

Tracking

({mobile})

Trunk
x86
Windows CE
mobile
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Reporter)

Description

9 years ago
Created attachment 328794 [details] [diff] [review]
Comment out building and including extensions for WinCE builds

WinCE does not use Extensions, so they should not be compiled or referenced in WinCE builds.
Why wouldn't we want the extension manager on WinCE? It's a pretty important part of our platform...
Component: General → Extension/Theme Manager
Product: Fennec → Firefox
QA Contact: general → extension.manager
Version: 1.0 → Trunk
(In reply to comment #1)
> Why wouldn't we want the extension manager on WinCE? It's a pretty important
> part of our platform...
> 

Agreed
(Reporter)

Comment 3

9 years ago
Doug, Brad, and I disabled Extensions to try to get XULRunner up and running initially.  After getting XULRunner working, then we can come back and make extensions work again.
Summary: Extensions should not be included in WinCE Build → Extensions currently not included in WinCE XULRunner Build
What was not working with the extension manager enabled?

Updated

9 years ago
Keywords: mobile

Comment 5

9 years ago
Comment on attachment 328794 [details] [diff] [review]
Comment out building and including extensions for WinCE builds

i'd rather not just disable extensions.  can we fix whatever was broken here?
Attachment #328794 - Flags: review-
(Reporter)

Comment 6

9 years ago
Created attachment 329596 [details] [diff] [review]
Same DIFF, just done in Unified instead of Context formatting
Attachment #328794 - Attachment is obsolete: true
(Reporter)

Comment 7

9 years ago
I just want to get something working first, then start adding in complexity.

Right now, we are having trouble getting startup and a single rendering working - so this patch is a temporary fix for any developers who want to play around with Windows Mobile builds.  It does not have to be landed, I suppose, but just serve as a place marker so that we remember to re-enable extensions and debug them before declaring victory on creating a functional XULRunner.exe

Comment 8

9 years ago
Comment on attachment 329596 [details] [diff] [review]
Same DIFF, just done in Unified instead of Context formatting

I agree that what you've done is a good starting point, but I don't think it should land on mozilla-central in this early state.
Attachment #329596 - Flags: review-
(Assignee)

Updated

9 years ago
Product: Firefox → Toolkit
If there is an actual problem with the extension manager in WinCE then please file that and we can figure out how to fix it. Without any knowledge of the problems though we aren't going to just disable it by default.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WONTFIX
(Reporter)

Comment 10

9 years ago
Created attachment 337359 [details] [diff] [review]
Same patch file, updated to work against mozilla-central repository

This patch is submitted for WinMobile developers who want to turn off extensions temporarily.
Attachment #329596 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.