Ticket #2593 (closed defect: wontfix)

Opened 2 years ago

Last modified 13 months ago

[dx2] Browse opens file chooser instead of journal entry selector

Reported by: m_anish Owned by: lucian
Priority: Urgent Milestone: not applicable
Component: Browse Version: Unspecified
Severity: Minor Keywords: dx2
Cc: dirakx, alsroot, lucian Distribution/OS: Dextrose
Bug Status: New

Description (last modified by m_anish) (diff)

Testcase
1. Open Browse
2. Log into GMail
3. Compose a new message
4. Try to add an attachment
5. Instead of Journal entry selector, browse opens the file chooser dialog.

This issue exists on Browse 115/116 running on Dextrose-2, but doesn't seem to be present on jhbuild.

Attachments

org.laptop.WebActivity-1.log Download (61.7 KB) - added by lucian 2 years ago.
Log of Browse running while this bug is reproduced.
org.laptop.WebActivity-2.log Download (32.1 KB) - added by lucian 2 years ago.
Log of a regular run of Browse

Change History

  Changed 2 years ago by m_anish

  • priority changed from Urgent to Immediate
  • severity changed from Major to Blocker

  Changed 2 years ago by alsroot

  • cc alsroot added
  • priority changed from Immediate to Urgent
  • severity changed from Blocker to Major

That issue is absent in v119 on dx-2

follow-up: ↓ 4   Changed 2 years ago by lucian

Could you please upload a log of 115/116 running on dx-2? If it's fixed in 119, I'd like to know why.

in reply to: ↑ 3   Changed 2 years ago by m_anish

  • description modified (diff)
  • severity changed from Major to Minor

Replying to lucian:

Could you please upload a log of 115/116 running on dx-2? If it's fixed in 119, I'd like to know why.

Update (Test results Browse-119 on a XO-1; tests performed in the order as below):

(1) Objectchooser works correctly on wiki.sl.o
(2) Objectchooser works correctly on gmail.com | Basic HTML view
(3) Objectchooser does NOT work correctly on gmail.com | Standard view -> Pops up standard gnome filepicker instead of journal entry selector
(4) Doing (1) or (2) now popup the standard gnome file picker

I think I got a error/warning from sugar.graphics.objectchooser. Will search and paste log.

Since this exists in only some cases (as opposed to occurring everytime, I'm reducing severity to Minor)

  Changed 2 years ago by lucian

I've managed to reproduce this bug.

I don't really have an idea what's going on, there doesn't seem to be anything relevant in the logs. It may be an obscure mozilla/xpcom bug.

follow-up: ↓ 7   Changed 2 years ago by lucian

I still can't figure out why this happens, but I have found an anomaly with Gmail's markup

<input id=":jg" type="file" name="f_gk5yiy530" class="LIODqc" size="50" tabindex="2" "="">

It sometimes appears as

<input id=":jg" type="file" name="f_gk5yiy530" class="LIODqc" size="50" tabindex="2" ">

in a file inspector.

Furthermore, putting this <input in a separate file doesn't reproduce the issue. This leads me to believe it has something to do with how Gmail manipulates the DOM in gecko/xulrunner.

Obviously Browse/xulrunner shouldn't choke on this, but Gmail should have the right markup anyway. I've reported the bug with Google.

in reply to: ↑ 6   Changed 2 years ago by lucian

Replying to lucian:

in a file inspector.

I meant in a DOM inspector (Webkits/Firebug).

Changed 2 years ago by lucian

Log of Browse running while this bug is reproduced.

Changed 2 years ago by lucian

Log of a regular run of Browse

  Changed 2 years ago by RafaelOrtiz

  • status changed from new to closed
  • resolution set to notsugar

Marking it as non-sugar, would be nice to have a link to the upstream bug.

follow-up: ↓ 10   Changed 2 years ago by lucian

I am not yet certain this is an upstream (i.e. mozilla/xulrunner/xpcom) bug.

About the Gmail bug reports, they're private. I can't get a link to it.

in reply to: ↑ 9 ; follow-up: ↓ 11   Changed 2 years ago by m_anish

Replying to lucian:

I am not yet certain this is an upstream (i.e. mozilla/xulrunner/xpcom) bug.

About the Gmail bug reports, they're private. I can't get a link to it.

For the time being, would including older (or newer) version of xulrunner that works well make sense?

For instance, I don't see the bug with up to date sugar-jhbuild running on a f13 machine.

in reply to: ↑ 10 ; follow-up: ↓ 12   Changed 2 years ago by lucian

Replying to m_anish:

Replying to lucian:

I am not yet certain this is an upstream (i.e. mozilla/xulrunner/xpcom) bug.

About the Gmail bug reports, they're private. I can't get a link to it.


For the time being, would including older (or newer) version of xulrunner that works well make sense?

For instance, I don't see the bug with up to date sugar-jhbuild running on a f13 machine.

If it doesn't happen on a newer xulrunner, that increases my confidence that the bug is there, not in Browse.

I don't really know if newer xulrunner is a feasible solution in the field.

in reply to: ↑ 11   Changed 2 years ago by RafaelOrtiz

Replying to lucian:

Replying to m_anish:

Replying to lucian:

I am not yet certain this is an upstream (i.e. mozilla/xulrunner/xpcom) bug.

About the Gmail bug reports, they're private. I can't get a link to it.


For the time being, would including older (or newer) version of xulrunner that works well make sense?

I'd say go for it, if we can fix the issue with a new xulrunner, might be work to try..we need to know wich version fits best.


For instance, I don't see the bug with up to date sugar-jhbuild running on a f13 machine.


If it doesn't happen on a newer xulrunner, that increases my confidence that the bug is there, not in Browse.

I don't really know if newer xulrunner is a feasible solution in the field.

  Changed 2 years ago by m_anish

  • status changed from closed to reopened
  • resolution notsugar deleted

This issue does not exist on Browse running on the latest f14-11.x development builds. Might be worth exploring what versions of relevant packages are being used there and see if we can do something about it in f11.

Reopening.

  Changed 2 years ago by RafaelOrtiz

  • cc lucian added

  Changed 2 years ago by RafaelOrtiz

I've investigated this, as anish said is not happening in F14

relevant packages versions.

xulrunner-1.9.2.10-1.fc14.1.i686
hulahop-0.7.1-3.fc14.i686

  Changed 2 years ago by RafaelOrtiz

tested in sweets stable=sugar0.88 and testing=sugar0.92, not happening.

  Changed 2 years ago by RafaelOrtiz

  • keywords dx2, dx3 added; dx2 removed
  • milestone changed from Unspecified by Release Team to not applicable

  Changed 2 years ago by RafaelOrtiz

should be fixed on dx3 with the new version of xulrunner.

  Changed 18 months ago by m_anish

  • keywords dx2 added; dx2, dx3 removed

[Keywords -= dx3] This doesn't happen on dx3.

  Changed 13 months ago by manuq

For the relevant ticket for current Browse featuring WebKit, see #3411 . Shall we close this one as obsolete?

  Changed 13 months ago by m_anish

+1 to closing this from my side (wontfix)

  Changed 13 months ago by manuq

  • status changed from reopened to closed
  • resolution set to wontfix
Note: See TracTickets for help on using tickets.