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

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

Issue 2154233003: Rewrite YouTube Flash embeds (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 4 years, 5 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 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 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
228 228
229 // Looks up and returns the WebFrame corresponding to a given opener frame 229 // Looks up and returns the WebFrame corresponding to a given opener frame
230 // routing ID. Also stores the opener's RenderView routing ID into 230 // routing ID. Also stores the opener's RenderView routing ID into
231 // |opener_view_routing_id|. 231 // |opener_view_routing_id|.
232 // 232 //
233 // TODO(alexmos): remove RenderViewImpl's dependency on 233 // TODO(alexmos): remove RenderViewImpl's dependency on
234 // opener_view_routing_id. 234 // opener_view_routing_id.
235 static blink::WebFrame* ResolveOpener(int opener_frame_routing_id, 235 static blink::WebFrame* ResolveOpener(int opener_frame_routing_id,
236 int* opener_view_routing_id); 236 int* opener_view_routing_id);
237 237
238 // Overwrites the given URL to use an HTML5 video player if possible.
mlamouri (slow - plz ping) 2016/07/21 15:55:08 s/video player/embed/
kdsilva 2016/07/22 13:55:58 Done.
239 blink::WebString overrideFlashEmbedWithHTML(
240 const blink::WebString& url) override;
241
238 ~RenderFrameImpl() override; 242 ~RenderFrameImpl() override;
239 243
240 // Called by RenderWidget when meaningful layout has happened. 244 // Called by RenderWidget when meaningful layout has happened.
241 // See RenderFrameObserver::DidMeaningfulLayout declaration for details. 245 // See RenderFrameObserver::DidMeaningfulLayout declaration for details.
242 void DidMeaningfulLayout(blink::WebMeaningfulLayout layout_type); 246 void DidMeaningfulLayout(blink::WebMeaningfulLayout layout_type);
243 247
244 // Out-of-process child frames receive a signal from RenderWidgetCompositor 248 // Out-of-process child frames receive a signal from RenderWidgetCompositor
245 // when a compositor frame has committed. 249 // when a compositor frame has committed.
246 void DidCommitCompositorFrame(); 250 void DidCommitCompositorFrame();
247 251
(...skipping 1053 matching lines...) Expand 10 before | Expand all | Expand 10 after
1301 mojom::FrameHostPtr frame_host_; 1305 mojom::FrameHostPtr frame_host_;
1302 1306
1303 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 1307 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
1304 1308
1305 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 1309 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
1306 }; 1310 };
1307 1311
1308 } // namespace content 1312 } // namespace content
1309 1313
1310 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 1314 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698