Closed Bug 444487 Opened 12 years ago Closed 12 years ago

Extensions currently not included in WinCE XULRunner Build

Categories

(Toolkit :: Add-ons Manager, defect)

x86
Windows CE
defect
Not set

Tracking

()

RESOLVED WONTFIX

People

(Reporter: wolfe, Unassigned)

Details

(Keywords: mobile)

Attachments

(1 file, 2 obsolete files)

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
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?
Keywords: mobile
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-
Attachment #328794 - Attachment is obsolete: true
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 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-
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
Closed: 12 years ago
Resolution: --- → WONTFIX
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.