Folder name is not updated when creating a folder and/or editing name in Properties

RESOLVED FIXED in Firefox 4.0b1

Status

()

defect
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: alice0775, Assigned: mak)

Tracking

({regression})

Trunk
Firefox 4.0b1
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a6pre) Gecko/20100619 Minefield/3.7a6pre ID:20100619040308
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a6pre) Gecko/20100619 Minefield/3.7a6pre ID:20100619040308

This happens only in "Bookmarks" menu,

Folder name does not update after create a folder into "Bookmarks" menu, the name of folder is[Folder Name] not a edited name.
And folder name does not update after edit name in "Properties" dialog.

After restart Browser the name folder is updated properly.

Reproducible: Always

Steps to Reproduce 1:
1. Start Minefield with new profile
2. "Bookmarks" > right click on any bookmark > Choose "New Folder..."
3. Edit folder name in "Name" field.and Click "Add".
4. Open "Bookmarks" menu  and you see 

Steps to Reproduce 2:
1. Start Minefield with new profile
2. Open 2-3 tabs with defferent URL
2. Right click on tab strip. And Choose "Bookmark All Tabs..."
3. Edit folder name in "Name" field.and Click "Add Bookmarks".
4. Open "Bookmarks" menu  and you see 

Actual Results:
 The created folder name appears as "Folder Name"

Expected Results:
 The created folder name should appears as a edited name.

Steps to Reproduce 3:
1. Start Minefield with new profile
2. "Bookmarks" > right click on a folder > Choose "Properties"
3. Edit folder name in "Name" field.And click "Save".
4. Open "Bookmarks" menu  and you see 

Actual Results:
 The folder name is not updated

Expected Results:
 The folder name should be updated

Regression Window:

Works:
http://hg.mozilla.org/mozilla-central/rev/00955067e4b5
Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a5pre) Gecko/20100504 Minefield/3.7a5pre ID:20100505053644

Fails:
http://hg.mozilla.org/mozilla-central/rev/d7393e28fb2d
Mozilla/5.0 (Windows; U; Windows NT 6.1; WOW64; en-US; rv:1.9.3a5pre) Gecko/20100504 Minefield/3.7a5pre ID:20100505071655

Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=00955067e4b5&tochange=d7393e28fb2d
(Reporter)

Updated

9 years ago
Summary: Folder name does not update when create a folter and/or edit name in Properties → Folder name is not update when create a folter and/or edit name in Properties
(Reporter)

Comment 1

9 years ago
Rebert to 11a25fc022ba in local build, then the problem does not happen.
Landing a patch of Bug 528884 causes the problem.
Blocks: 528884
(Reporter)

Comment 2

9 years ago
In adittion to comment#0
This happens chevron's popup also.
Blocks: 560198
(Reporter)

Updated

9 years ago
Summary: Folder name is not update when create a folter and/or edit name in Properties → Folder name is not update when create a folder and/or edit name in Properties
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Flags: in-testsuite?
OS: Windows 7 → All
Hardware: x86 → All
Posted patch patch v1.0Splinter Review
If prefer if this handler passes through the base view impl, even if that means duplicating a couple ifs.

The tests could probably be written in a more clear fashion, but it's not a priority atm, it just works.

There are also a couple fixes for js and cpp warning that I saw during the test.
Attachment #452724 - Flags: review?(dietrich)
Attachment #452724 - Flags: review?(dietrich) → review+
Summary: Folder name is not update when create a folder and/or edit name in Properties → Folder name is not updated when creating a folder and/or editing name in Properties
http://hg.mozilla.org/mozilla-central/rev/b6678d36e1a4
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Flags: in-testsuite? → in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.7a6
Depends on: 575558
You need to log in before you can comment on or make changes to this bug.