Closed Bug 235618 Opened 20 years ago Closed 20 years ago

The photon section of modules/libpref/src/init/all.js should be moved after the XP_UNIX section

Categories

(Core Graveyard :: Embedding: GRE Core, defect)

Product:
Core Graveyard
Component:
Embedding: GRE Core
Platform:
x86
Neutrino
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: amardare, Assigned: amardare)

References

(
URL
)

Details

Attachments

(1 file)

Patch
5.24 KB, patch
benjamin
: review+
darin.moz
: superreview+
Details | Diff | Splinter Review 
User-Agent:       
Build Identifier: Mozilla/5.0 (Photon; U; QNX x86pc; en-US; rv:1.6) Gecko/20040214 Firefox/0.8

This relates to the modules/libpref/src/init/all.js file.
Our platform ( photon )  also defines the XP_UNIX manifest, so the all.js will
contain:
... photon section
... xp_unix section

Before the unification was made, the js were executed in this order: all.js
unix.js photon.js
Now the photon and xp_unix changed places.
The photon section should be *after* the XP_UNIX section in the
modules/libpref/src/init/all.js.

Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Attached patch Patch — — Splinter Review 
Assignee: dougt → amardare
Status: NEW → ASSIGNED

    
  

        Attachment #142269 -
        Flags: review?

    
  

        Attachment #142269 -
        Flags: review? → review?(bsmedberg)

    
  

        Attachment #142269 -
        Attachment description: path → Patch

    
  

        Attachment #142269 -
        Flags: superreview?(darin)

        Attachment #142269 -
        Flags: review?(bsmedberg)

        Attachment #142269 -
        Flags: review+

    
  

        Attachment #142269 -
        Flags: superreview?(darin) → superreview+

    
  
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Product: Core → Core Graveyard

          You need to log in
          before you can comment on or make changes to this bug.