Expose performance.memory in workers

Source/web/WebWorkerSettings.cpp

+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "config.h"
+#include "WebWorkerSettings.h"
+
+#include "core/workers/WorkerSettings.h"
+#include "wtf/OwnPtr.h"
+
+using namespace WebCore;
+
+namespace blink {
+
+WebWorkerSettings WebWorkerSettings::create()
+{
+    return WebWorkerSettings(WorkerSettings::create());
+}
+
+void WebWorkerSettings::assign(const WebWorkerSettings& other)
+{
+    m_private.reset(PassOwnPtr<WorkerSettings>(other));
+}
+
+void WebWorkerSettings::reset()
+{
+    m_private.reset(0);
+}
+
+void WebWorkerSettings::setMemoryInfoEnabled(bool enabled)
+{
+    m_private->setMemoryInfoEnabled(enabled);
+}
+
+WebWorkerSettings::WebWorkerSettings(const PassOwnPtr<WorkerSettings>& settings)
+    : m_private(settings)
+{
+}
+
+WebWorkerSettings::operator PassOwnPtr<WorkerSettings>() const
+{
+    OwnPtr<WorkerSettings> settings = WorkerSettings::create();
+    settings->setMemoryInfoEnabled(m_private->memoryInfoEnabled());

[kinuko, 2014/03/10 05:55:52]

m_private itself is WorkerSettings, for our usage something like following looks
enough to me

PassOwnPtr<WorkerSettings> release() { return m_private.release(); }

[Pan, 2014/03/10 12:27:52]

this method is also used in assign(const WebWorkerSettings&), from copy
constructor and operator=.
otherwise I need to add a create(WorkerSettings*) to WebCore::WorkerSettings
then use it in assign, meanwhile, in WebWorkerSettings.h, add a '#include
"core/workers/WorkerSettings.h"' between BLINK_IMPLEMENTATION.
I don't have strong preference, is the later one acceptable? 
thanks
Pan

[kinuko, 2014/03/10 13:50:41]

I see, I overlooked it. I'd add WorkerSettings::create(const WorkerSettings&)
(which can just call new WorkerSettings(settings) as it's copyable) then, and
use it in assign() and in this operator (with *m_private.get()).  I don't see
why we need to include WorkerSettings.h in WebWorkerSettings.h though, it
doesn't seem necessary?

+    return settings.release();
+}
+
+} // namespace blink