Add docs

docs/appActions.md

@@ -322,6 +322,50 @@ Clears the data specified in dataDetail
 
 
 
+### addAutofillAddress(detail, originalDetail) 
+
+Add address data
+
+**Parameters**
+
+**detail**: `object`, the address to add as per doc/state.md's autofillAddressDetail
+
+**originalDetail**: `object`, the original address before editing
+
+
+
+### removeAutofillAddress(detail) 
+
+Remove address data
+
+**Parameters**
+
+**detail**: `object`, the address to remove as per doc/state.md's autofillAddressDetail
+
+
+
+### addAutofillCreditCard(detail, originalDetail) 
+
+Add credit card data
+
+**Parameters**
+
+**detail**: `object`, the credit card to add as per doc/state.md's autofillCreditCardDetail
+
+**originalDetail**: `object`, the original credit card before editing
+
+
+
+### removeAutofillCreditCard(detail) 
+
+Remove credit card data
+
+**Parameters**
+
+**detail**: `object`, the credit card to remove as per doc/state.md's autofillCreditCardDetail
+
+
+
 
 * * *
 

docs/state.md

@@ -166,6 +166,27 @@ AppStore
     locale: string, // en_US, en, or any other locale string
     ignoredWords: Array<string>, // List of words to ignore
     addedWords: Array<string> // List of words to add to the dictionary
+  },
+  autofill: {
+    addresses: [{
+      name: string,
+      organization: string,
+      streetAddress: string,
+      city: string,
+      state: string,
+      postalCode: string,
+      country: string,
+      phone: string,
+      email: string,
+      guid: Object.<string, <string>> // map of (partition, id) used to access the autofill entry in database
+    }],
+    creditCards: [{
+      name: string,
+      card: string,
+      month: string,
+      year: string,
+      guid: Object.<string, <string>> // map of (partition, id) used to access the autofill entry in database
+    }]
   }
 }
 ```
@@ -407,6 +428,23 @@ WindowStore
       site: string,
       score: ?
     }
+  },
+  autofillAddressDetail: {
+    name: string,
+    organization: string,
+    streetAddress: string,
+    city: string,
+    state: string,
+    postalCode: string,
+    country: string,
+    phone: string,
+    email: string
+  },
+  autofillCreditCardDetail: {
+    name: string,
+    card: string,
+    month: string,
+    year: string
   }
 }
 ```

docs/windowActions.md

@@ -735,6 +735,30 @@ Sets the clear browsing data popup detail
 
 
 
+### setAutofillAddressDetail(currentDetail, originalDetail) 
+
+Sets the manage autofill address popup detail
+
+**Parameters**
+
+**currentDetail**: `Object`, Properties of the address to change to
+
+**originalDetail**: `Object`, Properties of the address to edit
+
+
+
+### setAutofillCreditCardDetail(currentDetail, originalDetail) 
+
+Sets the manage autofill credit card popup detail
+
+**Parameters**
+
+**currentDetail**: `Object`, Properties of the credit card to change to
+
+**originalDetail**: `Object`, Properties of the credit card to edit
+
+
+
 
 * * *
 

js/about/aboutActions.js

@@ -135,26 +135,52 @@ const AboutActions = {
     ipc.send(messages.LEDGER_ENABLE, enabled)
   },
 
+  /**
+   * Open a adding address dialog
+   */
   addAutofillAddress: function () {
     ipc.sendToHost(messages.ADD_AUTOFILL_ADDRESS)
   },
 
+  /**
+   * Remove address
+   *
+   * @param {object} address - address to remove as per doc/state.md's autofillAddressDetail
+   */
   removeAutofillAddress: function (address) {
     ipc.send(messages.REMOVE_AUTOFILL_ADDRESS, address)
   },
 
+  /**
+   * Open a edit address dialog
+   *
+   * @param {object} address - address to edit as per doc/state.md's autofillAddressDetail
+   */
   editAutofillAddress: function (address) {
     ipc.sendToHost(messages.EDIT_AUTOFILL_ADDRESS, address)
   },
 
+  /**
+   * Open a adding credit card dialog
+   */
   addAutofillCreditCard: function () {
     ipc.sendToHost(messages.ADD_AUTOFILL_CREDIT_CARD)
   },
 
+  /**
+   * Remove credit card
+   *
+   * @param {object} card - credit card to remove as per doc/state.md's autofillCreditCardDetail
+   */
   removeAutofillCreditCard: function (card) {
     ipc.send(messages.REMOVE_AUTOFILL_CREDIT_CARD, card)
   },
 
