Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 Loading... | |
| 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 Loading... | |
| 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 |
| OLD | NEW |