Fixes component extension commands assignment on reload.

extensions/browser/uninstall_reason.h

Index: extensions/browser/uninstall_reason.h
diff --git a/extensions/browser/uninstall_reason.h b/extensions/browser/uninstall_reason.h
index 191b6a48db79bd14021b31e1215b3e30eb84c543..f94b88042b2a0df1244dbadd0f9ad0fb6556b9bf 100644
--- a/extensions/browser/uninstall_reason.h
+++ b/extensions/browser/uninstall_reason.h
@@ -22,7 +22,10 @@ enum UninstallReason {
   UNINSTALL_REASON_ORPHANED_EXTERNAL_EXTENSION,
   UNINSTALL_REASON_ORPHANED_SHARED_MODULE,
   UNINSTALL_REASON_INTERNAL_MANAGEMENT,  // Internal extensions (see usages)
-  UNINSTALL_REASON_REINSTALL
+  UNINSTALL_REASON_REINSTALL,
+  // Component extensions will only ever install once, but can notify uninstall
+  // repeatedly. crbug.com/458612.

[Finnur, 2015/02/18 11:51:31]

This comment seems out of place. Moving it to where you early-return would
probably help.

Also swap out:
"repeatedly. "
... for 
"repeatedly, see:http://"

[David Tseng, 2015/02/18 18:24:41]

Done.

+  UNINSTALL_REASON_COMPONENT

[Finnur, 2015/02/18 11:51:31]

nit: Add trailing comma.

[David Tseng, 2015/02/18 18:24:41]

Is this convention? This wasn't done before, so leaving as is.

[Finnur, 2015/02/20 10:31:49]

Yes, it is convention to add a trailing comma to prevent churn when checking in
new enums (don't have to modify the line before the one you add, just to add a
trailing comma).

[David Tseng, 2015/02/20 18:12:45]

Makes sense. Done.

 };
 
 }  // namespace extensions