Closed Bug 1203726 Opened 9 years ago Closed 9 years ago

[Gallery] Sharing Image through Gallery app gives an user a link to a web site not an image.

Categories

(Firefox OS Graveyard :: Gaia::System::Browser Chrome, defect, P2)

Product:
Firefox OS Graveyard
Component:
Gaia::System::Browser Chrome
Platform:
ARM
Gonk (Firefox OS)
Type:
defect

Tracking

(blocking-b2g:2.5+, b2g-v2.2 unaffected, b2g-master affected)

Status:
RESOLVED WORKSFORME
Project Flags:
blocking-b2g 2.5+
Tracking Flags:
Tracking Status
b2g-v2.2 --- unaffected
b2g-master --- affected

People

(Reporter: vbelonenko, Assigned: boris)

References

(
URL
)

Details

(Keywords: regression, Whiteboard: [2.5-Daily-Testing][Spark])

Attachments

(1 file)

logcat_20150910_1432.txt
83.35 KB, text/plain
Details 
Description:
When user tries to share an image inside the gallery app, it gives him a link to a site and not to picture.


Repro Steps:
1) Update a  to Unknown
2) Launch a Gallery app 
3) Click on Image and press Share Image button.
4) Press on Messages link.
5) Observe that there's no image inside a message entry field.

Actual:
User could not share image due to displaying a site link not  an actual image.
 

Expected:
Picture should appear inside SMS message not a link, when sharing an image to another user. 

Environmental Variables:
Device: Aries 2.5 kk Full Flash (319 mb)
Build ID: 20150909215207
Gaia: 47459eead04385e22f967012b824f5abdddcfb7c
Gecko: dd2a1d737a64d9a3f23714ec5cc623ec8933b51f
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 43.0a1 (2.5)
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

Repro frequency: 3/3
See attached: Video and logcat
This issue  also occurs 2.5 Flame 
Result: Sharing Image through Gallery app gives an user link to the  web site not an image.

Environmental Variables:
Device: Flame 2.5 kk Full Flash (319 mb)
Build ID: 20150909163019
Gaia: 47459eead04385e22f967012b824f5abdddcfb7c
Gecko: dd2a1d737a64d9a3f23714ec5cc623ec8933b51f
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 43.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

-------------------------------------------------------------------------------------------

This issue   does not occurs on 2.2 Flame 
Result: Sharing Image through Gallery app gives an user correct image.

Environmental Variables:
Device: Flame 2.2 kk Full Flash (319 mb)
Build ID: 20150909183023
Gaia: 7a427e0f8aa6c185a9e22358006b97c19435ca4a
Gecko: 0d9c46d01861
Gonk: bd9cb3af2a0354577a6903917bc826489050b40d
Version: 37.0 (2.2)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage+]
status-b2g-v2.2: --- → unaffected
status-b2g-master: --- → affected
Flags: needinfo?(ktucker)
Keywords: regression
Bad regression of a basic functionality so nominating 2.5+.
blocking-b2g: --- → 2.5?
Flags: needinfo?(ktucker)
Keywords: regressionwindow-wanted
QA Contact: mshuman
This issue appears to be caused by:
Bug 952456 - Implement non-text/rich text support for B2G clipboard

B2g-inbound Regression Window

Last Working 
Environmental Variables:
Device: Flame 2.5
BuildID: 20150821074538
Gaia: 87dfe24140394245b45d4e7afce81006ad7ef399
Gecko: ffcbb3d4e7a3c15e6f7ef6a1e64359eb40d39529
Version: 43.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

First Broken 
Environmental Variables:
Device: Flame 2.5
BuildID: 20150821080238
Gaia: ce2a1760b1d130cb24b53080e67dddf8b68b9de2
Gecko: 993235ea08c7ec40b87b38abfec19288ea99f1f3
Version: 43.0a1 (2.5)
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:43.0) Gecko/43.0 Firefox/43.0

Last Working gaia / First Broken gecko - Issue DOES reproduce
Gaia: 87dfe24140394245b45d4e7afce81006ad7ef399
Gecko: 993235ea08c7ec40b87b38abfec19288ea99f1f3

