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

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

Issue 212793004: Adds RenderFrameObserver::DidFinishDocumentLoad (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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
« no previous file with comments | « content/public/renderer/render_frame_observer.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 1738 matching lines...) Expand 10 before | Expand all | Expand 10 after
1749 WebDataSource* ds = frame->dataSource(); 1749 WebDataSource* ds = frame->dataSource();
1750 DocumentState* document_state = DocumentState::FromDataSource(ds); 1750 DocumentState* document_state = DocumentState::FromDataSource(ds);
1751 document_state->set_finish_document_load_time(Time::Now()); 1751 document_state->set_finish_document_load_time(Time::Now());
1752 1752
1753 Send(new FrameHostMsg_DidFinishDocumentLoad(routing_id_)); 1753 Send(new FrameHostMsg_DidFinishDocumentLoad(routing_id_));
1754 1754
1755 // Call back to RenderViewImpl for observers to be notified. 1755 // Call back to RenderViewImpl for observers to be notified.
1756 // TODO(nasko): Remove once we have RenderFrameObserver for this method. 1756 // TODO(nasko): Remove once we have RenderFrameObserver for this method.
1757 render_view_->didFinishDocumentLoad(frame); 1757 render_view_->didFinishDocumentLoad(frame);
1758 1758
1759 FOR_EACH_OBSERVER(RenderFrameObserver, observers_, DidFinishDocumentLoad());
1760
1759 // Check whether we have new encoding name. 1761 // Check whether we have new encoding name.
1760 render_view_->UpdateEncoding(frame, frame->view()->pageEncoding().utf8()); 1762 render_view_->UpdateEncoding(frame, frame->view()->pageEncoding().utf8());
1761 } 1763 }
1762 1764
1763 void RenderFrameImpl::didHandleOnloadEvents(blink::WebFrame* frame) { 1765 void RenderFrameImpl::didHandleOnloadEvents(blink::WebFrame* frame) {
1764 DCHECK(!frame_ || frame_ == frame); 1766 DCHECK(!frame_ || frame_ == frame);
1765 // TODO(nasko): Move implementation here. Needed state: 1767 // TODO(nasko): Move implementation here. Needed state:
1766 // * page_id_ 1768 // * page_id_
1767 render_view_->didHandleOnloadEvents(frame); 1769 render_view_->didHandleOnloadEvents(frame);
1768 } 1770 }
(...skipping 1150 matching lines...) Expand 10 before | Expand all | Expand 10 after
2919 selection_text_offset_ = offset; 2921 selection_text_offset_ = offset;
2920 selection_range_ = range; 2922 selection_range_ = range;
2921 // This IPC is dispatched by RenderWidetHost, so use its routing ID. 2923 // This IPC is dispatched by RenderWidetHost, so use its routing ID.
2922 Send(new ViewHostMsg_SelectionChanged( 2924 Send(new ViewHostMsg_SelectionChanged(
2923 GetRenderWidget()->routing_id(), text, offset, range)); 2925 GetRenderWidget()->routing_id(), text, offset, range));
2924 } 2926 }
2925 GetRenderWidget()->UpdateSelectionBounds(); 2927 GetRenderWidget()->UpdateSelectionBounds();
2926 } 2928 }
2927 2929
2928 } // namespace content 2930 } // namespace content
OLDNEW
« no previous file with comments | « content/public/renderer/render_frame_observer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698