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

Side by Side Diff: blimp/engine/session/page_load_tracker.cc

Issue 1986883002: blimp: Update page load status update indicator to use first paint. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test. Created 4 years, 7 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
« no previous file with comments | « blimp/engine/session/page_load_tracker.h ('k') | cc/trees/layer_tree_host.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "blimp/engine/session/page_load_tracker.h"
6
7 #include "content/public/browser/render_widget_host_view.h"
8
9 namespace blimp {
10 namespace engine {
11
12 namespace {
13
14 content::RenderWidgetHost* GetRenderWidgetHostIfMainFrame(
15 content::RenderFrameHost* render_frame_host) {
16 if (render_frame_host->GetParent() != nullptr)
17 return nullptr;
18
19 return render_frame_host->GetView()->GetRenderWidgetHost();
20 }
21
22 } // namespace
23
24 PageLoadTracker::PageLoadTracker(content::WebContents* web_contents,
25 PageLoadTrackerClient* client)
26 : client_(client) {
27 Observe(web_contents);
28 }
29
30 PageLoadTracker::~PageLoadTracker() {}
31
32 void PageLoadTracker::DidStartProvisionalLoadForFrame(
33 content::RenderFrameHost* render_frame_host,
34 const GURL& validated_url,
35 bool is_error_page,
36 bool is_iframe_srcdoc) {
37 content::RenderWidgetHost* render_widget_host =
38 GetRenderWidgetHostIfMainFrame(render_frame_host);
39 if (!render_widget_host)
40 return;
41
42 render_widget_load_status_[render_widget_host] = LoadStatus();
43
44 // Notify the client that a navigation was initiated.
45 client_->SendPageLoadStatusUpdate(PageLoadStatus::LOADING);
46 }
47
48 void PageLoadTracker::DidFinishLoad(content::RenderFrameHost* render_frame_host,
49 const GURL& validated_url) {
50 content::RenderWidgetHost* render_widget_host =
51 GetRenderWidgetHostIfMainFrame(render_frame_host);
52 if (!render_widget_host)
53 return;
54
55 RenderWidgetLoadStatusMap::iterator it =
56 render_widget_load_status_.find(render_widget_host);
57 DCHECK(it != render_widget_load_status_.end());
58
59 it->second.page_loaded = true;
60 if (it->second.Loaded()) {
61 client_->SendPageLoadStatusUpdate(PageLoadStatus::LOADED);
62 render_widget_load_status_.erase(it);
63 }
64 }
65
66 void PageLoadTracker::DidFailLoad(content::RenderFrameHost* render_frame_host,
67 const GURL& validated_url,
68 int error_code,
69 const base::string16& error_description,
70 bool was_ignored_by_handler) {
71 content::RenderWidgetHost* render_widget_host =
72 GetRenderWidgetHostIfMainFrame(render_frame_host);
73 if (!render_widget_host)
74 return;
75
76 RenderWidgetLoadStatusMap::iterator it =
77 render_widget_load_status_.find(render_widget_host);
78 DCHECK(it != render_widget_load_status_.end());
79
80 // If the navigation failed, the client should dismiss the load indicator.
81 client_->SendPageLoadStatusUpdate(PageLoadStatus::LOADED);
82 render_widget_load_status_.erase(it);
83 }
84
85 void PageLoadTracker::DidFirstPaintAfterLoad(
86 content::RenderWidgetHost* render_widget_host) {
87 RenderWidgetLoadStatusMap::iterator it =
88 render_widget_load_status_.find(render_widget_host);
89 DCHECK(it != render_widget_load_status_.end());
90
91 it->second.did_first_paint = true;
92 if (it->second.Loaded()) {
93 client_->SendPageLoadStatusUpdate(PageLoadStatus::LOADED);
94 render_widget_load_status_.erase(it);
95 }
96 }
97
98 bool PageLoadTracker::LoadStatus::Loaded() const {
99 return page_loaded && did_first_paint;
100 }
101
102 } // namespace engine
103 } // namespace blimp
OLDNEW
« no previous file with comments | « blimp/engine/session/page_load_tracker.h ('k') | cc/trees/layer_tree_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698