Closed Bug 1039935 Opened 10 years ago Closed 10 years ago

[User Story] Re-implement NFC URL sharing in System Browser

Categories

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

x86
macOS
defect
Not set
normal

Tracking

(feature-b2g:2.1, tracking-b2g:backlog)

VERIFIED FIXED
2.1 S2 (15aug)
feature-b2g 2.1
tracking-b2g backlog

People

(Reporter: pdol, Assigned: gduan, NeedInfo)

References

Details

(Keywords: feature, Whiteboard: [ucid:System232], [systemsfe])

Attachments

(1 file)

As a user, I want to be able to make use of NFC URL sharing in the System Browser as I was previously able to do in the 2.0 Browser so that I do not lose functionality. Acceptance Criteria: 1. NFC URL sharing in System Browser matches experience in 2.0 Browser.
Saminath, are you able to help with this bug since you did the original Browser implementation?
Flags: needinfo?(s.x.veerapandian)
Garner, do you know if the shrinking UI needs to be ported from the old Browser into the System Browser?
Flags: needinfo?(dgarnerlee)
(In reply to Peter Dolanjski [:pdol] from comment #2) > Garner, do you know if the shrinking UI needs to be ported from the old > Browser into the System Browser? ShrinkingUI is still a system "dialog", and not yet a separate application only UI, so much of this should be hidden from the webapp currently. I haven't actually verified it works yet. The system browser when it is used would require NFC privilages to be added if not already at least so it can access the NFC DOM. There was a separate thread on a more lmited audience NFC mailing list on whether there is any distinction between browserWindow and the appWindow. If they have the same ability to handle events, can receive an "onpeerready" event, and have an ability for the NFC Manager to send that event right back to the same parent gecko process, then the webapp content should still handle the event callback.
Flags: needinfo?(dgarnerlee)
QA Whiteboard: [COM=NFC]
I had some idea and we need to experiment. George, please take this one. The following are from my post in fxos-nfc channel: """ Hi all, The bug is not filed yet, but in coming v2.1 we are removing the browser app and use system browser to replace it. So that means the NFC share inside browser app needs to be done in system app. I know we have a plan to make app itself do its own nfc pairing UI, but its ETA is still unknown and we need to make sure the web page pairing feature is not broken. AFAIK, the new process will be: 1. tech-found event in nfc-manager. 2. Start event sent to shrinkUI. 3. ShrinkUI checks if the current app window is a browserWindow or a AppWindow. (by checking its manifestURL). 3a. if it’s an appWindow, tell nfc-manager to send nfc event to the app. 3b. if it’s a browserWindow, call mozNFC.sendmessage inside system app with the browserWindow’s url. I don’t know if 3b. works. Does mozNFC send message needs to be done in the onpeerfound event handler anyway? If not, could we send the manifestURL of system app to gecko to tell it to send onpeerfound to system app? ""
Flags: needinfo?(gduan)
Component: Gaia::System::Browser Chrome → Gaia::System::Window Mgmt
Assignee: nobody → gduan
Flags: needinfo?(gduan)
When passing apps://system.gaiamobile.org/manifest.webapp to checkP2PRegistration function may get unexpected result. I'm still investigating on it.
Attached file PR to master (deleted) —
Whiteboard: [ucid:System232], [ft:systemsfe] → [ucid:System232], [systemsfe]
Target Milestone: --- → 2.1 S2 (15aug)
Comment on attachment 8469096 [details] PR to master Hi Alive, could you help to check this patch and let me know your concern? Thanks.
Attachment #8469096 - Flags: review?(alive)
Comment on attachment 8469096 [details] PR to master Please make another nfc_handler.js to deal with onpeerready event. And new it in AppWindowManager. var AppWindowManager = { init: function() { new NfcHandler(this); } } var NfcHandler = function (appWindowManager) { this.appWindowManager = appWindowManager; }
Attachment #8469096 - Flags: review?(alive) → feedback+
Comment on attachment 8469096 [details] PR to master Hi Alive, I move the logic to nfc_handler.js and add tests. Could you check again? Thanks.
Attachment #8469096 - Flags: review?(alive)
Attachment #8469096 - Flags: review?(alive) → review+
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Verified on Gaia-Rev 477a9e61c3edf12f32a62a19d329cd277202cc6b Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/67573e422a0f Build-ID 20141015161201 Version 34.0
Status: RESOLVED → VERIFIED
blocking-b2g: backlog → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: