Bug 114982
| Summary: | Web Inspector: InspectorCSSAgent::SetPropertyTextAction has a bad mergeId | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Timothy Hatcher <timothy> |
| Component: | Web Inspector | Assignee: | Nobody <webkit-unassigned> |
| Status: | ASSIGNED | ||
| Severity: | Normal | CC: | inspector-bugzilla-changes, webkit-bug-importer |
| Priority: | P2 | Keywords: | InRadar |
| Version: | 528+ (Nightly build) | ||
| Hardware: | All | ||
| OS: | All | ||
| Bug Depends on: | |||
| Bug Blocks: | 116924 | ||
Timothy Hatcher
The mergeId for InspectorCSSAgent::SetPropertyTextAction is:
String::format("SetPropertyText %s:%u:%s", m_styleSheet->id().utf8().data(), m_propertyIndex, m_overwrite ? "true" : "false");
But if you have two properties from different rules but the same index those would merge if edited back to back. We should include m_cssId.ordinal() in there.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Timothy Hatcher
Moving to the right component.
Radar WebKit Bug Importer
<rdar://problem/15796837>