Cleanups to the refactored extension activity log code

chrome/browser/extensions/activity_log/activity_actions.h

Index: chrome/browser/extensions/activity_log/activity_actions.h
diff --git a/chrome/browser/extensions/activity_log/activity_actions.h b/chrome/browser/extensions/activity_log/activity_actions.h
index c867704b9f29d7d18994283f9194a01d35ef77ce..d68a2d07987097b8e418237bc0619a8d5c15b99d 100644
--- a/chrome/browser/extensions/activity_log/activity_actions.h
+++ b/chrome/browser/extensions/activity_log/activity_actions.h
@@ -33,7 +33,10 @@ class Action : public base::RefCountedThreadSafe<Action> {
     ACTION_CONTENT_SCRIPT = 3,
     ACTION_DOM_ACCESS = 4,
     ACTION_DOM_EVENT = 5,
-    ACTION_DOM_XHR = 6,
+    // Type 6 is reserved: It was for ACTION_DOM_XHR but that was never
+    // actually used.  In the future any injected XHRs, once tracked, will be
+    // classified as ACTION_DOM_ACCESS--no separate type is needed since
+    // api_name is sufficient for distinguishing.

[felt, 2013/08/02 00:41:53]

I am fine with just making ACTION_WEB_REQUEST=6 at this point. We have no users
-- I don't think we've even caught up to this schema internally -- so we have no
real backward compatibility concerns.

[mvrable, 2013/08/02 03:33:21]

Done.

     ACTION_WEB_REQUEST = 7,
   };