Rename a few macro and variables about intrinsic inline-size

RESOLVED FIXED in Firefox 68

Status

()

task
P3
normal
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: TYLin, Assigned: TYLin)

Tracking

unspecified
mozilla68
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox68 fixed)

Details

Attachments

(4 attachments)

Bug 1490611 added some FIXME to rename NS_INTRINSIC_WIDTH_UNKNOWN and nsBlockFrame::{mMinWidth, mPrefWidth}.

I'll post patches to rename them later.

This patch is generated by running the following script under layout/,
and then manually delete the FIXME comment in LayoutConstants.h

#!/bin/bash
function rename() {
find .
-type f
! -path "./obj*"
! -path "./.git"
! -path "./.hg"
( -name ".cpp" -or
-name "
.h" )
-exec sed -i -e "s/$1/$2/g" "{}" ;
}
rename NS_INTRINSIC_WIDTH_UNKNOWN NS_INTRINSIC_ISIZE_UNKNOWN

Instead of renaming it to mMinISize, I choose mCachedMinISize because
both grid and flex container frames use mCachedMinISize for similar
purpose.

Depends on D29746

Like Part 2, I choose mCachedPrefISize because grid and flex container
are using it.

Depends on D29747

Pushed by aethanyc@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a3ddb4c1ce28
Part 1 - Rename NS_INTRINSIC_WIDTH_UNKNOWN to NS_INTRINSIC_ISIZE_UNKNOWN. r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/2a7dcc0cc43d
Part 2 - Rename nsBlockFrame's mMinWidth to mCachedMinISize. r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/5ad9fb7a2713
Part 3 - Rename nsBlockFrame's mPrefWidth to mCachedPrefISize. r=jfkthame
https://hg.mozilla.org/integration/autoland/rev/00441f7cdd1c
Part 4 - Fix the display macro usage in nsFileControlFrame::GetPrefISize(). r=jfkthame
Assignee: nobody → aethanyc
You need to log in before you can comment on or make changes to this bug.