Session Close window appears when tagging a bookmark

VERIFIED FIXED

Status

Firefox for Android Graveyard
Bookmarks
VERIFIED FIXED
8 years ago
5 years ago

People

(Reporter: tchung, Assigned: mbrubeck)

Tracking

Details

Attachments

(2 attachments)

(Reporter)

Description

8 years ago
Created attachment 436214 [details]
session close screenshot

The session close window will always appear when trying to add a tag to a new bookmark.   We shouldn't be seeing this.

Screenshot.  Tested on both desktop builds and n900 build.

Repro:
1) Download 1.1a2pre 20100330 build
2) open any unbookmarked webpage (eg. amazon)
3) bookmark the site
4) in the bookmark window, click edit tags 
5) enter in something
6) Click done, and verify the session close window will always appear after saving a bookmark

Expected;
- no session close window here

Actual:
- session close window
(Reporter)

Updated

8 years ago
tracking-fennec: --- → ?
I am having trouble reproducing this bug. This are working ok for me.
This works for me as well on builds:

Mozilla/5.0 (X11; U; Linux armv7l; Nokia N900; en-US; rv:1.9.2.4pre) Gecko/20100406 Namoroka/3.6.4pre Fennec/1.1a2pre

and

Mozilla/5.0 (X11; U; Linux armv6l; en-US; rv:1.9.3a4pre) Gecko/20100406 Namoroka/3.7a4pre Fennec/1.1a2pre


Tony, can you try this with a new profile?
(Assignee)

Updated

8 years ago
Assignee: nobody → mbrubeck
(Assignee)

Comment 3

8 years ago
I can reproduce this in my Linux desktop trunk build, with slightly different repro steps:

1. Open at least two tabs.
2. In one of the tabs, bookmark a page OR load a bookmarked page.
3. Press the bookmark star and then press the "Edit" button.
4. Press "Done" in the boomark edit dialog.

This is also causing a timeout failure in chrome/tests/browser_bookmarks_star.js.
(Assignee)

Comment 4

8 years ago
Here's the stack trace.  The "close" event when the bookmark dialog closes is bubbling up and triggering the browser.xul onclose handler.

browser/content/browser.js :: closing :: line 549
browser/content/browser.xul :: onclose :: line 1
browser/content/bindings.xml :: stopEditing :: line 488
browser/content/browser-ui.js :: BH_save :: line 1164
browser/content/browser.xul :: oncommand :: line 1
Status: NEW → ASSIGNED
(Assignee)

Comment 5

8 years ago
Created attachment 437605 [details] [diff] [review]
patch

This patch disables bubbling on the placeitem "close" event.

I checked that nothing expects this event to bubble.  BookmarkHelper attaches its handler directly to the placeitem where the event is dispatched, and there are no close handlers on any of the parent elements (#bookmark-form, #bookmark-container, #stack).
Attachment #437605 - Flags: review?(mark.finkle)
Attachment #437605 - Flags: review?(mark.finkle) → review+
pushed:
http://hg.mozilla.org/mobile-browser/rev/59949ddbc133
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
(Reporter)

Comment 7

8 years ago
Verified fix on N900: Mozilla/5.0 (X11; U; Linux armv7l; en-US;
rv:1.9.2.4pre) Gecko/20100411 Namoroka/3.6.4pre Fennec/1.1a2pre 
and Desktop: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.6; en-US;
rv:1.9.2.4pre) Gecko/20100412 Namoroka/3.6.4pre Fennec/1.1a2pre
Status: RESOLVED → VERIFIED
tracking-fennec: ? → ---
You need to log in before you can comment on or make changes to this bug.