Closed Bug 1385170 Opened 7 years ago Closed 7 years ago

show button string by checking if the platform support set default browser in background

Categories

(Firefox :: New Tab Page, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 56
Tracking Status
firefox56 --- fixed

People

(Reporter: gasolin, Assigned: gasolin)

References

(Depends on 1 open bug)

Details

(Whiteboard: [photon-onboarding])

Attachments

(1 file, 1 obsolete file)

In UI spec it described only windows 7 allow set default browser in background, but after QA test in bug 1374717 comment 35 we found Ubuntu also support set default browser in background. That means only checking windows7 is not the right way to go.

We could show the correct string by checking UITour's canSetDefaultBrowserInBackground property for on the getConfiguration("appInfo") response
Whiteboard: [ph
Flags: qe-verify+
Whiteboard: [ph → [photon-onboarding][triage]
Assignee: nobody → gasolin
Status: NEW → ASSIGNED
The PR also fix the issue on window 7, which will show the disabled button instead of 2 lines of alternative message after user click the button.

The expect result should be showing 2 lines of alternative message after user click the button.
The PR send an event when page is rendered and listen in default-browser-tour's getPage method.
Once event is received, it will send another event to ask agent doing UITour check


The previous PR create the extra `processPage` method to call the action after page is rendered,
https://reviewboard.mozilla.org/r/162406/diff/3#index_header

But after sync with Rex, we think listen event in `getPage` might be more flexible than make calls on every entry-point.
Target Milestone: --- → Firefox 56
Priority: -- → P1
Can we reuse the event from bug 1384045 here?
You mean 'beforeshow'? yes, though we still need extra event to communicate with onboarding_agent for UITour related actions.
I'm a bit concerned about listening on the same `beforeshow` event in `gotoPage` function since it might be too late for async showing the button text.

But I found no visual regression when I test it on Macbook Air. So I'll remove my custom event and reuse `beforeshow` event from bug 1384045
Attachment #8891870 - Attachment is obsolete: true
Attachment #8891870 - Flags: review?(dtownsend)
Depends on: 1384045
QA Contact: jwilliams
Whiteboard: [photon-onboarding][triage] → [photon-onboarding]
Comment on attachment 8891191 [details]
Bug 1385170 - show button string by checking if the platform support set default browser in background;

https://reviewboard.mozilla.org/r/162406/#review168484
Attachment #8891191 - Flags: review?(dtownsend) → review+
Thanks! 

For QA please test on at least Windows 7 and Ubuntu to verify it works.
Keywords: checkin-needed
Pushed by rchien@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d000d7961749
show button string by checking if the platform support set default browser in background;r=mossop
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/d000d7961749
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
I have verified that this is fixed.
Status: RESOLVED → VERIFIED
Depends on: 1389558
You need to log in before you can comment on or make changes to this bug.