Bug 46668 - Add WebProcessConnection CreatePlugin message
Summary: Add WebProcessConnection CreatePlugin message
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-09-27 15:53 PDT by Anders Carlsson
Modified: 2010-09-27 16:38 PDT (History)
0 users

See Also:


Attachments
Patch (20.41 KB, patch)
2010-09-27 15:57 PDT, Anders Carlsson
aroben: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2010-09-27 15:53:55 PDT
Add WebProcessConnection CreatePlugin message
Comment 1 Anders Carlsson 2010-09-27 15:57:12 PDT
Created attachment 68983 [details]
Patch
Comment 2 Adam Roben (:aroben) 2010-09-27 16:03:13 PDT
Comment on attachment 68983 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=68983&action=review

> WebKit2/PluginProcess/WebProcessConnection.messages.in:26
> +    # Creates a plug-in instance with the given instance ID.

I think // comments will work just fine, if that's what you'd prefer.

> WebKit2/Scripts/webkit2/messages.py:219
> +def headers_for_type(type):
> +    special_cases = {
> +        'WTF::String': '<wtf/text/WTFString.h>',
> +        'WebKit::WebKeyboardEvent': '"WebEvent.h"',
> +        'WebKit::WebMouseEvent': '"WebEvent.h"',
> +        'WebKit::WebWheelEvent': '"WebEvent.h"',
> +        'WebKit::WebTouchEvent': '"WebEvent.h"',
> +    }
> +    if type in special_cases:
> +        return [special_cases[type]]
> +
> +    # We assume that we must include a header for a type iff it has a scope
> +    # resolution operator (::).
> +    split = type.split('::')
> +    if len(split) < 2:
> +        return []
> +    if split[0] == 'WebKit' or split[0] == 'CoreIPC':
> +        return ['"%s.h"' % split[1]]
> +    return ['<%s/%s.h>' % tuple(split)]

Why did you move this?
Comment 3 Anders Carlsson 2010-09-27 16:38:48 PDT
Committed r68445: <http://trac.webkit.org/changeset/68445>