Attachment #355250 for bug #191626

View Details Raw Unified Return to Bug 191626

| Differences between



2018-11-18  Fujii Hironori  <Hironori.Fujii@sony.com>

        REGRESSION(r238039) WebCore::JSDOMGlobalObject::createStructure is using JSC::Structure::create without including StructureInlines.h
        https://bugs.webkit.org/show_bug.cgi?id=191626

        Reviewed by Yusuke Suzuki.

        JSC::Structure::create is used everywhere. It should be defined in
        Structure.h, not in StructureInlines.h.

        * runtime/Structure.h:
        (JSC::Structure::create): Moved.
        * runtime/StructureInlines.h: Moved JSC::Structure::create.

2018-11-18  Yusuke Suzuki  <yusukesuzuki@slowstart.org>

        Unreviewed, rolling in the rest of r237254

Lines 828-831 private: a/Source/JavaScriptCore/runtime/Structure.h_sec1
- a/Source/JavaScriptCore/runtime/Structure.h +9 lines
828	uint32_t m_propertyHash;	828	uint32_t m_propertyHash;
829	};	829	};
830		830
		831	inline Structure* Structure::create(VM& vm, JSGlobalObject* globalObject, JSValue prototype, const TypeInfo& typeInfo, const ClassInfo* classInfo, IndexingType indexingType, unsigned inlineCapacity)
		832	{
		833	ASSERT(vm.structureStructure);
		834	ASSERT(classInfo);
		835	Structure* structure = new (NotNull, allocateCell<Structure>(vm.heap)) Structure(vm, globalObject, prototype, typeInfo, classInfo, indexingType, inlineCapacity);
		836	structure->finishCreation(vm);
		837	return structure;
		838	}
		839
831	} // namespace JSC	840	} // namespace JSC




namespace JSC {

inline Structure* Structure::create(VM& vm, JSGlobalObject* globalObject, JSValue prototype, const TypeInfo& typeInfo, const ClassInfo* classInfo, IndexingType indexingType, unsigned inlineCapacity)
{
    ASSERT(vm.structureStructure);
    ASSERT(classInfo);
    Structure* structure = new (NotNull, allocateCell<Structure>(vm.heap)) Structure(vm, globalObject, prototype, typeInfo, classInfo, indexingType, inlineCapacity);
    structure->finishCreation(vm);
    return structure;
}

inline Structure* Structure::createStructure(VM& vm)
{
    ASSERT(!vm.structureStructure);

Return to Bug 191626

Lines 1-3 a/Source/JavaScriptCore/ChangeLog_sec1
- a/Source/JavaScriptCore/ChangeLog +14 lines
		1	2018-11-18 Fujii Hironori <Hironori.Fujii@sony.com>
		2
		3	REGRESSION(r238039) WebCore::JSDOMGlobalObject::createStructure is using JSC::Structure::create without including StructureInlines.h
		4	https://bugs.webkit.org/show_bug.cgi?id=191626
		5
		6	Reviewed by Yusuke Suzuki.
		7
		8	JSC::Structure::create is used everywhere. It should be defined in
		9	Structure.h, not in StructureInlines.h.
		10
		11	* runtime/Structure.h:
		12	(JSC::Structure::create): Moved.
		13	* runtime/StructureInlines.h: Moved JSC::Structure::create.
		14
1	2018-11-18 Yusuke Suzuki <yusukesuzuki@slowstart.org>	15	2018-11-18 Yusuke Suzuki <yusukesuzuki@slowstart.org>
2		16
3	Unreviewed, rolling in the rest of r237254	17	Unreviewed, rolling in the rest of r237254

Lines 34-48 a/Source/JavaScriptCore/runtime/StructureInlines.h_sec1
- a/Source/JavaScriptCore/runtime/StructureInlines.h -9 lines
34		34
35	namespace JSC {	35	namespace JSC {
36		36
37	inline Structure* Structure::create(VM& vm, JSGlobalObject* globalObject, JSValue prototype, const TypeInfo& typeInfo, const ClassInfo* classInfo, IndexingType indexingType, unsigned inlineCapacity)
38	{
39	ASSERT(vm.structureStructure);
40	ASSERT(classInfo);
41	Structure* structure = new (NotNull, allocateCell<Structure>(vm.heap)) Structure(vm, globalObject, prototype, typeInfo, classInfo, indexingType, inlineCapacity);
42	structure->finishCreation(vm);
43	return structure;
44	}
45
46	inline Structure* Structure::createStructure(VM& vm)	37	inline Structure* Structure::createStructure(VM& vm)
47	{	38	{
48	ASSERT(!vm.structureStructure);	39	ASSERT(!vm.structureStructure);