Summary: | [EFL][WK2] EWK2CookieManagerTest should be defined by inheriting from EWK2UnitTestBase. | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Donggwan Kim <donggwan.kim> | ||||||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | cdumez, commit-queue, gyuyoung.kim, rakuco | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Other | ||||||||||
OS: | Linux | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 118716 | ||||||||||
Attachments: |
|
Description
Donggwan Kim
2013-07-16 01:48:21 PDT
Created attachment 206746 [details]
Patch
Comment on attachment 206746 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=206746&action=review > Source/WebKit2/ChangeLog:7 > + Missing Changelog. > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_cookie_manager.cpp:85 > +static int compareHostNames(const void* hostName1, const void* hostName2) Could you explain why some static functions are moved to EWK2CookieManagerTest while others are not? Created attachment 206759 [details]
Patch
(In reply to comment #2) > (From update of attachment 206746 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=206746&action=review > > > Source/WebKit2/ChangeLog:7 > > + > > Missing Changelog. Done. > > > Source/WebKit2/UIProcess/API/efl/tests/test_ewk2_cookie_manager.cpp:85 > > +static int compareHostNames(const void* hostName1, const void* hostName2) > > Could you explain why some static functions are moved to EWK2CookieManagerTest while others are not? I made a mistake. I will move all the missed static functions to EWK2CookieManagerTest and update the patch again. Created attachment 206761 [details]
Patch
Comment on attachment 206761 [details]
Patch
Ok, r=me.
Comment on attachment 206761 [details] Patch Clearing flags on attachment: 206761 Committed r152776: <http://trac.webkit.org/changeset/152776> All reviewed patches have been landed. Closing bug. |