Change: kill style.reason

background/update.js

@@ -182,7 +182,6 @@
 
       json.id = style.id;
       json.updateDate = Date.now();
-      // json.reason = 'update';
 
       // keep current state
       delete json.enabled;
@@ -197,7 +196,6 @@
       const newStyle = Object.assign({}, style, json);
       if (styleSectionsEqual(json, style, {checkSource: true})) {
         // update digest even if save === false as there might be just a space added etc.
-        // json.reason = 'update-digest';
         return styleManager.installStyle(newStyle)
           .then(saved => {
             style.originalDigest = saved.originalDigest;

background/usercss-helper.js

@@ -52,18 +52,13 @@
   }
 
   function assignVars(style) {
-    if (style.reason === 'config' && style.id) {
-      return style;
-    }
     return find(style)
       .then(dup => {
         if (dup) {
           style.id = dup.id;
-          if (style.reason !== 'config') {
+          // preserve style.vars during update
-            // preserve style.vars during update
+          return usercss.assignVars(style, dup)
-            return usercss.assignVars(style, dup)
+            .then(() => style);
-              .then(() => style);
-          }
         }
         return style;
       });

edit/edit.js

@@ -321,12 +321,14 @@ function preinit() {
 function onRuntimeMessage(request) {
   switch (request.method) {
     case 'styleUpdated':
-      if (editor.getStyleId() === request.style.id &&
+      if (
-          request.reason !== 'editPreview' &&
+        editor.getStyleId() === request.style.id &&
-          request.reason !== 'editSave' &&
+        !['editPreview', 'editPreviewEnd', 'editSave', 'config']
-          request.reason !== 'config') {
+          .includes(request.reason)
-        Promise.resolve(request.codeIsUpdated === false ?
+      ) {
-          request.style : API.getStyle(request.style.id)
+        Promise.resolve(
+          request.codeIsUpdated === false ?
+            request.style : API.getStyle(request.style.id)
         )
           .then(newStyle => {
             editor.replaceStyle(newStyle, request.codeIsUpdated);

install-usercss/install-usercss.js

@@ -88,7 +88,6 @@
 
       API.installUsercss({
         id: (installed || installedDup).id,
-        reason: 'update',
         sourceCode
       }).then(style => {
         updateMeta(style);

manage/config-dialog.js

@@ -125,7 +125,6 @@ function configDialog(style) {
     }
     style = style.sections ? Object.assign({}, style) : style;
     style.enabled = true;
-    style.reason = 'config';
     style.sourceCode = null;
     style.sections = null;
     const styleVars = style.usercssData.vars;

manage/manage.js

@@ -419,7 +419,6 @@ Object.assign(handleEvent, {
     event.preventDefault();
     const json = entry.updatedCode;
     json.id = entry.styleId;
-    // json.reason = 'update';
     API[json.usercssData ? 'installUsercss' : 'installStyle'](json);
   },
 
@@ -519,10 +518,7 @@ Object.assign(handleEvent, {
 
 
 function handleUpdate(style, {reason, method} = {}) {
-  if (reason === 'editPreview') return;
+  if (reason === 'editPreview' || reason === 'editPreviewEnd') return;
-  // the style was toggled and refreshAllTabs() sent a mini-notification,
-  // but we've already processed 'styleUpdated' sent directly from notifyAllTabs()
-  // if (!style.sections) return;
   let entry;
   let oldEntry = $(ENTRY_ID_PREFIX + style.id);
   if (oldEntry && method === 'styleUpdated') {

popup/popup.js

@@ -52,7 +52,7 @@ function onRuntimeMessage(msg) {
     case 'styleAdded':
     case 'styleUpdated':
     case 'exclusionsUpdated':
-      if (msg.reason === 'editPreview') return;
+      if (msg.reason === 'editPreview' || msg.reason === 'editPreviewEnd') return;
       handleUpdate(msg.style);
       break;
     case 'styleDeleted':