mirror of
https://github.com/flutter/flutter.git
synced 2026-02-20 02:29:02 +08:00
b50499d50d
This reverts commit 74823c212d418597775d332d8c272673c83f6f63 and relands our reland https://github.com/flutter/engine/pull/17915. Additionally, we fixed the cull rect logic in `OpacityLayer::Preroll` which is the root cause of https://github.com/flutter/flutter/issues/56298. We've always had that root problem before but it did not trigger performance issues because we were using the OpacityLayer's `paint_bounds`, instead of its child's `paint_bounds` for preparing the layer raster cache. A correct handling of the cull rect should allow us to cull at any level. It also turns out that our ios32 (iPhone4s) performacne can regress a lot without snapping. My theory is that although the picture has a fractional top left corner, many drawing operations inside the picture have integral coordinations. In older hardwares, keeping those coordinates integral seems to be performance critical. To avoid flutter/flutter#41654, the snapping will still be disabled if the matrix has non-scale-translation transformations.
62 lines
1.6 KiB
C++
62 lines
1.6 KiB
C++
// Copyright 2013 The Flutter Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
#ifndef FLUTTER_FLOW_RASTER_CACHE_KEY_H_
|
|
#define FLUTTER_FLOW_RASTER_CACHE_KEY_H_
|
|
|
|
#include <unordered_map>
|
|
#include "flutter/flow/matrix_decomposition.h"
|
|
#include "flutter/fml/logging.h"
|
|
|
|
namespace flutter {
|
|
|
|
template <typename ID>
|
|
class RasterCacheKey {
|
|
public:
|
|
RasterCacheKey(ID id, const SkMatrix& ctm) : id_(id), matrix_(ctm) {
|
|
matrix_[SkMatrix::kMTransX] = 0;
|
|
matrix_[SkMatrix::kMTransY] = 0;
|
|
}
|
|
|
|
ID id() const { return id_; }
|
|
const SkMatrix& matrix() const { return matrix_; }
|
|
|
|
struct Hash {
|
|
uint32_t operator()(RasterCacheKey const& key) const {
|
|
return std::hash<ID>()(key.id_);
|
|
}
|
|
};
|
|
|
|
struct Equal {
|
|
constexpr bool operator()(const RasterCacheKey& lhs,
|
|
const RasterCacheKey& rhs) const {
|
|
return lhs.id_ == rhs.id_ && lhs.matrix_ == rhs.matrix_;
|
|
}
|
|
};
|
|
|
|
template <class Value>
|
|
using Map = std::unordered_map<RasterCacheKey, Value, Hash, Equal>;
|
|
|
|
private:
|
|
ID id_;
|
|
|
|
// ctm where only fractional (0-1) translations are preserved:
|
|
// matrix_ = ctm;
|
|
// matrix_[SkMatrix::kMTransX] = SkScalarFraction(ctm.getTranslateX());
|
|
// matrix_[SkMatrix::kMTransY] = SkScalarFraction(ctm.getTranslateY());
|
|
SkMatrix matrix_;
|
|
};
|
|
|
|
// The ID is the uint32_t picture uniqueID
|
|
using PictureRasterCacheKey = RasterCacheKey<uint32_t>;
|
|
|
|
class Layer;
|
|
|
|
// The ID is the uint64_t layer unique_id
|
|
using LayerRasterCacheKey = RasterCacheKey<uint64_t>;
|
|
|
|
} // namespace flutter
|
|
|
|
#endif // FLUTTER_FLOW_RASTER_CACHE_KEY_H_
|