Chromium Code Reviews Chromium Code Reviews
Issue
1287113002:
Clean up PageBoundaryRule enum. (Closed)
DescriptionClean up PageBoundaryRule enum.
Replace the {Include,Exclude}PageBoundary values with
AssociateWith{Former,Latter}Page, since it was rather confusing what was meant
by "include" or "exclude" in most situations. Also remove default
PageBoundaryRule values from method declarations taking this enum as a
parameter, since there's really no natural default value, and each caller
should be aware of what it's asking for.
Realizing what's actually going on here allowed for some cleanup in
nextPageLogicalTop().
Also made the enum public, since there are public methods taking it as a
parameter.
No behavioral changes intended.
R=leviw@chromium.org
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200438
Patch Set 1 #
Messages
Total messages: 5 (1 generated)
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
