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

Side by Side Diff: content/renderer/render_frame_impl.cc

Issue 263973003: Move LoadProgressTracker to the browser process. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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 | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 #include "content/renderer/render_frame_impl.h" 5 #include "content/renderer/render_frame_impl.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 9
10 #include "base/auto_reset.h" 10 #include "base/auto_reset.h"
(...skipping 2899 matching lines...) Expand 10 before | Expand all | Expand 10 after
2910 void RenderFrameImpl::didStopLoading() { 2910 void RenderFrameImpl::didStopLoading() {
2911 if (!render_view_->is_loading()) 2911 if (!render_view_->is_loading())
2912 return; 2912 return;
2913 render_view_->FrameDidStopLoading(frame_); 2913 render_view_->FrameDidStopLoading(frame_);
2914 2914
2915 // NOTE: For now we're doing the safest thing, and sending out notification 2915 // NOTE: For now we're doing the safest thing, and sending out notification
2916 // when done loading. This currently isn't an issue as the favicon is only 2916 // when done loading. This currently isn't an issue as the favicon is only
2917 // displayed when done loading. Ideally we would send notification when 2917 // displayed when done loading. Ideally we would send notification when
2918 // finished parsing the head, but webkit doesn't support that yet. 2918 // finished parsing the head, but webkit doesn't support that yet.
2919 // The feed discovery code would also benefit from access to the head. 2919 // The feed discovery code would also benefit from access to the head.
2920 if (!render_view_->is_loading()) 2920 if (!render_view_->is_loading())
nasko 2014/05/02 22:48:23 Shouldn't this counting move to the browser proces
Avi (use Gerrit) 2014/05/05 15:15:02 Probably. Hmmm....
2921 Send(new FrameHostMsg_DidStopLoading(routing_id_)); 2921 Send(new FrameHostMsg_DidStopLoading(routing_id_));
2922 } 2922 }
2923 2923
2924 void RenderFrameImpl::didChangeLoadProgress(double load_progress) { 2924 void RenderFrameImpl::didChangeLoadProgress(double load_progress) {
2925 render_view_->FrameDidChangeLoadProgress(frame_, load_progress); 2925 Send(new FrameHostMsg_DidChangeLoadProgress(routing_id_, load_progress));
2926 } 2926 }
2927 2927
2928 WebNavigationPolicy RenderFrameImpl::DecidePolicyForNavigation( 2928 WebNavigationPolicy RenderFrameImpl::DecidePolicyForNavigation(
2929 RenderFrame* render_frame, 2929 RenderFrame* render_frame,
2930 WebFrame* frame, 2930 WebFrame* frame,
2931 WebDataSource::ExtraData* extraData, 2931 WebDataSource::ExtraData* extraData,
2932 const WebURLRequest& request, 2932 const WebURLRequest& request,
2933 WebNavigationType type, 2933 WebNavigationType type,
2934 WebNavigationPolicy default_policy, 2934 WebNavigationPolicy default_policy,
2935 bool is_redirect) { 2935 bool is_redirect) {
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
3237 selection_text_offset_ = offset; 3237 selection_text_offset_ = offset;
3238 selection_range_ = range; 3238 selection_range_ = range;
3239 // This IPC is dispatched by RenderWidetHost, so use its routing ID. 3239 // This IPC is dispatched by RenderWidetHost, so use its routing ID.
3240 Send(new ViewHostMsg_SelectionChanged( 3240 Send(new ViewHostMsg_SelectionChanged(
3241 GetRenderWidget()->routing_id(), text, offset, range)); 3241 GetRenderWidget()->routing_id(), text, offset, range));
3242 } 3242 }
3243 GetRenderWidget()->UpdateSelectionBounds(); 3243 GetRenderWidget()->UpdateSelectionBounds();
3244 } 3244 }
3245 3245
3246 } // namespace content 3246 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698