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

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

Issue 1488653002: Fix scroll restoration when exiting fullscreen mode. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Trying to fix test broken on Mac Created 5 years 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_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
7 7
8 #include <set> 8 #include <set>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 651 matching lines...) Expand 10 before | Expand all | Expand 10 after
662 virtual void GetManifest(const GetManifestCallback& callback) = 0; 662 virtual void GetManifest(const GetManifestCallback& callback) = 0;
663 663
664 typedef base::Callback<void(bool)> HasManifestCallback; 664 typedef base::Callback<void(bool)> HasManifestCallback;
665 665
666 // Returns true if the main frame has a <link> to a web manifest, otherwise 666 // Returns true if the main frame has a <link> to a web manifest, otherwise
667 // false. This method does not guarantee that the manifest exists at the 667 // false. This method does not guarantee that the manifest exists at the
668 // specified location or is valid. 668 // specified location or is valid.
669 virtual void HasManifest(const HasManifestCallback& callback) = 0; 669 virtual void HasManifest(const HasManifestCallback& callback) = 0;
670 670
671 // Requests the renderer to exit fullscreen. 671 // Requests the renderer to exit fullscreen.
672 virtual void ExitFullscreen() = 0; 672 // |will_cause_resize| can be false in the case where we exit tab fullscreen
673 // into browser fullscreen and thus won't cause bounds to change.
674 virtual void ExitFullscreen(bool will_cause_resize) = 0;
673 675
674 // Unblocks requests from renderer for a newly created window. This is 676 // Unblocks requests from renderer for a newly created window. This is
675 // used in showCreatedWindow() or sometimes later in cases where 677 // used in showCreatedWindow() or sometimes later in cases where
676 // delegate->ShouldResumeRequestsForCreatedWindow() indicated the requests 678 // delegate->ShouldResumeRequestsForCreatedWindow() indicated the requests
677 // should not yet be resumed. Then the client is responsible for calling this 679 // should not yet be resumed. Then the client is responsible for calling this
678 // as soon as they are ready. 680 // as soon as they are ready.
679 virtual void ResumeLoadingCreatedWebContents() = 0; 681 virtual void ResumeLoadingCreatedWebContents() = 0;
680 682
681 // Temporary to track down http://crbug.com/538612 683 // Temporary to track down http://crbug.com/538612
682 virtual base::debug::StackTrace GetCreationStackTrace() = 0; 684 virtual base::debug::StackTrace GetCreationStackTrace() = 0;
(...skipping 21 matching lines...) Expand all
704 706
705 private: 707 private:
706 // This interface should only be implemented inside content. 708 // This interface should only be implemented inside content.
707 friend class WebContentsImpl; 709 friend class WebContentsImpl;
708 WebContents() {} 710 WebContents() {}
709 }; 711 };
710 712
711 } // namespace content 713 } // namespace content
712 714
713 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 715 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698