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

Side by Side Diff: content/browser/frame_host/render_widget_host_view_child_frame.cc

Issue 1113573002: Maintain minimal error response. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove READBACK_NOT_SUPPORTED Created 5 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
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 #include "content/browser/frame_host/render_widget_host_view_child_frame.h" 5 #include "content/browser/frame_host/render_widget_host_view_child_frame.h"
6 6
7 #include "content/browser/accessibility/browser_accessibility_manager.h" 7 #include "content/browser/accessibility/browser_accessibility_manager.h"
8 #include "content/browser/frame_host/cross_process_frame_connector.h" 8 #include "content/browser/frame_host/cross_process_frame_connector.h"
9 #include "content/browser/renderer_host/render_widget_host_impl.h" 9 #include "content/browser/renderer_host/render_widget_host_impl.h"
10 #include "content/common/gpu/gpu_messages.h" 10 #include "content/common/gpu/gpu_messages.h"
(...skipping 244 matching lines...) Expand 10 before | Expand all | Expand 10 after
255 bool RenderWidgetHostViewChildFrame::PostProcessEventForPluginIme( 255 bool RenderWidgetHostViewChildFrame::PostProcessEventForPluginIme(
256 const NativeWebKeyboardEvent& event) { 256 const NativeWebKeyboardEvent& event) {
257 return false; 257 return false;
258 } 258 }
259 #endif // defined(OS_MACOSX) 259 #endif // defined(OS_MACOSX)
260 260
261 void RenderWidgetHostViewChildFrame::CopyFromCompositingSurface( 261 void RenderWidgetHostViewChildFrame::CopyFromCompositingSurface(
262 const gfx::Rect& src_subrect, 262 const gfx::Rect& src_subrect,
263 const gfx::Size& /* dst_size */, 263 const gfx::Size& /* dst_size */,
264 ReadbackRequestCallback& callback, 264 ReadbackRequestCallback& callback,
265 const SkColorType color_type) { 265 const SkColorType color_type) {
no sievers 2015/05/06 23:08:03 Should we rename this to |preferred_color_type| th
sivag 2015/05/07 14:14:20 Done.
266 callback.Run(SkBitmap(), READBACK_NOT_SUPPORTED); 266 callback.Run(SkBitmap(), READBACK_FAILED);
267 } 267 }
268 268
269 void RenderWidgetHostViewChildFrame::CopyFromCompositingSurfaceToVideoFrame( 269 void RenderWidgetHostViewChildFrame::CopyFromCompositingSurfaceToVideoFrame(
270 const gfx::Rect& src_subrect, 270 const gfx::Rect& src_subrect,
271 const scoped_refptr<media::VideoFrame>& target, 271 const scoped_refptr<media::VideoFrame>& target,
272 const base::Callback<void(bool)>& callback) { 272 const base::Callback<void(bool)>& callback) {
273 NOTIMPLEMENTED(); 273 NOTIMPLEMENTED();
274 callback.Run(false); 274 callback.Run(false);
275 } 275 }
276 276
(...skipping 26 matching lines...) Expand all
303 } 303 }
304 304
305 BrowserAccessibilityManager* 305 BrowserAccessibilityManager*
306 RenderWidgetHostViewChildFrame::CreateBrowserAccessibilityManager( 306 RenderWidgetHostViewChildFrame::CreateBrowserAccessibilityManager(
307 BrowserAccessibilityDelegate* delegate) { 307 BrowserAccessibilityDelegate* delegate) {
308 return BrowserAccessibilityManager::Create( 308 return BrowserAccessibilityManager::Create(
309 BrowserAccessibilityManager::GetEmptyDocument(), delegate); 309 BrowserAccessibilityManager::GetEmptyDocument(), delegate);
310 } 310 }
311 311
312 } // namespace content 312 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698