- 04 Jan, 2018 40 commits
-
-
Dominic Battre authored
TBR=paulirish@chromium.org Bug: 798957 Change-Id: Ie27c65a17125e987416ef4aa89ecc9ff03a9d30a Reviewed-on: https://chromium-review.googlesource.com/850353 Commit-Queue: Xida Chen <xidachen@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#527001}
-
Yoav Weiss authored
When the server sends down a pushed stream and later on cancels it, there's currently a race condition. If the pushed stream was not adopted a request is sent for it, but if it was, it currently fails in a user-visible way. This CL fixes that by retrying the request if a pushed stream was adopted and later cancelled. Bug: 798508 Change-Id: Ia580cd4bad355cf4c5738e03db8dbfcf80716ae9 Reviewed-on: https://chromium-review.googlesource.com/848776Reviewed-by:
Bence Béky <bnc@chromium.org> Commit-Queue: Yoav Weiss <yoav@yoav.ws> Cr-Commit-Position: refs/heads/master@{#527000}
-
James Cook authored
The window service runs in the browser process for --mus and in the ash_and_ui process for --mash. Use the support_lazy_serialization build flag for the window service mojom targets so that in-process mojo method calls avoid doing serialization for IPC. This should improve event dispatch performance, as the mojo call becomes equivalent to a PostTask. This causes a known 130KB code size increase on 64-bit Intel builds, so the feature is enabled on Chrome OS only. Bug: 798800 Test: bots Change-Id: Ia488f95e6dd35ce1947df7117e623cd222786fb0 Reviewed-on: https://chromium-review.googlesource.com/849657Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#526999}
-
James Cook authored
Grouping the ash service and ui service into the same process broke image cursors. Fix it by providing the necessary task runner for cursor resource loading. Make MashServiceRegistry into a class so it can hold the cursor set. Also refactor ui::Service InitParams to make it more clear when it is running in-process vs. out-of-process. Bug: 722527 Test: Run chrome --mash, hovering NTP tiles has hand cursor Change-Id: Ie2b3fc9d523af9f06e16a1b6137f8481182169de Reviewed-on: https://chromium-review.googlesource.com/832996Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#526998}
-
David Roger authored
When Mirror is enabled, authentication errors are reported for all accounts, but with Dice, only the main account is monitored. This CL adds a AccountMode parameter and a SetMainAccountID() function to SigninErrorController. SigninManagerBase now knows about the signin error controller and sets the main account ID. To make the CL easier to cherry-pick, the signin_error_controller parameter in SigninManagerBase is allowed to be null, and the fake signin manager passes nullptr by default. Bug: 793321 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ie88771368a5d8555a334bfc640a331258a82bc17 Reviewed-on: https://chromium-review.googlesource.com/836555Reviewed-by:
Sky Malice <skym@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#526997}
-
Yuqian Li authored
Change-Id: Ia08c56f2d1337ee23c7fd9423d021d93f78dbbbb Reviewed-on: https://chromium-review.googlesource.com/848089 Commit-Queue: Yuqian Li <liyuqian@chromium.org> Reviewed-by:
Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#526996}
-
Rune Lillesveen authored
Sibling invalidation sets may be (re-)scheduled on shadow roots on dom changes. If the removed node was the one that needed style invalidation, we may end up with a style-dirty shadow root with no style-dirty children. There is a DCHECK in RecalcDescendantStyles which triggers if we call it unnecessarily. We called it unconditionally in ShadowRoot. Discovered while minimizing the fuzzer case for the mentioned Bug. Bug: 798918 Change-Id: I4f59b1a77af6a89ed304abd88d7f8cca84775127 Reviewed-on: https://chromium-review.googlesource.com/850395Reviewed-by:
Eric Willigers <ericwilligers@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#526995}
-
Pramod B S authored
Required [Exposed] to get rid of [PrimaryGlobal] Spec: https://github.com/heycam/webidl/issues/365 Bug: 792432 Change-Id: I48abf9f25dc5968da5680d8e95598270199ba812 Reviewed-on: https://chromium-review.googlesource.com/848465Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#526994}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/45da0f2d84c9..e83a915ebd3a $ git log 45da0f2d8..e83a915eb --date=short --no-merges --format='%ad %ae %s' 2018-01-03 dsinclair Fold CXFA_FontData into CXFA_Font Created with: roll-dep src/third_party/pdfium The AutoRoll server is located here: https://pdfium-roll.skia.org 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'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: I542fb99d189bc604dbec1c0202e879c6aa2e9c4f Reviewed-on: https://chromium-review.googlesource.com/850433 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#526993}
-
Gauthier Ambard authored
The Toolbar protocol should be defined in its own file, not in the LegacyToolbarCoordinator header. Bug: 788705 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ifc3e70d8ebee93eb13c85f1fe97f31dc2419a7e8 Reviewed-on: https://chromium-review.googlesource.com/849007 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#526992}
-
Gauthier Ambard authored
This CL splits the use of the legacy method of the LegacyToolbarCoordinator and regroup them in a specific protocol. It allows a clearer interface description of the toolbar. Bug: 798418 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ibde8c9e2a819be056de801ff6560922b22ec59d3 Reviewed-on: https://chromium-review.googlesource.com/848855Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#526991}
-
chromeos-commit-bot@chromium.org authored
TBR=chrome-os-gardeners@google.com Change-Id: Id5bd2107c071a3f63eea686b66503eb45da52941 Reviewed-on: https://chromium-review.googlesource.com/850734Reviewed-by:
ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#526990}
-
John Chen authored
Re-generate ChromeDriver mobile device list to pick up some new devices. Bug: chromedriver:2207 Change-Id: I14e633b0cc7390b56cdb8d9d7a241aea8803e93d Reviewed-on: https://chromium-review.googlesource.com/847994Reviewed-by:
Jonathon Kereliuk <kereliuk@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#526989}
-
webrtc-autoroll@chromium.org authored
https://webrtc.googlesource.com/src.git/+log/8e126fca476a..3b116ad3d868 $ git log 8e126fca4..3b116ad3d --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=None The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org 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'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I4c898304ec881ae8468b7b335d4c761204bc9674 Reviewed-on: https://chromium-review.googlesource.com/850532Reviewed-by:
WebRTC Roll Bot <webrtc-autoroll@chromium.org> Commit-Queue: WebRTC Roll Bot <webrtc-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#526988}
-
skia-deps-roller@chromium.org authored
https://skia.googlesource.com/skia.git/+log/f4f3f0fb29d4..a3cf3659ed2c $ git log f4f3f0fb2..a3cf3659e --date=short --no-merges --format='%ad %ae %s' 2018-01-03 bsalomon Skip calling GrBlurUtils from SkGpuDevice::drawPath when possible. Created with: roll-dep src/third_party/skia The AutoRoll server is located here: https://autoroll.skia.org 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'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel TBR=jcgregorio@chromium.org Change-Id: If0586dd29aae17afddb96cf427576fb4d9351428 Reviewed-on: https://chromium-review.googlesource.com/850572Reviewed-by:
Skia Deps Roller <skia-deps-roller@chromium.org> Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#526987}
-
Helen Li authored
This CL applies two fixes to make run.py script run again. (1) The test data file needs to be under /chromium_test_root subdirectory in external storage. (2) Add --browser=any flag so telemetry doesn't complain. Bug: 763998 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester Change-Id: I445db215d3171f0167dd6e3741ed33f2747a0872 Reviewed-on: https://chromium-review.googlesource.com/849074Reviewed-by:
Paul Jensen <pauljensen@chromium.org> Commit-Queue: Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#526986}
-
Xida Chen authored
This reverts commit b791621b. Reason for revert: <INSERT REASONING HERE> There is another one here that does the same thing: https://chromium-review.googlesource.com/c/chromium/src/+/850353 Original change's description: > Mark audits2-successful-run.js timeout on Linux MSAN > > TBR=battre@chromium.org > NOTRY=true > > Bug: 680917 > Change-Id: Ib7d69027cd72f6a5c56cb4d6a637417f901f4d3d > Reviewed-on: https://chromium-review.googlesource.com/850712 > Reviewed-by: Xida Chen <xidachen@chromium.org> > Commit-Queue: Xida Chen <xidachen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#526983} TBR=battre@chromium.org,xidachen@chromium.org Change-Id: Ida54871284e8276d1ad6729969db765ec1d6c6b2 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 680917 Reviewed-on: https://chromium-review.googlesource.com/850533Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#526985}
-
Jochen Eisinger authored
BUG=798676 R=clamy@chromium.org Change-Id: Ie0999280241dd4da438be6b6be792c5472c3c672 Reviewed-on: https://chromium-review.googlesource.com/849997Reviewed-by:
Camille Lamy <clamy@chromium.org> Commit-Queue: Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#526984}
-
Xida Chen authored
TBR=battre@chromium.org NOTRY=true Bug: 680917 Change-Id: Ib7d69027cd72f6a5c56cb4d6a637417f901f4d3d Reviewed-on: https://chromium-review.googlesource.com/850712Reviewed-by:
Xida Chen <xidachen@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#526983}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/59d313a7..e1e249b9 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: I045d982d7989bb62d888062632338291c6eaf48f Reviewed-on: https://chromium-review.googlesource.com/850053Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#526982}
-
Hwanseung Lee authored
Instead of some_vector.push_back(std::make_pair(a, b)) we can use some_vector.emplace_back(a, b) Bug: 798758 Change-Id: Ib470b5b809a255933706fb03db3cf9ce81fa9fef Reviewed-on: https://chromium-review.googlesource.com/848485 Commit-Queue: Mathieu Perreault <mathp@chromium.org> Reviewed-by:
Mathieu Perreault <mathp@chromium.org> Cr-Commit-Position: refs/heads/master@{#526981}
-
catapult-deps-roller@chromium.org authored
https://chromium.googlesource.com/catapult.git/+log/2de84c0d21c2..aadff7def3d1 $ git log 2de84c0d2..aadff7def --date=short --no-merges --format='%ad %ae %s' 2018-01-04 perezju [Telemetry] Cleanup cros_browser_backend._GetDevToolsClientConfig Created with: roll-dep src/third_party/catapult The AutoRoll server is located here: https://catapult-roll.skia.org 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'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: I11979ce94dcc25f97662923f6b99b2369adaa594 Reviewed-on: https://chromium-review.googlesource.com/850432 Reviewed-by: <catapult-deps-roller@chromium.org> Commit-Queue: <catapult-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#526980}
-
Peter Wen authored
Canary has updated to alpha 4. BUG=620034 Change-Id: Ic8ded85e9a06dd7a4a610145ade1df2a027a60e9 Reviewed-on: https://chromium-review.googlesource.com/847938Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#526979}
-
Gauthier Ambard authored
When the animation to fade buttons out completes, the buttons have a transformation translating them. This translation is canceled in the contract animation. But if the contract animation doesn't occur, the hidden buttons keep this translation, making them visually misplaced. This CL fixes it by resetting the position of all elements in the completion of the animator. Bug: 798981 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: If84ab49f9fb1b6971f5901a741004dd4c623dcaf Reviewed-on: https://chromium-review.googlesource.com/849952Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#526978}
-
Julien Isorce authored
This allows to re-use generic GLImageTest templates in tests that requires EGL,GLES2. CQ-DEPEND=CL:817563 gl_unittests --gtest_filter=*GLImage* ozone_gl_unittests Bug: 785201 Tests: Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I3ce122b69bef136fdf8c544f0b432d438c528f07 Reviewed-on: https://chromium-review.googlesource.com/832226 Commit-Queue: Julien Isorce <julien.isorce@chromium.org> Reviewed-by:
Michael Spang <spang@chromium.org> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Reviewed-by:
Ian Vollick <vollick@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#526977}
-
Greg Thompson authored
BUG=NONE R=pastarmovj@chromium.org Change-Id: I29b2b3d2b0ecce290f858043d29e8a82b1b6d3a9 Reviewed-on: https://chromium-review.googlesource.com/849996Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Commit-Queue: Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#526976}
-
Boris Sazonov authored
This CL removes unused Context parameter from FirstRunPage.shouldSkipPageOnCreate. Bug: 783344 Change-Id: Ia9acd543dfac368c822e6d1d01ca0f5c42c51856 Reviewed-on: https://chromium-review.googlesource.com/849214 Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Reviewed-by:
Ted Choc (back but slow, ping me) <tedchoc@chromium.org> Cr-Commit-Position: refs/heads/master@{#526975}
-
Dominic Battre authored
TBR=rouslan@chromium.org Bug: 799028 Change-Id: I12904a56860ce6e185f6f9c20d2ebb48b43ab54e Reviewed-on: https://chromium-review.googlesource.com/850552Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#526974}
-
Dominic Battre authored
TBR=nasko@chromium.org,nick@chromium.org,rdevlin.cronin@chromium.org Bug: 799022 Change-Id: I7db594de0aaaecda8d3d7f0bcd8cf8ea16abb3c9 Reviewed-on: https://chromium-review.googlesource.com/850393Reviewed-by:
Dominic Battré <battre@chromium.org> Commit-Queue: Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#526973}
-
Vadim Petrov authored
Bug: 797280 Change-Id: Ie05b919e7e5f1aab2ebfc54c5f1dca50b4a18a23 Reviewed-on: https://chromium-review.googlesource.com/842864Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Vadim Petrov <lof84@yandex-team.ru> Cr-Commit-Position: refs/heads/master@{#526972}
-
Rune Lillesveen authored
Intent-to-ship thread: https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/XzdNrEvn4Qk/650SpqXTBAAJ Bug: 657748 Change-Id: I01a7804d2214ac19fb8f58e1f68c23957cf071f7 Reviewed-on: https://chromium-review.googlesource.com/848920Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#526971}
-
Sriram authored
Upstream the below tests from media/track/ to wpt 1. track-large-timestamp.html 2. track-load-error-readyState.html 3. track-load-from-element-readyState.html 4. track-load-from-src-readyState.html 5. track-mode-disabled-crash.html 6. track-mode-not-changed-by-new-track.html 7. track-mode-triggers-loading.html 8. track-mode.html Bug: 782555 Change-Id: Ic76b8f4d8010650b72c031dec3715d274a02a4ff Reviewed-on: https://chromium-review.googlesource.com/850155Reviewed-by:
Fredrik Söderquist <fs@opera.com> Commit-Queue: srirama chandra sekhar <srirama.m@samsung.com> Cr-Commit-Position: refs/heads/master@{#526970}
-
Rune Lillesveen authored
When a table part box is removed from the layout tree, we may end up with two anonymous layout part siblings which needs to be merged. We move the children of the latter into the former. When children of the latter are themselves anonymous, skip down the layout tree until we find non-anonymous boxes. This will make sure we merge multiple levels of anonymous boxes although it means we will have to recreate the anonymous ancestors of the moved box when the preceding box we are moving it to does not have anonymous descendants. Bug: 181374 Change-Id: I8a14cec2e036fb4e5b937f89c4a35411dcc98785 Reviewed-on: https://chromium-review.googlesource.com/846861Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#526969}
-
stkhapugin@chromium.org authored
Now that everything in ios/ is built with ARC, it's possible to use __weak everywhere instead of unsafe_unretained, except for where the API dictates otherwise, especially: 1. countByEnumeratingWithState:objects:count: 2. getArgument:atIndex: Bug: 717540 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I700f0ce62f7fd5801c4ed2e9bbfed81065b0bf0f Reviewed-on: https://chromium-review.googlesource.com/845686 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#526968}
-
Dominic Battre authored
TBR=tetsui@chromium.org,hirono@chromium.org Bug: 798772 Change-Id: Ib5190f6f5bd9e1aafbb83d06bc0c5cc07ef4a320 Reviewed-on: https://chromium-review.googlesource.com/850352 Commit-Queue: Dominic Battré <battre@chromium.org> Reviewed-by:
Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#526967}
-
Miguel Casas authored
This CL reorders the declarations of VaapiWrapper methods to follow their definitions [1]. No new code is intended, just moved around. [1] https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md#code-formatting Bug: Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I4c546341f5e26c026c50513468dfa79d77cdfe72 Reviewed-on: https://chromium-review.googlesource.com/848650Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#526966}
-
David Bokan authored
Since telemetry ref-builds have now moved past M63 when this was added, and all dependencies have been updated, we can now remove this flag. Bug: 610021 Change-Id: Ic3c4bf9d7ddc5019e18d187f352f8f477b1f0814 Reviewed-on: https://chromium-review.googlesource.com/846529Reviewed-by:
Eric Karl <ericrk@chromium.org> Commit-Queue: David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#526965}
-
Dominic Battre authored
If a request is redirected from a Google domain to a non-googel domain, the variation header (X-Client-Data) should not be forwarded to the new server. This is implemented by stripping the header in the ChromeNetworkDelegate when a redirect happens. Bug: 794644 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Ie11021509997f03ba886f7ed370a93ecba00519e Reviewed-on: https://chromium-review.googlesource.com/836869Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#526964}
-
Elly Fong-Jones authored
In Harmony, the distance between these two elements is supposed to be 3/4 of a layout unit instead of one layout unit, so: 1) Add DISTANCE_BUTTON_IMAGE_LABEL_PADDING to DistanceMetrics 2) Give it the same default value as DISTANCE_RELATED_CONTROL_HORIZONTAL 3) Add a Harmony value for the new DistanceMetric Bug: 794480 Change-Id: Ib7106612dd2c923386cdbf30219ff37180acdedd Reviewed-on: https://chromium-review.googlesource.com/848533Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#526963}
-
Patrik Höglund authored
This dependency is there because rtc_video_decoder_factory.h and a handful other files include video_codec_interface.h. This header file was not tracked, so this dependency could fly under the radar. Once this is done and landed upstream, I plan to move the header to a smaller more focused target; I will update this dependency then. Bug: webrtc:7620 Change-Id: I995b740d2d2f796c5744c9f86194cd56949e83d3 Reviewed-on: https://chromium-review.googlesource.com/847480Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Patrik Höglund <phoglund@chromium.org> Cr-Commit-Position: refs/heads/master@{#526962}
-