Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(595)

Unified Diff: content/child/blink_platform_impl.cc

Issue 2846843002: [blink] Unique pointers in Platform.h (Closed)
Patch Set: fix content_shell compilation Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/child/blink_platform_impl.cc
diff --git a/content/child/blink_platform_impl.cc b/content/child/blink_platform_impl.cc
index 74044ea67edd0c7c3f58e2869664c22be86c354e..0e041ce9e3cc39f391917fc01211f5cc09d83b69 100644
--- a/content/child/blink_platform_impl.cc
+++ b/content/child/blink_platform_impl.cc
@@ -52,6 +52,7 @@
#include "net/base/net_errors.h"
#include "third_party/WebKit/public/platform/WebData.h"
#include "third_party/WebKit/public/platform/WebFloatPoint.h"
+#include "third_party/WebKit/public/platform/WebGestureCurve.h"
#include "third_party/WebKit/public/platform/WebSecurityOrigin.h"
#include "third_party/WebKit/public/platform/WebString.h"
#include "third_party/WebKit/public/platform/WebURL.h"
@@ -402,13 +403,14 @@ WebURLError BlinkPlatformImpl::CancelledError(
return CreateWebURLError(unreachableURL, false, net::ERR_ABORTED);
}
-blink::WebThread* BlinkPlatformImpl::CreateThread(const char* name) {
+std::unique_ptr<blink::WebThread> BlinkPlatformImpl::CreateThread(
+ const char* name) {
std::unique_ptr<blink::scheduler::WebThreadBase> thread =
blink::scheduler::WebThreadBase::CreateWorkerThread(
name, base::Thread::Options());
thread->Init();
WaitUntilWebThreadTLSUpdate(thread.get());
- return thread.release();
+ return thread;
}
void BlinkPlatformImpl::SetCompositorThread(
@@ -685,14 +687,15 @@ blink::WebThread* BlinkPlatformImpl::CompositorThread() const {
return compositor_thread_;
}
-blink::WebGestureCurve* BlinkPlatformImpl::CreateFlingAnimationCurve(
+std::unique_ptr<blink::WebGestureCurve>
+BlinkPlatformImpl::CreateFlingAnimationCurve(
blink::WebGestureDevice device_source,
const blink::WebFloatPoint& velocity,
const blink::WebSize& cumulative_scroll) {
return ui::WebGestureCurveImpl::CreateFromDefaultPlatformCurve(
- gfx::Vector2dF(velocity.x, velocity.y),
- gfx::Vector2dF(cumulative_scroll.width, cumulative_scroll.height),
- IsMainThread()).release();
+ gfx::Vector2dF(velocity.x, velocity.y),
+ gfx::Vector2dF(cumulative_scroll.width, cumulative_scroll.height),
+ IsMainThread());
}
void BlinkPlatformImpl::DidStartWorkerThread() {
@@ -833,7 +836,7 @@ int BlinkPlatformImpl::DomKeyEnumFromString(const WebString& key_string) {
ui::KeycodeConverter::KeyStringToDomKey(key_string.Utf8()));
}
-blink::WebFeaturePolicy* BlinkPlatformImpl::CreateFeaturePolicy(
+std::unique_ptr<blink::WebFeaturePolicy> BlinkPlatformImpl::CreateFeaturePolicy(
const blink::WebFeaturePolicy* parent_policy,
const blink::WebParsedFeaturePolicy& container_policy,
const blink::WebParsedFeaturePolicy& policy_header,
@@ -842,16 +845,17 @@ blink::WebFeaturePolicy* BlinkPlatformImpl::CreateFeaturePolicy(
static_cast<const FeaturePolicy*>(parent_policy),
FeaturePolicyHeaderFromWeb(container_policy), url::Origin(origin));
policy->SetHeaderPolicy(FeaturePolicyHeaderFromWeb(policy_header));
- return policy.release();
+ return policy;
}
-blink::WebFeaturePolicy* BlinkPlatformImpl::DuplicateFeaturePolicyWithOrigin(
+std::unique_ptr<blink::WebFeaturePolicy>
+BlinkPlatformImpl::DuplicateFeaturePolicyWithOrigin(
const blink::WebFeaturePolicy& policy,
const blink::WebSecurityOrigin& new_origin) {
std::unique_ptr<FeaturePolicy> new_policy =
FeaturePolicy::CreateFromPolicyWithOrigin(
static_cast<const FeaturePolicy&>(policy), url::Origin(new_origin));
- return new_policy.release();
+ return new_policy;
}
} // namespace content

Powered by Google App Engine
This is Rietveld 408576698