Closed
Bug 1495994
Opened 6 years ago
Closed 6 years ago
merge ImageValue, URLValue, and URLValueData
Categories
(Core :: CSS Parsing and Computation, enhancement, P3)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla64
Tracking | Status | |
---|---|---|
firefox64 | --- | fixed |
People
(Reporter: heycam, Assigned: heycam)
References
Details
Attachments
(4 files, 1 obsolete file)
The split between these types is a needless complication after bug 1495645.
Assignee | ||
Comment 1•6 years ago
|
||
All things named "ImageValue" (member variables, arguments, etc.) are renamed to "URLValue".
Assignee | ||
Comment 2•6 years ago
|
||
Assignee | ||
Comment 3•6 years ago
|
||
Depends on D7593
Assignee | ||
Comment 4•6 years ago
|
||
Depends on D7594
Updated•6 years ago
|
Attachment #9013986 -
Attachment is obsolete: true
Updated•6 years ago
|
Attachment #9013986 -
Attachment is obsolete: false
Assignee | ||
Comment 5•6 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f2b993bf4b20c71d151162d9858078e260d7e0d&group_state=expanded Will probably need to rebase over the nsCSSValue simplifications but shouldn't be much.
Assignee | ||
Comment 6•6 years ago
|
||
Depends on D7595
Updated•6 years ago
|
Attachment #9013986 -
Attachment is obsolete: true
Pushed by cmccormack@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a6de39c7149c Part 1: Make URLValue constructor take CORSMode argument r=emilio https://hg.mozilla.org/integration/autoland/rev/68ce4cece28c Part 2: Remove some constructors/methods r=emilio https://hg.mozilla.org/integration/autoland/rev/48b5f15672ab Part 3: Move mLoadID up from ImageValue to URLValueData r=emilio https://hg.mozilla.org/integration/autoland/rev/f6e5bc9af12f Part 4: Merge css::{URLValueData, ImageValue} into css::URLValue r=emilio
Comment 8•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a6de39c7149c https://hg.mozilla.org/mozilla-central/rev/68ce4cece28c https://hg.mozilla.org/mozilla-central/rev/48b5f15672ab https://hg.mozilla.org/mozilla-central/rev/f6e5bc9af12f
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•