Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(128)

Side by Side Diff: chrome/browser/prerender/prerender_manager.h

Issue 138583003: Add back the code to PrerenderManager::PendingSwap to handle RenderViewCreated being called for a... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 6 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_ 5 #ifndef CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_
6 #define CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_ 6 #define CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_
7 7
8 #include <list> 8 #include <list>
9 #include <map> 9 #include <map>
10 #include <string> 10 #include <string>
(...skipping 436 matching lines...) Expand 10 before | Expand all | Expand 10 after
447 virtual void ProvisionalChangeToMainFrameUrl( 447 virtual void ProvisionalChangeToMainFrameUrl(
448 const GURL& url, 448 const GURL& url,
449 content::RenderFrameHost* render_frame_host) OVERRIDE; 449 content::RenderFrameHost* render_frame_host) OVERRIDE;
450 virtual void DidCommitProvisionalLoadForFrame( 450 virtual void DidCommitProvisionalLoadForFrame(
451 int64 frame_id, 451 int64 frame_id,
452 const base::string16& frame_unique_name, 452 const base::string16& frame_unique_name,
453 bool is_main_frame, 453 bool is_main_frame,
454 const GURL& validated_url, 454 const GURL& validated_url,
455 content::PageTransition transition_type, 455 content::PageTransition transition_type,
456 content::RenderViewHost* render_view_host) OVERRIDE; 456 content::RenderViewHost* render_view_host) OVERRIDE;
457 virtual void RenderViewCreated(
458 content::RenderViewHost* render_view_host) OVERRIDE;
457 virtual void DidFailProvisionalLoad( 459 virtual void DidFailProvisionalLoad(
458 int64 frame_id, 460 int64 frame_id,
459 const base::string16& frame_unique_name, 461 const base::string16& frame_unique_name,
460 bool is_main_frame, 462 bool is_main_frame,
461 const GURL& validated_url, 463 const GURL& validated_url,
462 int error_code, 464 int error_code,
463 const base::string16& error_description, 465 const base::string16& error_description,
464 content::RenderViewHost* render_view_host) OVERRIDE; 466 content::RenderViewHost* render_view_host) OVERRIDE;
465 virtual void WebContentsDestroyed(content::WebContents* web_contents) 467 virtual void WebContentsDestroyed(content::WebContents* web_contents)
466 OVERRIDE; 468 OVERRIDE;
467 469
468 private: 470 private:
469 void RecordEvent(PrerenderEvent event) const; 471 void RecordEvent(PrerenderEvent event) const;
470 472
471 void OnMergeCompleted(content::SessionStorageNamespace::MergeResult result); 473 void OnMergeCompleted(content::SessionStorageNamespace::MergeResult result);
472 void OnMergeTimeout(); 474 void OnMergeTimeout();
473 475
474 // Prerender parameters. 476 // Prerender parameters.
475 PrerenderManager* manager_; 477 PrerenderManager* manager_;
476 content::WebContents* target_contents_; 478 content::WebContents* target_contents_;
477 PrerenderData* prerender_data_; 479 PrerenderData* prerender_data_;
478 GURL url_; 480 GURL url_;
479 bool should_replace_current_entry_; 481 bool should_replace_current_entry_;
480 482
481 base::TimeTicks start_time_; 483 base::TimeTicks start_time_;
482 PrerenderTracker::ChildRouteIdPair render_frame_route_id_pair_; 484 std::vector<PrerenderTracker::ChildRouteIdPair> main_rfh_ids_;
483 base::OneShotTimer<PendingSwap> merge_timeout_; 485 base::OneShotTimer<PendingSwap> merge_timeout_;
484 bool swap_successful_; 486 bool swap_successful_;
485 487
486 base::WeakPtrFactory<PendingSwap> weak_factory_; 488 base::WeakPtrFactory<PendingSwap> weak_factory_;
487 }; 489 };
488 490
489 void SetPrerenderContentsFactory( 491 void SetPrerenderContentsFactory(
490 PrerenderContents::Factory* prerender_contents_factory); 492 PrerenderContents::Factory* prerender_contents_factory);
491 493
492 // Adds prerenders from the pending Prerenders, called by 494 // Adds prerenders from the pending Prerenders, called by
(...skipping 262 matching lines...) Expand 10 before | Expand all | Expand 10 after
755 757
756 DISALLOW_COPY_AND_ASSIGN(PrerenderManager); 758 DISALLOW_COPY_AND_ASSIGN(PrerenderManager);
757 }; 759 };
758 760
759 PrerenderManager* FindPrerenderManagerUsingRenderProcessId( 761 PrerenderManager* FindPrerenderManagerUsingRenderProcessId(
760 int render_process_id); 762 int render_process_id);
761 763
762 } // namespace prerender 764 } // namespace prerender
763 765
764 #endif // CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_ 766 #endif // CHROME_BROWSER_PRERENDER_PRERENDER_MANAGER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698