Bug 40987 - [Chromium] Add a WebPlugin implementation using a WebView to draw.
Summary: [Chromium] Add a WebPlugin implementation using a WebView to draw.
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: All OS X 10.5
: P2 Normal
Assignee: Bernhard Bauer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-06-22 08:33 PDT by Bernhard Bauer
Modified: 2013-04-09 16:09 PDT (History)
1 user (show)

See Also:


Attachments
Adds WebDOMUIPlugin, WebDOMUIPluginClient and WebDOMUIPluginImpl. (18.23 KB, patch)
2010-06-22 09:16 PDT, Bernhard Bauer
fishd: review-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bernhard Bauer 2010-06-22 08:33:08 PDT
Click-to-load for blocked plugins is going to use this pseudo-plugin.
Comment 1 Bernhard Bauer 2010-06-22 09:16:52 PDT
Created attachment 59379 [details]
Adds WebDOMUIPlugin, WebDOMUIPluginClient and WebDOMUIPluginImpl.
Comment 2 Darin Fisher (:fishd, Google) 2010-06-22 10:24:06 PDT
Comment on attachment 59379 [details]
Adds WebDOMUIPlugin, WebDOMUIPluginClient and WebDOMUIPluginImpl.

I think this code (minus the changes to existing WebKit API interfaces) belongs
in the chromium repository.  The concept of DOM UI is a chromium thing.

Am I missing a reason why this is best to have as part of the WebKit API?
Comment 3 Bernhard Bauer 2010-06-22 10:50:26 PDT
(In reply to comment #2)
> (From update of attachment 59379 [details])
> I think this code (minus the changes to existing WebKit API interfaces) belongs
> in the chromium repository.  The concept of DOM UI is a chromium thing.
> 
> Am I missing a reason why this is best to have as part of the WebKit API?

The drawing code (getting a GraphicsContext from a WebCanvas and doing the necessary transformations on it) depends on WebKit code that's not publicly exposed. I tried keeping the changes to the public interface as small as possible though.

I'm not really happy about the name WebDOMUIPlugin either, do you think something like WebViewPlugin would be better?
Comment 4 Darin Fisher (:fishd, Google) 2010-06-22 12:52:47 PDT
(In reply to comment #3)
> The drawing code (getting a GraphicsContext from a WebCanvas and doing the 
> necessary transformations on it) depends on WebKit code that's not publicly 
> exposed. I tried keeping the changes to the public interface as small as 
> possible though.

I see.  I would imagine that it is possible to do the equivalent transforms
using Skia/CG directly.


> I'm not really happy about the name WebDOMUIPlugin either, do you think something like WebViewPlugin would be better?

Yeah, WebViewPlugin would probably be better.
Comment 5 Stephen Chenney 2013-04-09 16:09:51 PDT
LayoutTest failures for Chromium are being marked WontFix. The Bug is still accessible and referenced from TestExpectations.