DescriptionRevert of Fix free memory calculation. (patchset #27 id:520001 of https://codereview.chromium.org/2558043007/ )
Reason for revert:
SysInfoTest.AmountOfAvailablePhysicalMemory is failing
BUG=703522
Original issue's description:
> Fix free memory calculation.
>
> The term "free memory" can mean different things: it can mean
> 1) absolutely/pure free physical memory (not consumed by anything) or
> 2) it can include the inactive part(s) of OS (file) cache(s) or
> 3) it can mean the maximum amount of physical memory that can be
> allocated quickly without swapping and thus can also include the
> active part(s) of OS cache(s).
>
> This CL fixes base::AmountOfAvailablePhysicalMemory() to function
> according to the 2nd meaning on all platforms where it is possible
> because this is what everyone needs most of the time. (Win platform
> has already had such implementation but others haven't).
>
> BUG=672745
>
> Review-Url: https://codereview.chromium.org/2558043007
> Cr-Commit-Position: refs/heads/master@{#458291}
> Committed: https://chromium.googlesource.com/chromium/src/+/a4258cad75d8cc63170555cc74bd584b5bfca965
TBR=primiano@chromium.org,danakj@chromium.org,brucedawson@chromium.org,xiyuan@chromium.org,chrisha@chromium.org,haraken@chromium.org,mkolom@yandex-team.ru
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672745
Review-Url: https://codereview.chromium.org/2766623002
Cr-Commit-Position: refs/heads/master@{#458333}
Committed: https://chromium.googlesource.com/chromium/src/+/ad039f65398a94f4a28751c303df388cff4bbca3
Patch Set 1 #
Messages
Total messages: 6 (3 generated)
|