[M34-Merge] Avoid overriding DIR_MODULE path also on desktop.
authorPiotr Tworek <p.tworek@samsung.com>
Thu, 9 Oct 2014 15:56:44 +0000 (17:56 +0200)
committerYoungsoo Choi <kenshin.choi@samsung.com>
Tue, 10 Jul 2018 06:57:09 +0000 (06:57 +0000)
On desktop DIR_MODULE is used to find files in build directory.
Overriding it makes efl_webview_app or content shell look for some of
the resources it needs in wrong directories under
out.x64/Debug/resources/resources instead of out.x64/Debug/resources.

The code was not present in the original code copied from chrome. For
unknown reasons it is only enabled in chromium-efl desktop builds.
Remove it as it's clearly wrong. We should not override paths which may
be used as base values for other paths.

M34-Merge: http://165.213.202.130:8080/#/c/70612/
Reviewed by: Antonio Gomes, Piotr Grad, SeungSeop Park, commitbot

Change-Id: If02c9d6ec314a9d327a04754cea3bd3bea5375a5
Signed-off-by: Arnaud Renevier <a.renevier@samsung.com>
tizen_src/impl/browser/webdata/web_data_service_factory.cc

index 024c1969c29e0b5b56c7285fb283d1c67019c8e0..42ea0ec76689c7142baee986dee4077aa88bda36 100644 (file)
@@ -46,12 +46,6 @@ WebDataServiceWrapper* WebDataServiceWrapper::GetInstance(){
 WebDataServiceWrapper::WebDataServiceWrapper() {
   base::FilePath db_path;
   PathService::Get(PathsEfl::WEB_DATABASE_DIR, &db_path);
-#ifndef OS_TIZEN
-  if(PathService::Override(base::DIR_MODULE, db_path) == false) {
-    LOG(ERROR)<<"Could not access web database path.";
-    return;
-  }
-#endif
   base::FilePath path = db_path.Append(FILE_PATH_LITERAL(".FormData.db"));
 
   scoped_refptr<base::MessageLoopProxy> ui_thread =