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

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

Issue 10349013: Prevent browser thumbnailer from trying to read from frontbuffer (surface texture) when none exists. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixing unittests and adding one EXPECT clause. Created 8 years, 7 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
« no previous file with comments | « content/browser/renderer_host/test_render_view_host.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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_RENDER_WIDGET_HOST_VIEW_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_
6 #define CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_ 6 #define CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_
7 #pragma once 7 #pragma once
8 8
9 #if defined(TOOLKIT_GTK) 9 #if defined(TOOLKIT_GTK)
10 #include <gdk/gdk.h> 10 #include <gdk/gdk.h>
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
75 // Retrieves the native view used to contain plugins and identify the 75 // Retrieves the native view used to contain plugins and identify the
76 // renderer in IPC messages. 76 // renderer in IPC messages.
77 virtual gfx::NativeView GetNativeView() const = 0; 77 virtual gfx::NativeView GetNativeView() const = 0;
78 virtual gfx::NativeViewId GetNativeViewId() const = 0; 78 virtual gfx::NativeViewId GetNativeViewId() const = 0;
79 virtual gfx::NativeViewAccessible GetNativeViewAccessible() = 0; 79 virtual gfx::NativeViewAccessible GetNativeViewAccessible() = 0;
80 80
81 // Set focus to the associated View component. 81 // Set focus to the associated View component.
82 virtual void Focus() = 0; 82 virtual void Focus() = 0;
83 // Returns true if the View currently has the focus. 83 // Returns true if the View currently has the focus.
84 virtual bool HasFocus() const = 0; 84 virtual bool HasFocus() const = 0;
85 // Returns true is the current display surface is available.
86 virtual bool HasSurface() const = 0;
mmocny 2012/05/03 19:46:08 This name is not nearly descriptive enough, but I
mazda 2012/05/03 20:19:33 This is also confusing because there is similar fu
mazda 2012/05/04 22:20:32 IsSurfaceAvailableForCopy in your latest patch loo
85 87
86 // Shows/hides the view. These must always be called together in pairs. 88 // Shows/hides the view. These must always be called together in pairs.
87 // It is not legal to call Hide() multiple times in a row. 89 // It is not legal to call Hide() multiple times in a row.
88 virtual void Show() = 0; 90 virtual void Show() = 0;
89 virtual void Hide() = 0; 91 virtual void Hide() = 0;
90 92
91 // Whether the view is showing. 93 // Whether the view is showing.
92 virtual bool IsShowing() = 0; 94 virtual bool IsShowing() = 0;
93 95
94 // Retrieve the bounds of the View, in screen coordinates. 96 // Retrieve the bounds of the View, in screen coordinates.
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 // Return value indicates whether the mouse is locked successfully or not. 144 // Return value indicates whether the mouse is locked successfully or not.
143 virtual bool LockMouse() = 0; 145 virtual bool LockMouse() = 0;
144 virtual void UnlockMouse() = 0; 146 virtual void UnlockMouse() = 0;
145 // Returns true if the mouse pointer is currently locked. 147 // Returns true if the mouse pointer is currently locked.
146 virtual bool IsMouseLocked() = 0; 148 virtual bool IsMouseLocked() = 0;
147 }; 149 };
148 150
149 } // namespace content 151 } // namespace content
150 152
151 #endif // CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_ 153 #endif // CONTENT_PUBLIC_BROWSER_RENDER_WIDGET_HOST_VIEW_H_
OLDNEW
« no previous file with comments | « content/browser/renderer_host/test_render_view_host.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698