This repository was archived by the owner on Feb 25, 2025. It is now read-only.
Add FlMessageCodec #18186
Merged
Add FlMessageCodec #18186
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
from
d4c0649 to
ebcc83f
Compare
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
from
ebcc83f to
424c111
Compare
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
from
424c111 to
5530dd7
Compare
This was referenced May 7, 2020
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
from
5530dd7 to
d1a9b4a
Compare
Contributor
stuartmorgan-g
left a comment
stuartmorgan-g
left a comment
There was a problem hiding this comment.
LGTM ignoring the FlValue parts. I'll hold of on approving until that part is finalized, but I don't think anything here will need to be changed.
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
2 times, most recently
from
8b95dea to
e0ef9af
Compare
stuartmorgan-g
approved these changes
May 7, 2020
Contributor
stuartmorgan-g
left a comment
stuartmorgan-g
left a comment
There was a problem hiding this comment.
Since FlValue is set, and the requested changes here are minor, approving so you don't need to wait on another round of review from me once it's rebased and the nits are addressed.
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
3 times, most recently
from
418c4c2 to
9aa5666
Compare
Abstract class for binary message encoding/decoding that matches MessageCodec in the Flutter services library.
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
from
9aa5666 to
eb22baf
Compare
robert-ancell
force-pushed
the
linux-shell-fl-codec
branch
from
eb22baf to
7501617
Compare
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
This was referenced May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 8, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 9, 2020
This was referenced May 9, 2020
engine-flutter-autoroll
added a commit
to engine-flutter-autoroll/flutter
that referenced
this pull request
May 9, 2020
GaryQian
pushed a commit
to flutter/flutter
that referenced
this pull request
May 11, 2020
* 9ea2db5 Add FlMessageCodec, FlBinaryCodec, FlStringCodec (flutter/engine#18186) * f422757 Roll src/fuchsia/sdk/linux from MhpFP... to c1q_S... (flutter/engine#18222) * 2ab918c Roll src/third_party/skia a14084ba1b41..8f6c3ed7c7be (1 commits) (flutter/engine#18223) * 40167b6 Make robolectric tests run against SDK 29 by default (flutter/engine#17996) * e5b0db6 Roll src/third_party/skia 8f6c3ed7c7be..b55372444d1b (4 commits) (flutter/engine#18224) * ed08c3e Roll src/fuchsia/sdk/mac from 4MCVP... to T5tT0... (flutter/engine#18228) * 6767517 Roll src/third_party/skia b55372444d1b..ac09f7cd7a28 (2 commits) (flutter/engine#18229) * 983de2c Remove pipeline in favor of layer tree holder (flutter/engine#17688) * a1218dd Roll src/third_party/skia ac09f7cd7a28..c683912173bb (2 commits) (flutter/engine#18230) * 68bf137 skip painting clipped out pictures (flutter/engine#18204) * 7035255 make compiler worker count configurable (flutter/engine#17616) * c8ff03c Publish validation layer deps as part of the fuchsia artifacts (flutter/engine#18214) * 576f0e1 Roll src/third_party/skia c683912173bb..7359165e660c (1 commits) (flutter/engine#18234) * 1b3b4ec skip font loading tests for safari (flutter/engine#18232) * 9319d7c Roll src/third_party/skia 7359165e660c..6913d1bb1d7a (1 commits) (flutter/engine#18237) * 1b56f35 Roll src/third_party/dart 617bc54b715d..2a14a62112e6 (30 commits) (flutter/engine#18239) * ff6942f Add fontFeatures and decorationThickness to textstyle (flutter/engine#18235) * 4418ce8 Revert "Remove pipeline in favor of layer tree holder (#17688)" (flutter/engine#18242) * 9d8daf2 Roll src/third_party/skia 6913d1bb1d7a..bf1904fd4898 (3 commits) (flutter/engine#18243) * Updated bin/internal/fuchsia-linux.version * Updated bin/internal/fuchsia-mac.version
wandyers
pushed a commit
to wandyers/engine
that referenced
this pull request
May 23, 2020
Classs for binary message encoding/decoding that matches the ones in the Flutter services library.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Builds on #18185