+  /**
+   * Open a editing credit card dialog
+   *
+   * @param {object} card - credit card to edit as per doc/state.md's autofillCreditCardDetail
+   */
   editAutofillCreditCard: function (card) {
     ipc.sendToHost(messages.EDIT_AUTOFILL_CREDIT_CARD, card)
   }

js/actions/appActions.js

@@ -369,6 +369,11 @@ const appActions = {
     })
   },
 
+  /**
+   * Add address data
+   * @param {object} detail - the address to add as per doc/state.md's autofillAddressDetail
+   * @param {object} originalDetail - the original address before editing
+   */
   addAutofillAddress: function (detail, originalDetail) {
     AppDispatcher.dispatch({
       actionType: AppConstants.APP_ADD_AUTOFILL_ADDRESS,
@@ -377,13 +382,22 @@ const appActions = {
     })
   },
 
+  /**
+   * Remove address data
+   * @param {object} detail - the address to remove as per doc/state.md's autofillAddressDetail
+   */
   removeAutofillAddress: function (detail) {
     AppDispatcher.dispatch({
       actionType: AppConstants.APP_REMOVE_AUTOFILL_ADDRESS,
       detail
     })
   },
 
+  /**
+   * Add credit card data
+   * @param {object} detail - the credit card to add as per doc/state.md's autofillCreditCardDetail
+   * @param {object} originalDetail - the original credit card before editing
+   */
   addAutofillCreditCard: function (detail, originalDetail) {
     AppDispatcher.dispatch({
       actionType: AppConstants.APP_ADD_AUTOFILL_CREDIT_CARD,
@@ -392,6 +406,10 @@ const appActions = {
     })
   },
 
+  /**
+   * Remove credit card data
+   * @param {object} detail - the credit card to remove as per doc/state.md's autofillCreditCardDetail
+   */
   removeAutofillCreditCard: function (detail) {
     AppDispatcher.dispatch({
       actionType: AppConstants.APP_REMOVE_AUTOFILL_CREDIT_CARD,

js/actions/windowActions.js

@@ -955,6 +955,8 @@ const windowActions = {
 
   /**
    * Sets the manage autofill address popup detail
+   * @param {Object} currentDetail - Properties of the address to change to
+   * @param {Object} originalDetail - Properties of the address to edit
    */
   setAutofillAddressDetail: function (currentDetail, originalDetail) {
     dispatch({
@@ -965,7 +967,9 @@ const windowActions = {
   },
 
   /**
-   * Sets the manage autofill credit cards popup detail
+   * Sets the manage autofill credit card popup detail
+   * @param {Object} currentDetail - Properties of the credit card to change to
+   * @param {Object} originalDetail - Properties of the credit card to edit
    */
   setAutofillCreditCardDetail: function (currentDetail, originalDetail) {
     dispatch({

js/components/frame.js

@@ -599,9 +599,6 @@ class Frame extends ImmutableComponent {
     this.webview.addEventListener('show-autofill-settings', (e) => {
       windowActions.newFrame({ location: 'about:autofill' }, true)
     })
-    this.webview.addEventListener('update-autofill-popup-data-list-values', (e) => {
-      console.log(e)
-    })
     this.webview.addEventListener('show-autofill-popup', (e) => {
       contextMenus.onShowAutofillMenu(e.suggestions, e.rect, this.frame)
     })

js/constants/messages.js

@@ -136,6 +136,7 @@ const messages = {
   CHECK_FLASH_INSTALLED: _,
   ABOUT_COMPONENT_INITIALIZED: _,
   CLEAR_BROWSING_DATA_NOW: _,
+  // Autofill
   ADD_AUTOFILL_ADDRESS: _,
   REMOVE_AUTOFILL_ADDRESS: _,
   EDIT_AUTOFILL_ADDRESS: _,

js/stores/windowStore.js

@@ -707,7 +707,7 @@ const doAction = (action) => {
           originalDetail: action.originalDetail
         })
       }
-      // Since the input values of addresses are bound, we need to notify the controls sync.
+      // Since the input values of address are bound, we need to notify the controls sync.
       windowStore.emitChanges()
       break
     case WindowConstants.WINDOW_SET_AUTOFILL_CREDIT_CARD_DETAIL:
@@ -719,7 +719,7 @@ const doAction = (action) => {
           originalDetail: action.originalDetail
         })
       }
-      // Since the input values of credit cards are bound, we need to notify the controls sync.
+      // Since the input values of credit card are bound, we need to notify the controls sync.
       windowStore.emitChanges()
       break
     case WindowConstants.WINDOW_SET_DOWNLOADS_TOOLBAR_VISIBLE: