Closed
Bug 1190498
Opened 9 years ago
Closed 6 years ago
[Gallery] Sharing photo from Gallery to Facebook does not set photo in post box.
Categories
(Tech Evangelism Graveyard :: Preinstalled B2G Apps, defect)
Tracking
(blocking-b2g:2.5+, b2g-v2.2 affected, b2g-master affected)
RESOLVED
WONTFIX
blocking-b2g | 2.5+ |
People
(Reporter: jthomas, Unassigned)
References
()
Details
(Keywords: foxfood, Whiteboard: [2.5-Daily-Testing][Spark])
Attachments
(1 file)
113.76 KB,
text/plain
|
Details |
Description: If a photo is being shared to Facebook from Gallery or Preview then Facebook will launch without the photo being set within the post box. This is inconsistent when compared to Windows or I-Phone as the photo will already be set within the posting box with the options to write text. Repro Steps: Prereq: Previously logged into Facebook. 1) Update a Flame to 2) Select a Photo within Gallery 3) Select Share > Facebook Actual: User will have to select the photo button and select the photo they want to share a 2nd time. Expected: It is expected that the photo previously selected will appear ready to post when sharing to Facebook. Environmental Variables: Device: Flame 2.5 Kk Fullflash (319mb) Build ID: 20150803030210 Gaia: 2ca27bbdd84526c6a3b198d9cf10f2caff1dadde Gecko: 32712cd01159 Gonk: 41d3e221039d1c4486fc13ff26793a7a39226423 Version: 42.0a1 (2.5) Firmware Version: v18D User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0 User Impact: User will manually have to select the photo they want to share on Facebook through the Facebook website. Repro frequency: 100% See attached: Logcat & Video
Reporter | ||
Comment 2•9 years ago
|
||
This issue does NOT occur on Aries 2.5, Flame 2.2, 2.1 or 2.0 Environmental Variables: Device: Aries 2.5 Kk Build ID: 20150727151800 Gaia: 4e3e21a4ba3f188b45623ee2297f21d0791f8667 Gecko: 21ca97268bae Gonk: 2916e2368074b5383c80bf5a0fba3fc83ba310bd Version: 42.0a1 (2.5) Firmware Version: D5803_23.1.A.1.28_NCB.ftf User Agent: Mozilla/5.0 (Mobile; rv:42.0) Gecko/42.0 Firefox/42.0 Environmental Variables: Device: Flame 2.2 Kk Fullflash (319mb) BuildID: 20150803032504 Gaia: f8b119ac30e97df991c97682ac4d4f9ca22e1793 Gecko: 429b9d2d4566 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 Environmental Variables: Device: Flame 2.1 Kk Fullflash (319mb) Build ID: 20150724001207 Gaia: 9dba58d18006e921546cec62c76074ce81e16518 Gecko: 41e10c6740be Gonk: bd9cb3af2a0354577a6903917bc826489050b40d Version: 34.0 (2.1) Firmware Version: v18D User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0 Environmental Variables: Device: Flame 2.0 Kk Fullflash (319mb) Build ID: 20150723000207 Gaia: b16ba05481e577bc644ed8966f587a70fe2148e6 Gecko: 2e6f1d4deff9 Gonk: bd9cb3af2a0354577a6903917bc826489050b40d Version: 32.0 (2.0) Firmware Version: v18D User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.2:
--- → unaffected
status-b2g-master:
--- → affected
Flags: needinfo?(ktucker)
Keywords: regression
Whiteboard: [2.5-Daily-Testing][Spark]
Reporter | ||
Updated•9 years ago
|
Flags: needinfo?(ktucker) → needinfo?(pbylenga)
Comment 3•9 years ago
|
||
[Blocking Requested - why for this release]: Functional regression. Requesting a window.
blocking-b2g: --- → 2.5?
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(pbylenga)
Keywords: regressionwindow-wanted
Reporter | ||
Comment 5•9 years ago
|
||
I sincerely apologize. I left a typo regarding Comment 2. This issue DOES occur on Aries 2.5, Flame 2.2, 2.1 or 2.0.
Comment 6•9 years ago
|
||
Not a regression.
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Keywords: regression,
regressionwindow-wanted
Updated•9 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Comment 7•9 years ago
|
||
Investigated and share activity is successfully invoked from gallery app and is sending shared blob as data to facebook app. https://github.com/mozilla-b2g/gaia/blob/master/apps/gallery/js/gallery.js#L714 The issue seems to be at the receiving end for handling shared data in facebook app. Setting NI flag for David to help re-direct bug to appropriate component. Thanks!
Flags: needinfo?(dflanagan)
Comment 8•9 years ago
|
||
Isn't that related to bug 1178290 where the picture is named "blob" (when uploading to Flickr)?
Comment 9•9 years ago
|
||
(In reply to Hubert Figuiere [:hub] from comment #8) > Isn't that related to bug 1178290 where the picture is named "blob" (when > uploading to Flickr)? It shouldn't be as bug 1178290 is about downsized and unnamed blob, this issue appears to be displaying the returned shared blob inside facebook app.
Comment 10•9 years ago
|
||
Tested and the shared image from camera app is not getting set on facebook app.
Updated•9 years ago
|
Blocks: b2g-facebook
Updated•9 years ago
|
Assignee: nobody → dflanagan
Comment 12•9 years ago
|
||
I can reproduce this on a foxfooding device, both with the pre-installed Facebook app, and with the Facebook (probably the same thing) from Marketplace. It is not a regression. It is not specific to Gallery, because Camera can't share images with Facebook either. Gallery and Camera can share images with other apps like the SMS app, however. So I'm pretty sure that this is a bug on Facebook's end. I think this is a job for partner engineering. We need to ask Facebook to fix m.facebook.com to correctly handle share activities for FirefoxOS. And if they can't do that, then they should modify https://m.facebook.com/openwebapp/manifest.webapp so that it does not incorrectly state that the app can handle the share activity. And while we're working with Facebook, it might be worth pointing out that the Pick activity from the facebook app does not usually seem to display previews of the images that the user has attached to their post before posting. That seems like a bug, too. Harald: could you or one of your teammates work with Facebook on this?
Flags: needinfo?(dflanagan) → needinfo?(hkirschner)
Comment 13•9 years ago
|
||
Changing the component to Gaia:Foxfooding, since this does seem like a serious foxfooding issue. And unassigning myself, since it is not something I can fix. Doug: if we can't get Facebook to fix this on their end, can we work around it for foxfooders by having a local copy of Facebook's manifest that deletes the share activity instead of linking to https://m.facebook.com/openwebapp/manifest.webapp in distros/spark/remote.list?
Assignee: dflanagan → nobody
Component: Gaia::Gallery → Gaia::Foxfooding
Flags: needinfo?(drs)
Comment 14•9 years ago
|
||
Thank you for investigating this bug. We will contact our partners @ FB and try for a resolution.
Comment 15•9 years ago
|
||
David, Foxfooding isn't the component for that. It is the component for the app we wanted to ship be didn't. (we should remove that component /me think) Doesn't activity require installed apps at least (for permissions)? Isn't the Facebook app just hosted?
Component: Gaia::Foxfooding → Gaia::Gallery
Comment 17•9 years ago
|
||
Correct me if I am wrong but this should be a Tech Evangelism bug. As far as I can understand this is a regression on the FB side. Sharing web activity worked before and correctly populated the composer image attachment. It sounds like this is completely broken by now, which isn't surprising as this probably didn't have any automated tests on the FB side (their testing cycle is mostly dogfooding based).
Flags: needinfo?(hkirschner)
Flags: needinfo?(drs)
Comment 18•9 years ago
|
||
Thanks, Harald. Changing this to tech evangelism. Resetting the needinfo for Doug, though: since we're pre-installing the facebook app on Spark builds, do you want to use a local manifest so you can workaround this by deleting the busted share activity?
Component: Gaia::Gallery → Preinstalled B2G Apps
Flags: needinfo?(drs)
Product: Firefox OS → Tech Evangelism
Comment 19•9 years ago
|
||
When bug 1196992 lands, we'll also be preloading Facebook into the base Gaia build. I'm surfacing this to the product team. In my opinion, we should reach out to Facebook to have them fix this, and avoid building our own custom workarounds which will be difficult to back out of once the issues are fixed.
Flags: needinfo?(drs) → needinfo?(wmathanaraj)
Comment 20•9 years ago
|
||
NI'ed Product team as Wilfred is on vacation. Thanks
Flags: needinfo?(ffos-product)
Updated•9 years ago
|
Flags: needinfo?(wmathanaraj)
Comment 22•9 years ago
|
||
@DRS - This bug had been communicated to the partner. The issue is that we are having a less-than-awesome time getting priority/attention on our bugs. Although a work-around on our side may not be ideal, it may be the fastest route to a resolution.
Comment 23•6 years ago
|
||
B2G: we won't work on that.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
Updated•6 years ago
|
Flags: needinfo?(ffos-product)
Updated•6 years ago
|
Product: Tech Evangelism → Tech Evangelism Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•