First Broken gaia / Last Working gecko - Issue does NOT reproduce
Gaia: ce2a1760b1d130cb24b53080e67dddf8b68b9de2
Gecko: ffcbb3d4e7a3c15e6f7ef6a1e64359eb40d39529

Gaia Pushlog:
http://hg.mozilla.org/integration/b2g-inbound/pushloghtml?fromchange=ffcbb3d4e7a3c15e6f7ef6a1e64359eb40d39529&tochange=993235ea08c7ec40b87b38abfec19288ea99f1f3
Blocks: 952456
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(jmercado)
Keywords: regressionwindow-wanted
Boris the changes for bug 952456 seem to have caused this issue.  Can you please take a look?
Flags: needinfo?(jmercado) → needinfo?(boris.chiou)
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
(In reply to Jayme Mercado [:JMercado] from comment #4)
> Boris the changes for bug 952456 seem to have caused this issue.  Can you
> please take a look?

Sure, I will take a look at this problem soon. Thanks for your reproduction.
Flags: needinfo?(boris.chiou)
Assignee: nobody → boris.chiou
See Also: → 1198522
Depends on: 1198522
See Also: 1198522 → 
(In reply to Boris Chiou [:boris] from comment #6)
> (In reply to Boris Chiou [:boris] from comment #5)
> > (In reply to Jayme Mercado [:JMercado] from comment #4)
> > > Boris the changes for bug 952456 seem to have caused this issue.  Can you
> > > please take a look?
> > 
> > Sure, I will take a look at this problem soon. Thanks for your reproduction.
> 
> Actually, I cannot reproduce this bug on my flame-kk built at master branch
> tip today. I can see the image in the message entry field.

Sorry, after checking the video, I think we shouldn't use the "share image" on context menu after long press.

Here is another way to "share" the image in Gallery by message successfully:
1. Launch the Gallery app
2. Click the image
3. Click the share icon on bottom of the screen
4. There are many options we can choose, ex. Wallpaper, Facebook, E-mail, Messages, and others, so just click Message option.
5. We can see the image in the message entry field

I think we have to disable the context menu by long press because it doesn't work on this case, and Bug 1198522 should fix this problem as well.
Regression. Blocks 2.5 with P2 priority
blocking-b2g: 2.5? → 2.5+
Priority: -- → P2
Status: NEW → ASSIGNED
Thanks for working on this, Boris. I've changed the component to reflect that the bug is in the system app, not in the gallery app.
Component: Gaia::Gallery → Gaia::System::Browser Chrome
Can we re-test based on the last sentence in comment 7?
Keywords: qawanted
(In reply to Gregor Wagner [:gwagner] from comment #10)
> Can we re-test based on the last sentence in comment 7?

I believe we can not reproduce this bug on the tip of master branch (mozilla-central). Could anyone re-test this bug? Thank you.
This issue can no longer occur as it required sharing from the long press menu that has been disabled.  Closing this issue as WFM for now.  Please reopen if it occurs again.

Environmental Variables:
Device: Aries 2.5
BuildID: 20151008110640
Gaia: 4973f57cd8f9a62a95f783a24eac32da2bde99fc
Gecko: a955ea9382afedd66ea0da21fcd2932465168d84
Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd
Version: 44.0a1 (2.5) 
Firmware Version: D5803_23.1.A.1.28_NCB.ftf
User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0

Environmental Variables:
Device: Flame 2.5
BuildID: 20151008062742
Gaia: 4973f57cd8f9a62a95f783a24eac32da2bde99fc
Gecko: 1f4cf75c894862cf3634d6014d8de9c807a054a7
Gonk: c4779d6da0f85894b1f78f0351b43f2949e8decd
Version: 44.0a1 (2.5) 
Firmware Version: v18D
User Agent: Mozilla/5.0 (Mobile; rv:44.0) Gecko/44.0 Firefox/44.0
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Resolution: --- → WORKSFORME
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: