Skip to content

Conversation

@dragostis
Copy link
Contributor

This fixes a naming inconsistency. Also fixes #4713.

@Hixie
Copy link
Contributor

Hixie commented Sep 15, 2016

there's also some comments that say "tool bar" in constants.dart and text_selection.dart that we should fix

@abarth
Copy link
Contributor

abarth commented Sep 16, 2016

LGTM w/ @Hixie 's comments addressed.

@abarth
Copy link
Contributor

abarth commented Sep 16, 2016

Also, you might want to change the title of your commit. :)

@dragostis dragostis changed the title Renamed toolbar to toolBar. Fixed capitalization of toolbar. Sep 16, 2016
@dragostis dragostis merged commit b1a7720 into flutter:master Sep 16, 2016
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 30, 2018
flutter/engine@af855e0...46b7793

git log af855e0..46b7793 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 30, 2018
flutter/engine@af855e0...9336082

git log af855e0..9336082 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 30, 2018
flutter/engine@af855e0...659e49e

git log af855e0..659e49e --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 30, 2018
flutter/engine@af855e0...cd85329

git log af855e0..cd85329 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 30, 2018
flutter/engine@af855e0...10effc7

git log af855e0..10effc7 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 [email protected] Replace travis with cirrus (flutter#5899)
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 30, 2018
flutter/engine@af855e0...430ca29

git log af855e0..430ca29 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 62501c12c83e..f7dcd76c552a (5 commits) (flutter#5906)
2018-07-30 [email protected] Replace travis with cirrus (flutter#5899)
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 31, 2018
flutter/engine@af855e0...5b09e14

git log af855e0..5b09e14 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 946a162a5239..0bf5408286ea (1 commits) (flutter#5909)
2018-07-30 [email protected] Add word movement granularities to android (flutter#5902)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia f7dcd76c552a..946a162a5239 (3 commits) (flutter#5908)
2018-07-30 [email protected] Remove the &flutter#34;VSYNC&flutter#34; trace event on Fuchsia (flutter#5907)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 62501c12c83e..f7dcd76c552a (5 commits) (flutter#5906)
2018-07-30 [email protected] Replace travis with cirrus (flutter#5899)
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 31, 2018
flutter/engine@af855e0...3e6b681

git log af855e0..3e6b681 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-31 37626415&flutter#43;[email protected] Roll src/third_party/skia 0bf5408286ea..74c6ed3d1f17 (1 commits) (flutter#5910)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 946a162a5239..0bf5408286ea (1 commits) (flutter#5909)
2018-07-30 [email protected] Add word movement granularities to android (flutter#5902)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia f7dcd76c552a..946a162a5239 (3 commits) (flutter#5908)
2018-07-30 [email protected] Remove the &flutter#34;VSYNC&flutter#34; trace event on Fuchsia (flutter#5907)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 62501c12c83e..f7dcd76c552a (5 commits) (flutter#5906)
2018-07-30 [email protected] Replace travis with cirrus (flutter#5899)
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 31, 2018
flutter/engine@af855e0...d06e9e8

git log af855e0..d06e9e8 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-31 [email protected] Deprecate -[FlutterDartProject initFromDefaultSourceForConfiguration] (flutter#18886) (flutter#5858)
2018-07-31 37626415&flutter#43;[email protected] Roll src/third_party/skia 0bf5408286ea..74c6ed3d1f17 (1 commits) (flutter#5910)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 946a162a5239..0bf5408286ea (1 commits) (flutter#5909)
2018-07-30 [email protected] Add word movement granularities to android (flutter#5902)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia f7dcd76c552a..946a162a5239 (3 commits) (flutter#5908)
2018-07-30 [email protected] Remove the &flutter#34;VSYNC&flutter#34; trace event on Fuchsia (flutter#5907)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 62501c12c83e..f7dcd76c552a (5 commits) (flutter#5906)
2018-07-30 [email protected] Replace travis with cirrus (flutter#5899)
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
rmistry pushed a commit to rmistry/flutter that referenced this pull request Jul 31, 2018
flutter/engine@af855e0...91ff524

git log af855e0..91ff524 --date=short --no-merges --format='%%ad %%ae %%s'
2018-07-31 37626415&flutter#43;[email protected] Roll src/third_party/skia 74c6ed3d1f17..1935aa3d27cd (11 commits) (flutter#5911)
2018-07-31 [email protected] Deprecate -[FlutterDartProject initFromDefaultSourceForConfiguration] (flutter#18886) (flutter#5858)
2018-07-31 37626415&flutter#43;[email protected] Roll src/third_party/skia 0bf5408286ea..74c6ed3d1f17 (1 commits) (flutter#5910)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 946a162a5239..0bf5408286ea (1 commits) (flutter#5909)
2018-07-30 [email protected] Add word movement granularities to android (flutter#5902)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia f7dcd76c552a..946a162a5239 (3 commits) (flutter#5908)
2018-07-30 [email protected] Remove the &flutter#34;VSYNC&flutter#34; trace event on Fuchsia (flutter#5907)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 62501c12c83e..f7dcd76c552a (5 commits) (flutter#5906)
2018-07-30 [email protected] Replace travis with cirrus (flutter#5899)
2018-07-30 [email protected] Use container and GKE to accelerate build (flutter#5897)
2018-07-30 [email protected] Add docker container generation scripts (flutter#5898)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia c06b8a4e0ca3..62501c12c83e (11 commits) (flutter#5905)
2018-07-30 [email protected] Clear the EGL context only if our context is currently active (flutter#5895)
2018-07-30 [email protected] Return proper JSON-RPC error responses from service protocol failures (flutter#5889)
2018-07-30 37626415&flutter#43;[email protected] Roll src/third_party/skia 227424b57d28..c06b8a4e0ca3 (22 commits) (flutter#5904)

The AutoRoll server is located here: http://localhost:8000

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC&flutter#39;d on the roll, and stop the roller if necessary.
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Be consistent about toolbar and toolBar

3 participants