70099 - Remove new MailNews dependency on Navigator

Status: VERIFIED FIXED

(MailNews Core :: Localization, defect)

Description

[Blake Ross]

Tao, your chrome/ui separation added a MailNews dependency on Navigator.  The 
search panel is available to all apps, and you had it include 
region.properties, located in xpfe/browser.  This needs to move to 
xpfe/communicator so the Search panel continues to work properly from MailNews 
when Navigator isn't installed.

Comment 1

[tao]

there were two copies of this values, one in search-panels.properties
and used in search-panel.js; while the other was in navigator.properties.

I'd suggest we find out if both are valid before moving the url again.

Comment 2

[tao]

Hi, Paul:

Any idea who owns "Search" component now?

reassign to the owner of "search" until we know where the search url should live.

Comment 3

[Blake Ross]

The one in search-panel.properties is the valid one.

Comment 4

[Viswanath Ramachandran]

nav pretriage: moving to Future. priority P5. 

Comment 5

[Blake Ross]

Why is this on Paul's plate?  The person who caused the regression should fix 
it, please.

Comment 6

[Blake Ross]

Doesn't look like this is getting fixed before the freeze tomorrow night. 
Pushing out a milestone.  Please correct if I'm mistaken.

Comment 7

[tao]

Attachment: move search url to communicator

Comment 8

[[e-mail] (away - not reading bugmail)]

this looks OK - r=jbetak

Comment 9

[Scott MacGregor]

sr=mscott

Comment 10

[tao]

Attachment: package the new region.properties into US.jar

Comment 11

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

a=dbaron (on behalf of drivers)

Comment 12

[tao]

thanks, checked in!

Comment 13

[ji]

Verified with 0906 0.9.4 build. It's fixed.