Use an explicit boolean has_new_first_party_for_cookies instead...

webkit/glue/resource_loader_bridge.h

 // Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 //
 // The intent of this file is to provide a type-neutral abstraction between
 // Chrome and WebKit for resource loading. This pure-virtual interface is
 // implemented by the embedder, which also provides a factory method Create
 // to instantiate this object.
 //
 // One of these objects will be created by WebKit for each request. WebKit
@@ skipping 92 matching lines @@
    public:
     virtual ~Peer() {}
 
     // Called as upload progress is made.
     // note: only for requests with LOAD_ENABLE_UPLOAD_PROGRESS set
     virtual void OnUploadProgress(uint64 position, uint64 size) = 0;
 
     // Called when a redirect occurs.  The implementation may return false to
     // suppress the redirect.  The given ResponseInfo provides complete
     // information about the redirect, and new_url is the URL that will be
-    // loaded if this method returns true.  If this method returns true, it
-    // stores in *new_first_party_for_cookies the new URL that should be
-    // consulted for the third-party cookie blocking policy.  If the cookie
-    // policy URL doesn't need changing, it stores an empty, invalid URL in
-    // *new_first_party_for_cookies.
+    // loaded if this method returns true.  If this method returns true, the
+    // output parameter *has_new_first_party_for_cookies indicates whether the
+    // output parameter *new_first_party_for_cookies contains the new URL that
+    // should be consulted for the third-party cookie blocking policy.
     virtual bool OnReceivedRedirect(const GURL& new_url,
                                     const ResponseInfo& info,
+                                    bool* has_new_first_party_for_cookies,
                                     GURL* new_first_party_for_cookies) = 0;
 
     // Called when response headers are available (after all redirects have
     // been followed).  |content_filtered| is set to true if the contents is
     // altered or replaced (usually for security reasons when the resource is
     // deemed unsafe).
     virtual void OnReceivedResponse(const ResponseInfo& info,
                                     bool content_filtered) = 0;
 
     // Called when a chunk of response data is available. This method may
@@ skipping 21 matching lines @@
   // must be encoded.
   //
   // For HTTP(S) POST requests, the AppendDataToUpload and AppendFileToUpload
   // methods may be called to construct the body of the request.
   //
   // For HTTP(S) requests, the headers parameter can be a \r\n-delimited and
   // \r\n-terminated list of MIME headers.  They should be ASCII-encoded using
   // the standard MIME header encoding rules.  The headers parameter can also
   // be null if no extra request headers need to be set.
   //
-  // policy_url is the URL of the document in the top-level window, which may be
-  // checked by the third-party cookie blocking policy.
+  // first_party_for_cookies is the URL of the document in the top-level
+  // window, which may be checked by the third-party cookie blocking policy.
   //
   // load_flags is composed of the values defined in url_request_load_flags.h
   //
   // request_type indicates if the current request is the main frame load, a
   // sub-frame load, or a sub objects load.
   //
   // appcache_host_id identifies that appcache host this request is
   // associated with.
   //
   // routing_id passed to this function allows it to be associated with a
   // frame's network context.
   static ResourceLoaderBridge* Create(const std::string& method,
                                       const GURL& url,
-                                      const GURL& policy_url,
+                                      const GURL& first_party_for_cookies,
                                       const GURL& referrer,
                                       const std::string& frame_origin,
                                       const std::string& main_frame_origin,
                                       const std::string& headers,
                                       int load_flags,
                                       int requestor_pid,
                                       ResourceType::Type request_type,
                                       int appcache_host_id,
                                       int routing_id);
 
@@ skipping 44 matching lines @@
   // construction must go through Create()
   ResourceLoaderBridge();
 
  private:
   DISALLOW_EVIL_CONSTRUCTORS(ResourceLoaderBridge);
 };
 
 }  // namespace webkit_glue
 
 #endif  // WEBKIT_GLUE_RESOURCE_LOADER_BRIDGE_H_