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

Side by Side Diff: content/public/browser/web_contents_observer.h

Issue 13467038: Browser Plugin: Expose frame name changes to the content API (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Expose RenderViewHost::SetName to the content API Created 7 years, 8 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
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 CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
7 7
8 #include "base/process.h" 8 #include "base/process.h"
9 #include "base/process_util.h" 9 #include "base/process_util.h"
10 #include "content/common/content_export.h" 10 #include "content/common/content_export.h"
(...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after
130 virtual void DidNavigateMainFrame( 130 virtual void DidNavigateMainFrame(
131 const LoadCommittedDetails& details, 131 const LoadCommittedDetails& details,
132 const FrameNavigateParams& params) {} 132 const FrameNavigateParams& params) {}
133 133
134 // And regardless of what frame navigated, this method is invoked after 134 // And regardless of what frame navigated, this method is invoked after
135 // DidCommitProvisionalLoadForFrame was invoked. 135 // DidCommitProvisionalLoadForFrame was invoked.
136 virtual void DidNavigateAnyFrame( 136 virtual void DidNavigateAnyFrame(
137 const LoadCommittedDetails& details, 137 const LoadCommittedDetails& details,
138 const FrameNavigateParams& params) {} 138 const FrameNavigateParams& params) {}
139 139
140 virtual void DidUpdateFrameName(int frame_id,
jam 2013/04/09 16:11:51 nit: all the methods here are documented. please f
141 bool is_top_level,
142 const std::string& name,
143 RenderViewHost* render_view_host) {}
jam 2013/04/09 16:11:51 seems this should match the order of WCImpl
144
140 // This method is invoked once the window.document object was created. 145 // This method is invoked once the window.document object was created.
141 virtual void DocumentAvailableInMainFrame() {} 146 virtual void DocumentAvailableInMainFrame() {}
142 147
143 // This method is invoked when the document in the given frame finished 148 // This method is invoked when the document in the given frame finished
144 // loading. At this point, scripts marked as defer were executed, and 149 // loading. At this point, scripts marked as defer were executed, and
145 // content scripts marked "document_end" get injected into the frame. 150 // content scripts marked "document_end" get injected into the frame.
146 virtual void DocumentLoadedInFrame(int64 frame_id, 151 virtual void DocumentLoadedInFrame(int64 frame_id,
147 RenderViewHost* render_view_host) {} 152 RenderViewHost* render_view_host) {}
148 153
149 // This method is invoked when the navigation is done, i.e. the spinner of 154 // This method is invoked when the navigation is done, i.e. the spinner of
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
284 void WebContentsImplDestroyed(); 289 void WebContentsImplDestroyed();
285 290
286 WebContentsImpl* web_contents_; 291 WebContentsImpl* web_contents_;
287 292
288 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver); 293 DISALLOW_COPY_AND_ASSIGN(WebContentsObserver);
289 }; 294 };
290 295
291 } // namespace content 296 } // namespace content
292 297
293 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_ 298 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_OBSERVER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698