Remove "web" word from ewk files in order to be consistent with WK2. Bug 97886 and Bug 97887 removed *web* from WK2. This was discussed in webkit-efl mailing list. http://lists.webkit.org/pipermail/webkit-efl/2012-September/000308.html.
Created attachment 167695 [details] Patch
Created attachment 167702 [details] Patch
I disagree with blind replacements like this -- some features simply have 'web' in their names, such as Web Audio or Web Databases (which is actually Web SQL Database).
(In reply to comment #3) > I disagree with blind replacements like this -- some features simply have 'web' in their names, such as Web Audio or Web Databases (which is actually Web SQL Database). The WK API name for Web Database is just WKDatabaseXXXXX. So, I think that we can remove "web" word from the implementation for web databases.
(In reply to comment #3) > I disagree with blind replacements like this -- some features simply have 'web' in their names, such as Web Audio or Web Databases (which is actually Web SQL Database). As Raphael said, we may argue with web database and web audio in WK1. Because other ports in WK1 are still using *web*. It looks I need to consider this further. BTW, how about removing web from web_inspector ? I think *web* is redundant in inspector.
(In reply to comment #5) > (In reply to comment #3) > > I disagree with blind replacements like this -- some features simply have 'web' in their names, such as Web Audio or Web Databases (which is actually Web SQL Database). > > As Raphael said, we may argue with web database and web audio in WK1. Because other ports in WK1 are still using *web*. It looks I need to consider this further. > > BTW, how about removing web from web_inspector ? I think *web* is redundant in inspector. Yeah, I think removing this one is OK.
Created attachment 167732 [details] Patch
I only remove *web* from web inspector. Other will be considered further. Jongseok and Seokju, how do you think ?
Beside other ports just use *inspector* except for gtk and win ports.
Comment on attachment 167732 [details] Patch LGTM
(In reply to comment #8) > I only remove *web* from web inspector. Other will be considered further. Jongseok and Seokju, how do you think ? No problem. I have also removed "web" from web inspector on Bug 98639.
Comment on attachment 167732 [details] Patch This change makes the api more consistent. r=me.
Comment on attachment 167732 [details] Patch Clearing flags on attachment: 167732 Committed r131119: <http://trac.webkit.org/changeset/131119>
All reviewed patches have been landed. Closing bug.