Merge "Fix "use of memory after freed" warning." am: 3f5160ca am: db5075c9

SimpleBestFitAllocator::~SimpleBestFitAllocator()

{

    while(!mList.isEmpty()) {

        delete mList.remove(mList.head());

        chunk_t* removed = mList.remove(mList.head());

#ifdef __clang_analyzer__

        // Clang static analyzer gets confused in this loop

        // and generates a false positive warning about accessing

        // memory that is already freed.

        // Add an "assert" to avoid the confusion.

        LOG_ALWAYS_FATAL_IF(mList.head() == removed);

#endif

        delete removed;

    }

}