Convert multiple boolean params to mask in getExtensionByIdInternal.

chrome/browser/extensions/extension_service.h

Index: chrome/browser/extensions/extension_service.h
diff --git a/chrome/browser/extensions/extension_service.h b/chrome/browser/extensions/extension_service.h
index c326490a1a8cb6f32e3d039fe3a0dcb5e02b4897..e907e8821309a7bd0beb185be8f2c98e3324dc2c 100644
--- a/chrome/browser/extensions/extension_service.h
+++ b/chrome/browser/extensions/extension_service.h
@@ -669,13 +669,18 @@ class ExtensionService
       const extensions::ExtensionSyncData& extension_sync_data,
       syncer::ModelType type);
 
+  enum IncludeFlag {
+    INCLUDE_NONE = 0,
+    INCLUDE_ENABLED = 1 << 0,
+    INCLUDE_DISABLED = 1 << 1,
+    INCLUDE_TERMINATED = 1 << 2
+  };
+
   // Look up an extension by ID, optionally including either or both of enabled
   // and disabled extensions.
   const extensions::Extension* GetExtensionByIdInternal(
       const std::string& id,
-      bool include_enabled,
-      bool include_disabled,
-      bool include_terminated) const;
+      const int include_mask) const;

[Mihai Parparita -not on Chrome, 2012/08/15 00:10:07]

Nit: We usually don't go this const crazy in extensions code
(http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Use_of...
has some discussion), I don't think it's needed here.

 
   // Adds the given extension to the list of terminated extensions if
   // it is not already there and unloads it.