Closed Bug 1100282 Opened 10 years ago Closed 10 years ago

[Stingray][System] only allow smart-settings app to handle configure activity

Categories

(Firefox OS Graveyard :: Gaia, defect)

x86
macOS
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
2.2 S1 (5dec)

People

(Reporter: johnhu, Assigned: johnhu)

References

Details

(Whiteboard: [ft:conndevices])

Attachments

(1 file)

To prevent overriding by third-party apps, we should lock the 'configure' activity only to be handled by smart-settings app.
Rex,

This is the patch to lock "configure" activity only to be handled by smart settings. Please review it.
Attachment #8523792 - Flags: review?(rexboy)
Target Milestone: --- → 2.2 S1 (5dec)
Comment on attachment 8523792 [details] [review]
let smart-settings be the only one to handle 'configure' activity

r+ with 1 typo.

Removing DRM logic is temporarily OK for us, but it may become required one day for TV (correct me if i'm wrong). We need to keep in mind or let it become part of comment for this.
Attachment #8523792 - Flags: review?(rexboy) → review+
(In reply to KM Lee [:rexboy][:蠟蠟蠟蠟] from comment #2)
> Removing DRM logic is temporarily OK for us, but it may become required one
> day for TV (correct me if i'm wrong). We need to keep in mind or let it
> become part of comment for this.
No, the DRM staffs are not removed. I had put them in my mapping list. So, I merged the code.


merged to master
https://github.com/mozilla-b2g/gaia/commit/ecab09637388095548c4cd2533e25ff51921cd76

gaia try is almost green with one known error:
https://treeherder.mozilla.org/ui/#/jobs?repo=gaia-try&revision=ba2380ecd484

    Bug 1101227 Intermittent test_system_notification_bar.py TestNotificationBar.test_notification_bar | AssertionError: 'TestNotificationBar_BODY' != u'
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: