Make blink::ResourceRequest ctors explicit

third_party/WebKit/Source/core/page/CreateWindow.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 177 matching lines @@
                            : first_frame.GetDocument()->CompleteURL(url_string);
   if (!completed_url.IsEmpty() && !completed_url.IsValid()) {
     UseCounter::Count(active_frame, UseCounter::kWindowOpenWithInvalidURL);
     // Don't expose client code to invalid URLs.
     calling_window.PrintErrorMessage(
         "Unable to open a window with invalid URL '" +
         completed_url.GetString() + "'.\n");
     return nullptr;
   }
 
-  FrameLoadRequest frame_request(calling_window.document(), completed_url,
-                                 frame_name);
+  FrameLoadRequest frame_request(calling_window.document(),
+                                 ResourceRequest(completed_url), frame_name);
   frame_request.SetShouldSetOpener(window_features.noopener ? kNeverSetOpener
                                                             : kMaybeSetOpener);
   frame_request.GetResourceRequest().SetFrameType(
       WebURLRequest::kFrameTypeAuxiliary);
   frame_request.GetResourceRequest().SetRequestorOrigin(
       SecurityOrigin::Create(active_frame->GetDocument()->Url()));
 
   // Normally, FrameLoader would take care of setting the referrer for a
   // navigation that is triggered from javascript. However, creating a window
   // goes through sufficient processing that it eventually enters FrameLoader as
@@ skipping 26 matching lines @@
   // TODO(dcheng): Special case for window.open("about:blank") to ensure it
   // loads synchronously into a new window. This is our historical behavior, and
   // it's consistent with the creation of a new iframe with src="about:blank".
   // Perhaps we could get rid of this if we started reporting the initial empty
   // document's url as about:blank? See crbug.com/471239.
   // TODO(japhet): This special case is also necessary for behavior asserted by
   // some extensions tests.  Using NavigationScheduler::scheduleNavigationChange
   // causes the navigation to be flagged as a client redirect, which is
   // observable via the webNavigation extension api.
   if (created) {
-    FrameLoadRequest request(calling_window.document(), completed_url);
+    FrameLoadRequest request(calling_window.document(),
+                             ResourceRequest(completed_url));
     request.GetResourceRequest().SetHasUserGesture(has_user_gesture);
     new_frame->Navigate(request);
   } else if (!url_string.IsEmpty()) {
     new_frame->Navigate(*calling_window.document(), completed_url, false,
                         has_user_gesture ? UserGestureStatus::kActive
                                          : UserGestureStatus::kNone);
   }
   return new_frame->DomWindow();
 }
 
@@ skipping 34 matching lines @@
   }
 
   // TODO(japhet): Form submissions on RemoteFrames don't work yet.
   FrameLoadRequest new_request(0, request.GetResourceRequest());
   new_request.SetForm(request.Form());
   if (new_frame->IsLocalFrame())
     ToLocalFrame(new_frame)->Loader().Load(new_request);
 }
 
 }  // namespace blink