Summary: | [GTK] [EFL] Install TestNetscapePlugin apart from the other libs to avoid scanning all of them on every test. | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Carlos Alberto Lopez Perez <clopez> | ||||||
Component: | Tools / Tests | Assignee: | Carlos Alberto Lopez Perez <clopez> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cgarcia, commit-queue, glenn, gustavo, gyuyoung.kim, mrobinson, ossy, rakuco, ryuan.choi, sergio | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Carlos Alberto Lopez Perez
2014-09-29 07:23:57 PDT
This is not exactly a regression, but an existing issue that has been revealed thanks to the error dialogs introduced in r174054. We should not use the same directory for the test plugin and the other libraries, because we end up scanning all the .so in the lib dir for every test. (In reply to comment #1) > This is not exactly a regression, but an existing issue that has been revealed thanks to the error dialogs introduced in r174054. We should not use the same directory for the test plugin and the other libraries, because we end up scanning all the .so in the lib dir for every test. Agreed. Renamed the subject of the bug. Created attachment 238854 [details]
Patch
Comment on attachment 238854 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=238854&action=review Thanks! Gyuyoung, is this ok for the EFL port? > Tools/Scripts/webkitpy/port/efl.py:68 > - env['TEST_RUNNER_PLUGIN_PATH'] = self._build_path('lib') > + env['TEST_RUNNER_PLUGIN_PATH'] = self._build_path('lib/plugins') Nit: this should be self._build_path('lib', 'plugins') so that build_path can use the appropriate dir separator on every platform. Not a big deal, since this is only used in unix, but still. > Tools/Scripts/webkitpy/port/gtk.py:113 > - environment['TEST_RUNNER_TEST_PLUGIN_PATH'] = self._build_path('lib') > + environment['TEST_RUNNER_TEST_PLUGIN_PATH'] = self._build_path('lib/plugins') Ditto. Created attachment 238859 [details]
Patch
Comment on attachment 238859 [details]
Patch
OK, Gyuyoung, please cq+ this if it it's ok for EFL.
Comment on attachment 238859 [details]
Patch
LGTM. libTestNetscapePlugin.so is placed at lib/plugins.
Comment on attachment 238859 [details] Patch Clearing flags on attachment: 238859 Committed r174093: <http://trac.webkit.org/changeset/174093> All reviewed patches have been landed. Closing bug. |