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

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

Issue 2932453002: PlzNavigate: Release StreamHandle. (Closed)
Patch Set: Experiment: Use a RenderFrameObserver. Created 3 years, 6 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 745 matching lines...) Expand 10 before | Expand all | Expand 10 after
756 #if defined(OS_MACOSX) 756 #if defined(OS_MACOSX)
757 void OnCopyToFindPboard(); 757 void OnCopyToFindPboard();
758 #endif 758 #endif
759 759
760 // Dispatches the current state of selection on the webpage to the browser if 760 // Dispatches the current state of selection on the webpage to the browser if
761 // it has changed. 761 // it has changed.
762 // TODO(varunjain): delete this method once we figure out how to keep 762 // TODO(varunjain): delete this method once we figure out how to keep
763 // selection handles in sync with the webpage. 763 // selection handles in sync with the webpage.
764 void SyncSelectionIfRequired(); 764 void SyncSelectionIfRequired();
765 765
766 // PlzNavigate: Let the browser know the StreamHandle associated with this
767 // url can be released.
768 void StreamHandleConsumed(const GURL& stream_url);
769
766 protected: 770 protected:
767 explicit RenderFrameImpl(const CreateParams& params); 771 explicit RenderFrameImpl(const CreateParams& params);
768 772
769 private: 773 private:
770 friend class RenderFrameImplTest; 774 friend class RenderFrameImplTest;
771 friend class RenderFrameObserver; 775 friend class RenderFrameObserver;
772 friend class RenderAccessibilityImplTest; 776 friend class RenderAccessibilityImplTest;
773 friend class TestRenderFrame; 777 friend class TestRenderFrame;
774 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuDisplayNoneTest, SelectItem); 778 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuDisplayNoneTest, SelectItem);
775 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuRemoveTest, RemoveOnChange); 779 FRIEND_TEST_ALL_PREFIXES(ExternalPopupMenuRemoveTest, RemoveOnChange);
(...skipping 645 matching lines...) Expand 10 before | Expand all | Expand 10 after
1421 std::vector<media::RoutingTokenCallback> pending_routing_token_callbacks_; 1425 std::vector<media::RoutingTokenCallback> pending_routing_token_callbacks_;
1422 1426
1423 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 1427 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
1424 1428
1425 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 1429 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
1426 }; 1430 };
1427 1431
1428 } // namespace content 1432 } // namespace content
1429 1433
1430 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 1434 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698