Use bounds instead of size for prerender requests

chrome/browser/prerender/prerender_manager.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_
 #define CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 32 matching lines @@
 
 namespace chrome_browser_net {
 enum class NetworkPredictionStatus;
 }
 
 namespace content {
 class WebContents;
 }
 
 namespace gfx {
 class Size;

[mmenke, 2016/08/19 15:22:29]

Need to add Rect here.

 }
 
 namespace offline_pages {
 class PrerenderAdapterTest;
 }
 
 namespace prerender {
 
 class PrerenderHandle;
 class PrerenderHistory;
@@ skipping 60 matching lines @@
   // PrerenderHandle or NULL.
   std::unique_ptr<PrerenderHandle> AddPrerenderFromOmnibox(
       const GURL& url,
       content::SessionStorageNamespace* session_storage_namespace,
       const gfx::Size& size);
 
   std::unique_ptr<PrerenderHandle> AddPrerenderFromExternalRequest(
       const GURL& url,
       const content::Referrer& referrer,
       content::SessionStorageNamespace* session_storage_namespace,
-      const gfx::Size& size);
+      const gfx::Rect& bounds);
 
   // Adds a prerender from an external request that will prerender even on
   // cellular networks as long as the user setting for prerendering is ON.
   std::unique_ptr<PrerenderHandle> AddPrerenderOnCellularFromExternalRequest(
       const GURL& url,
       const content::Referrer& referrer,
       content::SessionStorageNamespace* session_storage_namespace,
-      const gfx::Size& size);
+      const gfx::Rect& bounds);
 
   // Adds a prerender for Instant Search |url| if valid. The
   // |session_storage_namespace| matches the namespace of the active tab at the
   // time the prerender is generated. Returns a PrerenderHandle or NULL.
   std::unique_ptr<PrerenderHandle> AddPrerenderForInstant(
       const GURL& url,
       content::SessionStorageNamespace* session_storage_namespace,
       const gfx::Size& size);
 
   // Adds a prerender for the background loader. Returns a PrerenderHandle if
@@ skipping 253 matching lines @@
   // Returns whether prerendering is currently enabled or the reason why it is
   // disabled.
   chrome_browser_net::NetworkPredictionStatus GetPredictionStatus() const;
 
   // Returns whether prerendering is currently enabled or the reason why it is
   // disabled after taking into account the origin of the request.
   chrome_browser_net::NetworkPredictionStatus GetPredictionStatusForOrigin(
       Origin origin) const;
 
   // Adds a prerender for |url| from |referrer|. The |origin| specifies how the
-  // prerender was added. If |size| is empty, then
+  // prerender was added. If |bounds| is empty, then
   // PrerenderContents::StartPrerendering will instead use a default from
   // PrerenderConfig. Returns a PrerenderHandle or NULL.
   std::unique_ptr<PrerenderHandle> AddPrerender(
       Origin origin,
       const GURL& url,
       const content::Referrer& referrer,
-      const gfx::Size& size,
+      const gfx::Rect& bounds,
       content::SessionStorageNamespace* session_storage_namespace);
 
   void StartSchedulingPeriodicCleanups();
   void StopSchedulingPeriodicCleanups();
 
   void EvictOldestPrerendersIfNecessary();
 
   // Deletes stale and cancelled prerendered PrerenderContents, as well as
   // WebContents that have been replaced by prerendered WebContents.
   // Also identifies and kills PrerenderContents that use too much
@@ skipping 126 matching lines @@
   // Set of process hosts being prerendered.
   using PrerenderProcessSet = std::set<content::RenderProcessHost*>;
   PrerenderProcessSet prerender_process_hosts_;
 
   DISALLOW_COPY_AND_ASSIGN(PrerenderManager);
 };
 
 }  // namespace prerender
 
 #endif  // CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_