- 03 May, 2018 40 commits
-
-
Mike Klein authored
This will let us keep skcms `gn analyze` compatible more easily. Change-Id: I8d3f1134945859c112862cf7b6066ca6f2af812c Reviewed-on: https://chromium-review.googlesource.com/1039845 Commit-Queue: Mike Klein <mtklein@chromium.org> Reviewed-by:
Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#555780}
-
https://chromium.googlesource.com/chromiumos/chromite.git/+log/5295eaac6d22..8642bcc3248f $ git log 5295eaac6..8642bcc32 --date=short --no-merges --format='%ad %ae %s' 2018-05-01 bmgordon cbuildbot_launch: Remove old state handling Created with: roll-dep src/third_party/chromite BUG=chromium:829665 The AutoRoll server is located here: https://chromite-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. TBR=chrome-os-gardeners@chromium.org Change-Id: I657a9225a76249b9e4a18304f426bc5bee42a305 Reviewed-on: https://chromium-review.googlesource.com/1042378Reviewed-by:
Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#555779}
-
Theresa authored
Add a render test that checks layout of: - peeking bar - card while its thumbnail is loading - card after the thumbnail has loaded - card with offline badge - full sheet - full sheet scrolled BUG=835450 Change-Id: I8860d9eca6fd7f54e794b08076749cae0f3ecbe1 Reviewed-on: https://chromium-review.googlesource.com/1039876 Commit-Queue: Theresa <twellington@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Cr-Commit-Position: refs/heads/master@{#555778}
-
Wez authored
Previously WaitUntilEnded() would not install an on-error closure, and so would fail to exit on pipeline errors, causing the test to hang rather than fail. WaitUntilEnded() now waits for either end or error, and EXPECTs a PIPELINE_OK status before returning. Also tweaked the naming of RunUntilIdle* to RunUntilQuit to avoid confusion with the RunLoop::RunIntilIdle() API behaviour. Bug: 836785 Change-Id: Idcf9bbec5ac0d7d78e2ba0530eaed0748a1626c8 Reviewed-on: https://chromium-review.googlesource.com/1041807Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#555777}
-
My Name authored
This reverts commit e02bb35e. Why? Previous-step cl/193316519 is rolled back as this feature breaks working sites, and further refinement is necessary before it should be enabled by default. See https://crbug.com/729954#c26 onwards. Bug: 729954,837771 Change-Id: I806364dc9a5812a0e57f76f4a3db02e6803a8d5f Reviewed-on: https://chromium-review.googlesource.com/1042535Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Ami Fischman <fischman@chromium.org> Cr-Commit-Position: refs/heads/master@{#555776}
-
Antoine Labour authored
Bug: None Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I502f8f0c154fcc319cd3c5170cee53d9eb940f32 Reviewed-on: https://chromium-review.googlesource.com/1040847Reviewed-by:
weiliangc <weiliangc@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#555775}
-
Kevin McNee authored
Versions /1 and /2 (viz hit testing) are flaky, but /0 is fine, so we'll keep /0 enabled. Bug: 838835 Change-Id: If535c37a49d8f251283712f7b2e280ad60efebd9 Reviewed-on: https://chromium-review.googlesource.com/1042487Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Kevin McNee <mcnee@chromium.org> Cr-Commit-Position: refs/heads/master@{#555774}
-
Wez authored
- Remove |enable_webrtc=false| from the Cast Audio bot configuration. - Set |enable_webrtc=true| by default, rather than based on whether |is_cast_audio_only| is set. This requires a roll of OpenH264 to pick up changes that fix it not to use pthreads APIs that are not present under Fuchsia. Bug: 800653 Change-Id: Ieaa5b10bd64aab421db8163dd260b1283068d362 Reviewed-on: https://chromium-review.googlesource.com/1039766Reviewed-by:
Henrik Boström <hbos@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
Stephen Lanham <slan@chromium.org> Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#555773}
-
Sky Malice authored
Bug: 838609 Change-Id: Idd7f0043064894b414565d2d4e4955658b9cfccb Reviewed-on: https://chromium-review.googlesource.com/1041069Reviewed-by:
Patrick Noland <pnoland@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#555772}
-
David Roger authored
When signin cookies are "cleared on exit" through the Chrome settings, Dice would rebuild them on next startup, which defeats the point of that cookie settings. To fix this, tokens are not loaded when signin cookies are cleared on exit. Instead, the tokens are revoked on the server, and deleted from the database. The secondary accounts are completely removed from Chrome, but the primary account is kept in authentication error state, so that Sync can display an error. TBR=treib, eugenebut Bug: 836862 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I5a72b3a173242494ad1c996765c838bb9a7f09f1 Reviewed-on: https://chromium-review.googlesource.com/1042392 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#555771}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/e043a8b3..1f462965 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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: Ic0de3034053e0f7368ba535b770ae0bd2e92bfa8 Reviewed-on: https://chromium-review.googlesource.com/1042211Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#555770}
-
Matt Menke authored
Also adds an integration test for path stripping, but not for quick check. Bug: 715695 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I45d37089d305c5a6683afbac762ef767b910fb0f Reviewed-on: https://chromium-review.googlesource.com/1028177 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#555769}
-
Qiang Xu authored
Changes: When empty results from Android, barrier closure immediately runs, we should first initialize items_ to avoid nullptr crash. Bug: 803291 Test: tested on device saw crash fixed Change-Id: I4fd3a7dc29a056ac2e7e22a5c535d6a38a69e171 Reviewed-on: https://chromium-review.googlesource.com/1041576Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Qiang Xu <warx@google.com> Cr-Commit-Position: refs/heads/master@{#555768}
-
Mostyn Bramley-Moore authored
CL https://chromium-review.googlesource.com/c/chromium/src/+/1019245 seems to have introduced a jumbo build failure: ./../../content/browser/renderer_host/media/media_devices_manager.cc:704:37: error: reference to overloaded function could not be resolved; did you mean to call it? ignore_group_id ? operator== : EqualDeviceAndGroupID)) { ^~~~~~~~~~ ../../content/common/media/media_devices.h:52:6: note: possible target for call bool operator==(const MediaDeviceInfo& first, const MediaDeviceInfo& second); ^ ../../content/public/common/page_state.h:63:13: note: possible target for call inline bool operator==(const PageState& a, const PageState& b) { ^ Let's move page_state.h's ==/!= operators out of the global scope, to avoid this. TBR=avi@chromium.org Bug: 834281 Change-Id: I27f0493ee2d2957b8318c969edd5615ec4dab687 Reviewed-on: https://chromium-review.googlesource.com/1042308 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#555767}
-
Alexandr Ilin authored
The old predictor is automatically disabled when the new one is enabled. Change-Id: Ia5c1a79dbeb0e096720fc3d50da3dc707b3761a6 Reviewed-on: https://chromium-review.googlesource.com/1042388Reviewed-by:
Egor Pasko <pasko@chromium.org> Commit-Queue: Alexandr Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#555766}
-
Xiaohan Wang authored
Most changes are in preparation for supporting MojoDecryptorService directly in MediaService. Bug: 806018 Test: Tests updated. Change-Id: I4ea5a6098ee62f1b1a2ecf4ada559e2825076768 Reviewed-on: https://chromium-review.googlesource.com/1041651 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Frank Liberato <liberato@chromium.org> Cr-Commit-Position: refs/heads/master@{#555765}
-
iOS Autoroller authored
TBR=smut Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ic5060153174b0258ee7505b8760a9d143afe489b Reviewed-on: https://chromium-review.googlesource.com/1042449 Commit-Queue: iOS Autoroller <ios-autoroller@chromium.org> Reviewed-by:
iOS Autoroller <ios-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#555764}
-
Max Moroz authored
Bug: 836663 Change-Id: Ia5c37b86624e4ff546130027297a167185f174a4 Reviewed-on: https://chromium-review.googlesource.com/1042374 Commit-Queue: Max Moroz <mmoroz@chromium.org> Reviewed-by:
Abhishek Arya <inferno@chromium.org> Reviewed-by:
Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#555763}
-
Wei Li authored
As Views based UI is undergoing new changes, that applies to MacViews browser as well, but diverges from Cocoa based browser. This CL solves that problem by creating a function for Cocoa browser only. This function can get Cocoa specific constants which won't be affected by Views side of changes. BUG=838630,838856 TEST=Pls make sure bookmark bar, icons and menus look ok for both Cocoa version and MacViews version on Mac. They should be same as before for other platforms. Change-Id: I73152c05cfac77eb6b24a4d6204acb78767c833d Reviewed-on: https://chromium-review.googlesource.com/1041141Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Wei Li <weili@chromium.org> Cr-Commit-Position: refs/heads/master@{#555762}
-
Scott Violet authored
There are no new files, this just splits sources into public and sources. This is in preparation to adding some implementation specific files. BUG=680698 TEST=build file change only Change-Id: I198f7961094909ea9015719efb4ec563b83abb7d Reviewed-on: https://chromium-review.googlesource.com/1041526Reviewed-by:
Jay Civelli <jcivelli@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#555761}
-
Kevin Bailey authored
Refs (or hashes) are useful for navigating, but not useful for comparing URLs. We compare URLs when deduping URLs in Omnibox matches, and when testing eligibility for switching to already open tabs. This CL changes GURLToStrippedGURL() to remove refs from the stripped version of destination URLs (only used for comparison) and when testing URLs for tab switching. We only remove refs when the input doesn't specify a ref. This is critical to proper deduping. Also added unit test. Bug: 780835 Change-Id: Ifef8d8b9d2efd4394b598342c13ef8aa439089ce Reviewed-on: https://chromium-review.googlesource.com/1025953 Commit-Queue: Kevin Bailey <krb@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#555760}
-
Guido Urdaneta authored
Bug: 834281 Change-Id: I5ef11a3343e5b534bee6e088a36fb3365bd218e8 Reviewed-on: https://chromium-review.googlesource.com/1019323 Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Harald Alvestrand <hta@chromium.org> Cr-Commit-Position: refs/heads/master@{#555759}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/4722a4135911..2f62de01d3a7 $ git log 4722a4135..2f62de01d --date=short --no-merges --format='%ad %ae %s' 2018-05-03 borenet [recipes] Mostly remove bot_update patch_ref hack 2018-05-02 khushalsagar fonts: Cleanup header guard and stray printfs. 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;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=brianosman@chromium.org Change-Id: Ib5863c6e9da4c25334bba511b39dae62a4c54c07 Reviewed-on: https://chromium-review.googlesource.com/1042490 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#555758}
-
https://chromium.googlesource.com/catapult.git/+log/978be7439d1e..a87e87c2a7f4 $ git log 978be7439..a87e87c2a --date=short --no-merges --format='%ad %ae %s' 2018-05-01 perezju [soundwave] Replace bugs.DataFrameFromApi with DataFrameFromJson 2018-05-03 perezju [soundwave] Store timeseries data in database 2018-05-03 wangqing-hf [MIPS64] Add MIPS64 d8 binary (version 5.7.492) 2018-05-03 wangqing-hf [MIPS64] Update the update_v8 script to support MIPS64. 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. TBR=sullivan@chromium.org Change-Id: I60e30aad7a212a48f02d21645dd079a1b3fcb0a1 Reviewed-on: https://chromium-review.googlesource.com/1042526Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#555757}
-
Sammie Quon authored
Allows user drop down menu to activatble so that it can be cycled through. Add a custom focus search so the drop down menu closes after all items have been cycled through. Migrate Closure to OnceClosure. Make drop down menu a transient child of system bubble. Test: manual Bug: 799401 Change-Id: I67af2bb3e58c88f6926ab65ecf565d69325d00b8 Reviewed-on: https://chromium-review.googlesource.com/1038656Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Sammie Quon <sammiequon@chromium.org> Cr-Commit-Position: refs/heads/master@{#555756}
-
Douglas Creager authored
We weren't including the `include-subdomains` field of each endpoint group in the data we were handing over to the net-internals page. (The net-internals code that reads this field is already there; we just weren't filling in a value.) Bug: 829353 Change-Id: Ifdff1d6e0baf802eb7f094571060b93c761936af Reviewed-on: https://chromium-review.googlesource.com/1019661Reviewed-by:
Julia Tuttle <juliatuttle@chromium.org> Commit-Queue: Douglas Creager <dcreager@google.com> Cr-Commit-Position: refs/heads/master@{#555755}
-
Koji Ishii authored
Following bot results are included. 5144 5146 5148 5149 5150 5151 5152 5154 5155 1 lines were removed and 0 lines were deflaked by consecutive results since 5100. TBR=eae@chromium.org, mstensho@chromium.org NOTRY=true Bug: 591099 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: Ib0614dfcb6f39c7e101920875079278f1bb795c7 Reviewed-on: https://chromium-review.googlesource.com/1040225 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#555754}
-
Roman Sorokin authored
Don't DCHECK on password being plain text in existing user controller. Make it if statement. BUG=chromium:839314 TEST=manual Change-Id: I7a649ddb922edd2105563cb2b4959b764d0383a4 Reviewed-on: https://chromium-review.googlesource.com/1041958 Commit-Queue: Roman Sorokin <rsorokin@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#555753}
-
Steve Kobes authored
These virtual test suites no longer exist. Bug: 823365 Change-Id: I62503d3b99f192f46e09acd6b8f38fad84e1ee9a Reviewed-on: https://chromium-review.googlesource.com/1042369Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#555752}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 4ff5b671. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/16757 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: jsbell@chromium.org, mek@chromium.org: external/wpt/FileAPI TBR=kereliuk No-Export: true Change-Id: I3a8ae73b98b0099bbc66b21fbffd1c46b01a7412 Reviewed-on: https://chromium-review.googlesource.com/1042207 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#555751}
-
https://pdfium.googlesource.com/pdfium.git/+log/6312bf94bb44..ccd942158992 $ git log 6312bf94b..ccd942158 --date=short --no-merges --format='%ad %ae %s' 2018-05-03 tsepez Add CPDF_Page::Extension::GetDocExtension() 2018-05-03 tsepez Make GetPDFDocument return a CPDF_Docment*, not void*. 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: I57a6b3d5a99ada684414265fb3d323869a4d13f9 Reviewed-on: https://chromium-review.googlesource.com/1042371 Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#555750}
-
Amirhossein Simjour authored
This is a reland of 480d1516. Patch was not responsible for bot failures. Original change's description: > Scroll support for hosted dialogs in VR > > Now that VR supports html <select>, we should send scroll events > to dialogs. > > Bug: 836292 > Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr > Change-Id: I45d44af2069e21bedd0662d3f26ab6c89ff82288 > Reviewed-on: https://chromium-review.googlesource.com/1028584 > Commit-Queue: Amirhossein Simjour <asimjour@chromium.org> > Reviewed-by: Ian Vollick <vollick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#555454} TBR=vollick@chromium.org, asimjour@chromium.org Bug: 836292 Change-Id: I8391daee7a1908273d5349106237e2177ad3c293 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr Reviewed-on: https://chromium-review.googlesource.com/1042546Reviewed-by:
Matthew Jones <mdjones@chromium.org> Commit-Queue: Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#555749}
-
Greg Thompson authored
The new text makes it more clear how the setting impacts Chrome OS vs Chrome browser. BUG=839308 R=atwilson@chromium.org Change-Id: I73de539820bd1fb7b822c88fad2eb6f2f2e8b3ab Reviewed-on: https://chromium-review.googlesource.com/1042306Reviewed-by:
Drew Wilson <atwilson@chromium.org> Commit-Queue: Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#555748}
-
Alexander Timin authored
R=alexclarke@chromium.org Change-Id: I77cf2e26643a862217d009cc7539fd838d3c1583 Reviewed-on: https://chromium-review.googlesource.com/1042189 Commit-Queue: Alexander Timin <altimin@chromium.org> Reviewed-by:
Alex Clarke <alexclarke@chromium.org> Cr-Commit-Position: refs/heads/master@{#555747}
-
Marc Treib authored
The version without the "virtual/video-surface-layer/" prefix already had it. Bug: 520736,838128 Change-Id: I815f420dd134143837d1602e795075176f4e4eb5 Reviewed-on: https://chromium-review.googlesource.com/1042565Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#555746}
-
Oskar Sundbom authored
Now that AudioInputDevice doesn't automatically redirect calls to the correct thread, ProcessedLocalAudioSource needs to do that itself. Bug: 838929 Change-Id: I55f1cf9c6446b90b464b229e4c9a621613bf9dfb Reviewed-on: https://chromium-review.googlesource.com/1041954 Commit-Queue: Oskar Sundbom <ossu@chromium.org> Reviewed-by:
Max Morin <maxmorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#555745}
-
Egor Pasko authored
I think digit@chromium.org is more preferable these days, so change the remaining cases of digit@google.com to the preferable way for extra consistency. TBR=estevenson@chromium.org NOTRY=true Bug: None Change-Id: Ie545282b334f056c7e72836caabec8539441ec3b Reviewed-on: https://chromium-review.googlesource.com/1030376 Commit-Queue: Egor Pasko <pasko@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Reviewed-by:
David Turner <digit@chromium.org> Cr-Commit-Position: refs/heads/master@{#555744}
-
Kevin McNee authored
If we pinch an element with a touch-action that prevents pinch in an OOPIF, we route the pinch events to the child so that the child's TouchActionFilter filters them. Due to crbug.com/771330, the TouchActionFilter could let the pinch events through. We'll filter such events in RenderWidgetHostViewChildFrame::FilterInputEvent to prevent them from being sent to the child renderer. Bug: 827182 Change-Id: I3782a2d135a008df271a73d79ecade25dcc9f1a5 Reviewed-on: https://chromium-review.googlesource.com/1040354Reviewed-by:
James MacLean <wjmaclean@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Kevin McNee <mcnee@chromium.org> Cr-Commit-Position: refs/heads/master@{#555743}
-
Marc Treib authored
TBR=pauljensen@chromium.org Bug: 838524 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I1ae71d8c07f91827e164b046d4c379aeee4125ac Reviewed-on: https://chromium-review.googlesource.com/1042397 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#555742}
-
Antonio Gomes authored
This method is creating a net::HttpRequestHeaders object, serializing it to a string, and then parsing that string to create a new net::HttpRequestHeaders object. CL just returns the net::HttpRequestHeaders object directly. No expected behavior change. TESTS=Covered by existing tests. Bug: None Change-Id: I356c5987e0dcf60e454efc08f93d25ec75954c11 Reviewed-on: https://chromium-review.googlesource.com/1040705 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#555741}
-