Clarify wording around blocking 3rd party cookies and add lab to block reading of 3rd party cookies

net/base/static_cookie_policy.h

Index: net/base/static_cookie_policy.h
diff --git a/net/base/static_cookie_policy.h b/net/base/static_cookie_policy.h
index 46331617f42363f25fd33677976aa0172d06939a..39484761d3d81a82b05d7871598836ef6cf2f740 100644
--- a/net/base/static_cookie_policy.h
+++ b/net/base/static_cookie_policy.h
@@ -23,10 +23,17 @@ namespace net {
 //
 class StaticCookiePolicy : public CookiePolicy {
  public:
+  // Do not change the order of these types as they are persisted in
+  // preferences.
   enum Type {
-    ALLOW_ALL_COOKIES = 0,      // Do not perform any cookie blocking.
-    BLOCK_THIRD_PARTY_COOKIES,  // Prevent third-party cookies from being set.
-    BLOCK_ALL_COOKIES           // Disable cookies.
+    // Do not perform any cookie blocking.
+    ALLOW_ALL_COOKIES = 0,
+    // Prevent third-party cookies from being set.
+    BLOCK_SETTING_THIRD_PARTY_COOKIES,
+    // Disable cookies.
+    BLOCK_ALL_COOKIES,
+    // Prevent third-party cookies from being set or read.
+    BLOCK_ALL_THIRD_PARTY_COOKIES

[Randy Smith (Not in Mondays), 2011/01/11 18:38:42]

This section seems a bit awkward to me; the combination of the ordering (which I
understand is enforced for versioning purposes) and the block all cookies
comment being very short makes this a bit confusing.  Maybe change "Disable
cookies." to "Disable all cookies (third party or not) from being set or read."?

   };
 
   StaticCookiePolicy()