Add WorkerSettings to expose certain flag values in WorkerGlobalScope

third_party/WebKit/Source/core/workers/WorkerSettings.h

+// Copyright 2016 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.
+
+#ifndef WorkerSettings_h
+#define WorkerSettings_h
+
+#include "core/CoreExport.h"
+#include "core/dom/ExecutionContext.h"
+#include "core/frame/Settings.h"
+#include "core/workers/WorkerClients.h"
+#include "platform/Supplementable.h"
+#include "platform/heap/Handle.h"
+
+namespace blink {
+
+class CORE_EXPORT WorkerSettings : public GarbageCollectedFinalized<WorkerSettings>, public Supplement<WorkerClients> {

[kinuko, 2016/07/22 09:02:54]

Does this need to be on-heap? (Just asking)

[xlai (Olivia), 2016/07/22 21:53:30]

Hmmm, not necessary. But isn't better to be on heap so that we don't worry about
garbage collecting it?

+    USING_GARBAGE_COLLECTED_MIXIN(WorkerSettings);
+    WTF_MAKE_NONCOPYABLE(WorkerSettings);
+
+public:
+    virtual ~WorkerSettings() {}
+    DECLARE_VIRTUAL_TRACE();
+
+    static WorkerSettings* create(Settings*);
+
+    static const char* supplementName();
+    static WorkerSettings* from(ExecutionContext*);
+
+    bool disableReadingFromCanvas() const { return m_disableReadingFromCanvas; }
+
+private:
+    WorkerSettings() {}
+    void copyFlagValuesFromSettings(Settings*);
+    // The settings that are to be copied from main thread to worker thread
+    // These setting's flag values must remain unchanged throughout the document
+    // lifecycle.
+    // We hard-code the flags as there're very few flags at this moment.
+    bool m_disableReadingFromCanvas = false;
+};
+
+CORE_EXPORT void provideWorkerSettingsToWorker(WorkerClients*, WorkerSettings*);
+}

[kinuko, 2016/07/22 09:02:54]

nit: please have empty line before line 43, and make the line 43 as following
for consistency

} // namespace blink

[xlai (Olivia), 2016/07/22 21:53:30]

Done.

+
+#endif // WorkerSettings_h