Summary: | Support for Contacts API | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Donggwan Kim <donggwan.kim> | ||||||
Component: | Platform | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED WONTFIX | ||||||||
Severity: | Normal | CC: | abarth, christian.sejersen, christian.webkit, ddahl, dinu.jacob, donggwan.kim, dwim79, efidler, jarred, kenneth, kihong.kwon, laszlo.gombos, mark.finkle, mike, ravi.kasibhatla, sam, sriram.yadavalli, tobie.langel, webkit.review.bot, wonsuk11.lee | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | All | ||||||||
OS: | All | ||||||||
URL: | http://www.w3.org/TR/contacts-api/ | ||||||||
Bug Depends on: | |||||||||
Bug Blocks: | 78947 | ||||||||
Attachments: |
|
Description
Donggwan Kim
2011-06-22 19:36:01 PDT
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. 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
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.
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.
Created attachment 99240 [details]
Fix style check problem
fix style check problem with LayoutTestController.cpp.
remove build and port related files from patch
Comment on attachment 99240 [details]
Fix style check problem
R- for the reasons above.
(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? 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. |