Closed Bug 714468 Opened 14 years ago Closed 14 years ago

use require('runtime') in system and xpcom modules

Categories

(Add-on SDK Graveyard :: General, defect, P5)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: erikvvold, Assigned: erikvvold)

Details

Attachments

(1 file)

Atm the system and xpcom modules are creating instances of nsIXULRuntime, but the runtime module already does this and exports all that these other two modules require.
Assignee: nobody → erikvvold
Severity: normal → trivial
Status: NEW → ASSIGNED
OS: Mac OS X → All
Priority: -- → P5
Hardware: x86 → All
Pointer to Github pull-request
Attachment #585156 - Flags: review?(rFobic)
Attachment #585156 - Flags: review?(rFobic) → review?(myk)
Commit pushed to https://github.com/mozilla/addon-sdk https://github.com/mozilla/addon-sdk/commit/4d01e9f6ebeb50b4811715cc99c4b4623cb55012 Merge pull request #314 from erikvold/use-runtime fix bug 714468 - using require('runtime') in system and xpcom modules; r=@mykmelez
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Attachment #585156 - Flags: review?(myk) → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: