From cc77a227813c7185bfbecf5744e50250bdd1e24c Mon Sep 17 00:00:00 2001 From: "stuartmorgan@google.com" Date: Fri, 31 Jul 2009 20:30:33 +0000 Subject: Replace WebPluginDelegateProxy::BackgroundChanged NOTIMPLEMENTED with a bug comment on Mac This is a big source of console spew, so we should turn it off before enabling plugins by default. BUG=18193 TEST=No "Not implemented reached" spew to console for WebPluginDelegateProxy::BackgroundChanged Review URL: http://codereview.chromium.org/159724 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@22183 0039d316-1c4b-4281-b951-d872f2087c98 --- chrome/renderer/webplugin_delegate_proxy.cc | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'chrome/renderer') diff --git a/chrome/renderer/webplugin_delegate_proxy.cc b/chrome/renderer/webplugin_delegate_proxy.cc index a2ffeb4..f4eddc4 100644 --- a/chrome/renderer/webplugin_delegate_proxy.cc +++ b/chrome/renderer/webplugin_delegate_proxy.cc @@ -602,12 +602,15 @@ bool WebPluginDelegateProxy::BackgroundChanged( return true; } return false; -#else - // Mac implementation note: |rect| is in content-area-relative coordinates, +#elif defined(OS_MACOSX) + // Needs implementation; see http://crbug.com/18193 + // Implementation note: |rect| is in content-area-relative coordinates, // but we may be given a context that is a subset of the content area // with a transform that makes the coordinates work out. We will likely // need to do fixup work like that done in BlitContextToContext when we // implement this. + return true; +#else NOTIMPLEMENTED(); return true; #endif -- cgit v1.1