Bug 73919 – Layout Test inspector/debugger/dom-breakpoints.html fails on chromium linux debug with ENABLE(MUTATION_OBSERVERS)

• Status: RESOLVED
• Resolution: FIXED
• Component: DOM

[Vsevolod Vlasov]

The following layout test is failing on chromium linux debug

inspector/debugger/dom-breakpoints.html

Probable cause:

http://trac.webkit.org/changeset/102100/

[Vsevolod Vlasov]

Marked test as crashing for now: <http://trac.webkit.org/changeset/102133>

[Vsevolod Vlasov]

Removed expectations now that the original patch was rolled out.

Committed r102144: <http://trac.webkit.org/changeset/102144>

[Adam Klein]

Here's the stack trace

ASSERTION FAILED: s_currentDecl == decl
third_party/WebKit/Source/WebCore/css/CSSMutableStyleDeclaration.cpp(59) : WebCore::<unnamed>::StyleAttributeMutationScope::StyleAttributeMutationScope(WebCore::CSSMutableStyleDeclaration*)
1   0x109475c
2   0x109799d
3   0x10978d2
4   0x10e768e
5   0x183550e
6   0x6181ec62c01
[38988:38988:8694432581833:ERROR:process_util_posix.cc(139)] Received signal 11
	base::debug::StackTrace::StackTrace() [0x715496]
	base::(anonymous namespace)::StackDumpSignalHandler() [0x6d1b8d]
	0x7f2cbffd8af0
	WebCore::(anonymous namespace)::StyleAttributeMutationScope::StyleAttributeMutationScope() [0x1094761]
	WebCore::CSSMutableStyleDeclaration::setProperty() [0x109799d]
	WebCore::CSSMutableStyleDeclaration::setProperty() [0x10978d2]
	WebCore::CSSStyleDeclaration::setProperty() [0x10e768e]
	WebCore::CSSStyleDeclarationInternal::setPropertyCallback() [0x183550e]
	0x6181ec62c01

[Adam Klein]

Created attachment 118131 [details]
Patch

[Adam Klein]

Adding pfeldman for inspector knowledge. In some cases, this patch will cause InspectorInstrumentation::didInvalidateStyleAttr() to be called slightly later, but it should still be called whenever it would have been called before.

[Early Warning System Bot]

Comment on attachment 118131 [details]
Patch

Attachment 118131 [details] did not pass qt-ews (qt):
Output: http://queues.webkit.org/results/10736895

[Gyuyoung Kim]

Comment on attachment 118131 [details]
Patch

Attachment 118131 [details] did not pass efl-ews (efl):
Output: http://queues.webkit.org/results/10746079

[Adam Klein]

Created attachment 118137 [details]
Move ifdef guards appropriately

[WebKit Review Bot]

Comment on attachment 118137 [details]
Move ifdef guards appropriately

Clearing flags on attachment: 118137

Committed r102264: <http://trac.webkit.org/changeset/102264>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.