158376 - Sidebar fails to select a panel when told to do so

Status: VERIFIED FIXED

(SeaMonkey :: Sidebar, defect)

Description

[Samir Gehani]

If you select a tab other than Search in Sidebar (e.g., Bookmarks), then close
the Sidebar (F9), then run a context menu selection search:

    * The Sidebar opens, but
    * The Sidebar tab remains "Bookmarks", not "Search"

Comment 1

[Samir Gehani]

Attachment: Patch v.1.0.

Catch and ignore the exception for the case when we are auto opening the search
tab (and sidebar).  We don't want to focus the search field anyways.

Comment 2

[Samir Gehani]

This fixes the regression caused as a side-effect fixing bug 76621.

Comment 3

[Brian Ryner (not reading)]

Comment on attachment 92009 [details] [diff] [review]
Patch v.1.0.

r=bryner

Comment 4

[Blake Ross]

Comment on attachment 92009 [details] [diff] [review]
Patch v.1.0.

sr=blake

Comment 5

[Samir Gehani]

Mailed drivers for trunk approval.

Comment 6

[Asa Dotzler [:asa]]

Comment on attachment 92009 [details] [diff] [review]
Patch v.1.0.

a=asa (on behalf of drivers) for checkin to 1.1

Comment 7

[Samir Gehani]

Nav triage team: nsbeta1+/adt1

Comment 8

[Samir Gehani]

Checked in on trunk.

Comment 9

[Samir Gehani]

Mailed drivers and adt for branch checkin approval.

Comment 10

[Randell Jesup [:jesup] (needinfo me)]

Unless there's a serious push for this, mozilla1.0.1-

We're very late in the 1.0.1 cycle.  Please ask for 1.0 branch approval (for
1.0.2) after 1.0.1 is cut.

1.1 is already out.

Retargeting.

Comment 11

[shliang]

.

Comment 12

[Asa Dotzler [:asa]]

Comment on attachment 92009 [details] [diff] [review]
Patch v.1.0.

requesting branch approval for shliang (from an email request)

Comment 13

[shliang]

resolving fixed since this is on the trunk

Comment 14

[Grace Bush]

verified on all platforms 3/12 builds