Bug (ContactsAPI) – Support for Contacts API

• Component: Platform
• Status: RESOLVED WONTFIX

[Donggwan Kim]

There is no Contacts API implementation in WebKit
This API is a new feature that is published by W3C.
The Device APIs Working Group of W3C has just released a Last Call Working Draft of its Contacts API:
    http://www.w3.org/TR/2011/WD-contacts-api-20110616/
So we need to implement it.

I'd like to implement it and I'm implementing it now.

I'm going to commit patch for Contacts API implementation very soon

Thank you

[Sam Weinig]

Please email the webkit-dev mailing list explain why you think this API should be implemented in WebKit as requested at http://www.webkit.org/coding/adding-features.html.

[Donggwan Kim]

Created attachment 99237 [details]
Add new feature for Contacts API

I've implemented new feature for contacts API.
Because of implementing new feature, file size of patch is larger than other patches.
If you feel any inconvenience for reviewing my patch, please let me know.
I'll split it as your recommendation.

Thank you.

Donggwan

[WebKit Review Bot]

Attachment 99237 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/fast..." exit_code: 1

Tools/DumpRenderTree/LayoutTestController.cpp:2220:  Use 0 instead of NULL.  [readability/null] [5]
Source/WebCore/DerivedSources.cpp:58:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/DerivedSources.cpp:60:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/DerivedSources.cpp:62:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebCore/DerivedSources.cpp:64:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp:102:  Alphabetical sorting problem.  [build/include_order] [4]
Source/WebKit/qt/WebCoreSupport/DumpRenderTreeSupportQt.cpp:897:  Missing space before ( in switch(  [whitespace/parens] [5]
Total errors found: 7 in 83 files


If any of these errors are false positives, please file a bug against check-webkit-style.

[Adam Barth]

Comment on attachment 99237 [details]
Add new feature for Contacts API

This patch is extremely large and there's some question on webkit-dev about whether we want to implement this API in this form.

[Donggwan Kim]

Created attachment 99240 [details]
Fix style check problem

fix style check problem with LayoutTestController.cpp.
remove build and port related files from patch

[Adam Barth]

Comment on attachment 99240 [details]
Fix style check problem

R- for the reasons above.

[Donggwan Kim]

(In reply to comment #6)
> (From update of attachment 99240 [details])
> R- for the reasons above.

do you want for me to split patch to smaller patches?
and what should i do for webkit-dev discussion issue?
and do i wait until that discussion is completed?

[Adam Barth]

I think the main next step is to convince the WebKit project that we want to implement this feature.  The discussion on webkit-dev seemed to indicate that folks had some concerns about some structural design questions.

Generally speaking, the APIs from the DAP working group haven't received as much scrutiny from browser vendors as those from the WebApps or the HTML working groups.  That's not to say that WebKit isn't interested in APIs from DAP (for example, we've implemented FileSystem), but you might find that there's some effort required to get folks on board.