Bug 1323517 Part 3 - Rename GetLowestFloatTop to LowestFloatBStart. r=jfkthame

Both the method's documentation and implementation already use logical
coordinate.

Differential Revision: https://phabricator.services.mozilla.com/D151205

layout/generic/BlockReflowState.cpp

@@ -720,7 +720,7 @@ bool BlockReflowState::FlowAndPlaceFloat(nsIFrame* aFloat) {
 
   // Enforce CSS2 9.5.1 rule [2], i.e., make sure that a float isn't
   // ``above'' another float that preceded it in the flow.
-  mBCoord = std::max(FloatManager()->GetLowestFloatTop(), mBCoord);
+  mBCoord = std::max(FloatManager()->LowestFloatBStart(), mBCoord);
 
   // See if the float should clear any preceding floats...
   // XXX We need to mark this float somehow so that it gets reflowed

layout/generic/nsFloatManager.cpp

@@ -432,7 +432,7 @@ void nsFloatManager::PopState(SavedState* aState) {
   mFloats.TruncateLength(aState->mFloatInfoCount);
 }
 
-nscoord nsFloatManager::GetLowestFloatTop() const {
+nscoord nsFloatManager::LowestFloatBStart() const {
   if (mPushedLeftFloatPastBreak || mPushedRightFloatPastBreak) {
     return nscoord_MAX;
   }

layout/generic/nsFloatManager.h

@@ -306,7 +306,7 @@ class nsFloatManager {
    *
    * The result is relative to the current translation.
    */
-  nscoord GetLowestFloatTop() const;
+  nscoord LowestFloatBStart() const;
 
   /**
    * Return the coordinate of the lowest float matching aBreakType in