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

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

Issue 23841002: Create a new RenderFrameHost per child frame when --site-per-process is enabled. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: custom deleters rock. Created 7 years, 3 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 (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_RENDERER_RENDER_VIEW_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_VIEW_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_VIEW_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_VIEW_IMPL_H_
7 7
8 #include <deque> 8 #include <deque>
9 #include <map> 9 #include <map>
10 #include <set> 10 #include <set>
(...skipping 1325 matching lines...) Expand 10 before | Expand all | Expand 10 after
1336 bool cached_is_main_frame_pinned_to_right_; 1336 bool cached_is_main_frame_pinned_to_right_;
1337 1337
1338 // These store the "has scrollbars" state last sent to the browser. 1338 // These store the "has scrollbars" state last sent to the browser.
1339 bool cached_has_main_frame_horizontal_scrollbar_; 1339 bool cached_has_main_frame_horizontal_scrollbar_;
1340 bool cached_has_main_frame_vertical_scrollbar_; 1340 bool cached_has_main_frame_vertical_scrollbar_;
1341 1341
1342 // Helper objects ------------------------------------------------------------ 1342 // Helper objects ------------------------------------------------------------
1343 1343
1344 scoped_ptr<RenderFrameImpl> main_render_frame_; 1344 scoped_ptr<RenderFrameImpl> main_render_frame_;
1345 1345
1346 template <typename T> struct Closer {
Charlie Reis 2013/09/24 18:25:58 I'm having trouble following what this does and wh
awong 2013/09/26 21:25:24 This has been removed. It was incorrect. What it
1347 inline void operator()(T* ptr) const {
1348 if (ptr)
1349 ptr->close();
1350 }
1351 };
1352 scoped_ptr<WebKit::WebFrame, Closer<WebKit::WebFrame> > main_frame_;
1353
1346 RendererWebCookieJarImpl cookie_jar_; 1354 RendererWebCookieJarImpl cookie_jar_;
1347 1355
1348 // The next group of objects all implement RenderViewObserver, so are deleted 1356 // The next group of objects all implement RenderViewObserver, so are deleted
1349 // along with the RenderView automatically. This is why we just store 1357 // along with the RenderView automatically. This is why we just store
1350 // weak references. 1358 // weak references.
1351 1359
1352 // Holds a reference to the service which provides desktop notifications. 1360 // Holds a reference to the service which provides desktop notifications.
1353 NotificationProvider* notification_provider_; 1361 NotificationProvider* notification_provider_;
1354 1362
1355 // The geolocation dispatcher attached to this view, lazily initialized. 1363 // The geolocation dispatcher attached to this view, lazily initialized.
(...skipping 173 matching lines...) Expand 10 before | Expand all | Expand 10 after
1529 // use the Observer interface to filter IPC messages and receive frame change 1537 // use the Observer interface to filter IPC messages and receive frame change
1530 // notifications. 1538 // notifications.
1531 // --------------------------------------------------------------------------- 1539 // ---------------------------------------------------------------------------
1532 1540
1533 DISALLOW_COPY_AND_ASSIGN(RenderViewImpl); 1541 DISALLOW_COPY_AND_ASSIGN(RenderViewImpl);
1534 }; 1542 };
1535 1543
1536 } // namespace content 1544 } // namespace content
1537 1545
1538 #endif // CONTENT_RENDERER_RENDER_VIEW_IMPL_H_ 1546 #endif // CONTENT_RENDERER_RENDER_VIEW_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698