Open Bug 873829 Opened 11 years ago Updated 2 years ago

Enhanced Customization API

Categories

(Core :: General, defect)

24 Branch
x86_64
Linux
defect

Tracking

()

UNCONFIRMED

People

(Reporter: riadh.chtara, Unassigned)

Details

Attachments

(1 file, 18 obsolete files)

176.86 KB, patch
Details | Diff | Splinter Review
Attached patch internalapi.diff (obsolete) — Splinter Review
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.22 (KHTML, like Gecko) Ubuntu Chromium/25.0.1364.160 Chrome/25.0.1364.160 Safari/537.22

Steps to reproduce:

GSOC Enhanced Customization APIs: Internal APIs prototype
Hey guys,
This the prototype of the internal apis I was talking about.
The prototype supports the following things
* synchronous set of a protected pref
* synchronous get of a protected pref
* asynchronous set of a protected pref
* asynchronous get of a protected pref
* synchronous set of many protected prefs
* synchronous get of many protected prefs
* asynchronous set of many protected prefs
* asynchronous get of many protected prefs
There is a UI asking the user for confirmation that is shown when setting same prefs. But this part is not finished yet. 
I will focus on solving same bugs as Kris suggested to me to improve my chances to get selected for gsoc.
If i will be chosen (which will be by the way awesome), I will continue working to improve the internal apis based on the prototype.
This an example of addon that uses the internal apis:

const { Cu } = require("chrome");
Cu.import("resource://gre/modules/EnhancedCustomizationAPIs.jsm");
DBhandler.init();
DBhandler.dbconnect();


DBhandler.dbset("browser.newtab.url", "www.gmail.com" , "{59c81df5-4b7a-477b-912d-4e0fdf64e5f2}", console.log);


(By the way the following lines will be included to a js file that's loaded at firefox start (browser.js) so the developer will not need to include them)  
DBhandler.init();
DBhandler.dbconnect();


DBhandler will be also renamed to CustomizeAPIs or something similar to that
Component: Untriaged → General
Product: Firefox → Core
Attached patch adding ui+message (obsolete) — Splinter Review
Attachment #751442 - Attachment is obsolete: true
Attachment #756541 - Flags: review?
Attached patch adding ui+message (obsolete) — Splinter Review
Attachment #756541 - Attachment is obsolete: true
Attachment #756541 - Flags: review?
Attachment #756544 - Flags: review?(kmaglione+bmo)
Attached image screenshot (obsolete) —
Attachment #756547 - Flags: review?(kmaglione+bmo)
Attachment #756544 - Attachment is obsolete: true
Attachment #756544 - Flags: review?(kmaglione+bmo)
Attachment #757168 - Flags: review?(kmaglione+bmo)
Attachment #757168 - Attachment is obsolete: true
Attachment #757168 - Flags: review?(kmaglione+bmo)
Attachment #758726 - Flags: review?(kmaglione+bmo)
Internal apis almost finished
Attached patch fixing bugs (obsolete) — Splinter Review
Attachment #758726 - Attachment is obsolete: true
Attachment #758726 - Flags: review?(kmaglione+bmo)
Attachment #760109 - Flags: review?(kmaglione+bmo)
Attached patch internal apis (obsolete) — Splinter Review
Attachment #756547 - Attachment is obsolete: true
Attachment #756547 - Flags: review?(kmaglione+bmo)
Attachment #763729 - Flags: review?(kmaglione+bmo)
Internal apis with tests
Attachment #760109 - Attachment is obsolete: true
Attachment #763729 - Attachment is obsolete: true
Attachment #760109 - Flags: review?(kmaglione+bmo)
Attachment #763729 - Flags: review?(kmaglione+bmo)
Attachment #764302 - Flags: review?(kmaglione+bmo)
Attachment #765367 - Flags: review?(kmaglione+bmo)
Attachment #764302 - Attachment is obsolete: true
Attachment #765367 - Attachment is obsolete: true
Attachment #764302 - Flags: review?(kmaglione+bmo)
Attachment #765367 - Flags: review?(kmaglione+bmo)
Attachment #769339 - Flags: review?(kmaglione+bmo)
Attachment #769339 - Attachment is obsolete: true
Attachment #769339 - Flags: review?(kmaglione+bmo)
Attachment #772857 - Flags: review?(kmaglione+bmo)
Attachment #772857 - Attachment is obsolete: true
Attachment #772857 - Flags: review?(kmaglione+bmo)
Attachment #791821 - Flags: review?(kmaglione+bmo)
Attached patch deleteoldentries.diff (obsolete) — Splinter Review
Fixing bugs, tests are now working
Attachment #791821 - Attachment is obsolete: true
Attachment #791821 - Flags: review?(kmaglione+bmo)
Attachment #798305 - Flags: review?(kmaglione+bmo)
Attached patch newcmdmsg.diff (obsolete) — Splinter Review
Attachment #798305 - Attachment is obsolete: true
Attachment #798305 - Flags: review?(kmaglione+bmo)
Attachment #802870 - Flags: review?(kmaglione+bmo)
Attachment #802870 - Attachment is obsolete: true
Attachment #802870 - Flags: review?(kmaglione+bmo)
Attachment #807702 - Flags: review?(kmaglione+bmo)
Adding saving result to file option to external api
Attachment #807702 - Attachment is obsolete: true
Attachment #807702 - Flags: review?(kmaglione+bmo)
Attachment #809675 - Flags: review?(kmaglione+bmo)
Attachment #809675 - Attachment description: src_135816_to_135782.diff → Adding save results to file option
Summary: Enhanced Customization APIs: Internal APIs → Enhanced Customization APIs
Summary: Enhanced Customization APIs → Enhanced Customization API
Attached patch refactoring (obsolete) — Splinter Review
Attachment #809675 - Attachment is obsolete: true
Attachment #809675 - Flags: review?(kmaglione+bmo)
Attachment #810560 - Flags: review?(kmaglione+bmo)
Merging the updates + refactoring
Attachment #810560 - Attachment is obsolete: true
Attachment #810560 - Flags: review?(kmaglione+bmo)
Attachment #813099 - Attachment description: src_148827_to_148833.diff → Merging+ refactoring
Attachment #813099 - Flags: review?(kmaglione+bmo)
Attachment #813099 - Flags: review?(kmaglione+bmo)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: