[M69 Dev] fixup! [M51_2704] Chromium upversion to m51_2704 branch. 30/194930/2
authoryh106.jung <yh106.jung@samsung.com>
Tue, 6 Dec 2016 09:42:03 +0000 (18:42 +0900)
committerBot Blink <blinkbot@samsung.com>
Mon, 10 Dec 2018 07:43:58 +0000 (07:43 +0000)
It's wrong to dereference a std::unique_ptr after std::move because
the ownership of std::unique_ptr is transferred by std::move.

Reference:
https://review.tizen.org/gerrit/#/c/platform/framework/web/chromium-efl/+/174884/

Change-Id: Id6b2521ba70f0247b9df71ca1d6ea90ba52a18a8
Signed-off-by: yh106.jung <yh106.jung@samsung.com>
tizen_src/ewk/efl_integration/url_request_context_getter_efl.cc

index 87d2f86..e4cfd5c 100644 (file)
@@ -316,7 +316,6 @@ void URLRequestContextGetterEfl::CreateSQLitePersistentCookieStore(
   // longer referenced.
   std::unique_ptr<net::CookieMonster> cookie_monster(
       new net::CookieMonster(persistent_store.get(), NULL));
-  storage_->set_cookie_store(std::move(cookie_monster));
 
   if (persistent_store.get() && persist_session_cookies)
     cookie_monster->SetPersistSessionCookies(true);
@@ -328,6 +327,8 @@ void URLRequestContextGetterEfl::CreateSQLitePersistentCookieStore(
   schemes.push_back(url::kFileScheme);
   cookie_monster->SetCookieableSchemes(schemes);
 
+  storage_->set_cookie_store(std::move(cookie_monster));
+
   cookie_store_path_ = path;
 }