Summary: | Renaming of overrides in RenderBox | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Manuel Rego Casasnovas <rego> | ||||
Component: | New Bugs | Assignee: | Manuel Rego Casasnovas <rego> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, jfernandez, webkit-bug-importer, zalan | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Manuel Rego Casasnovas
2018-05-14 07:12:06 PDT
Created attachment 340309 [details]
Patch
This is just a renaming, I'll try to fix the FIXME in LayoutBox in follow-up patches. Comment on attachment 340309 [details] Patch Clearing flags on attachment: 340309 Committed r231757: <https://trac.webkit.org/changeset/231757> All reviewed patches have been landed. Closing bug. (In reply to Manuel Rego Casasnovas from comment #0) > Renaming of overrides in LayoutBox What is a LayoutBox? (In reply to zalan from comment #6) > (In reply to Manuel Rego Casasnovas from comment #0) > > Renaming of overrides in LayoutBox > > What is a LayoutBox? guess this is some leftover from a blink patch. (In reply to zalan from comment #6) > (In reply to Manuel Rego Casasnovas from comment #0) > > Renaming of overrides in LayoutBox > > What is a LayoutBox? Ooops sorry, this is becuase we're doing things in both Blink and WebKit, and now in Blink RenderBox is called LayoutBox. Should I do a quick patch fixing the ChangeLog or not? (In reply to Manuel Rego Casasnovas from comment #8) > (In reply to zalan from comment #6) > > (In reply to Manuel Rego Casasnovas from comment #0) > > > Renaming of overrides in LayoutBox > > > > What is a LayoutBox? > > Ooops sorry, this is becuase we're doing things in both Blink and WebKit, > and now in Blink RenderBox is called LayoutBox. > > Should I do a quick patch fixing the ChangeLog or not? nah. It's fine. I was just a bit surprised (especially since I also started using LayoutBox in webcore/layout) |