- 27 Jul, 2018 40 commits
-
-
jonross authored
While work on Android is on going, the remaining Viz failures on other platforms have been resolved. The filter file is no longer needed. This change removes it. TEST=viz_content_browsertests Bug: 794961 Change-Id: I46f8a7094aa10f72cf383607a8b7a68a19141fc8 Reviewed-on: https://chromium-review.googlesource.com/1150684Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#578606}
-
Colin Blundell authored
When clearing the primary account, three behaviors are possible: 1. Explicitly remove all accounts. 2. Explicitly keep all accounts. 3. Let the signin code internally decide between 1 and 2 based on its own default policy. The IdentityManager test APIS (IdentityTestEnvironment and identity_test_utils.h) currently support only 3. However, an upcoming conversion will require 1 and 2. This CL adds that support in to the APIS for clearing the primary account. Bug: 798699, 809923 Change-Id: Idb717836c843a14a5f6d33aab5d60d7e14ff4d1c Reviewed-on: https://chromium-review.googlesource.com/1149861 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#578605}
-
Keishi Hattori authored
Change-Id: I63eae05326391dcb2c62f571ac13650a6e2967b0 Reviewed-on: https://chromium-review.googlesource.com/1148210Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#578604}
-
fuchsia-sdk-chromium-autoroll authored
The AutoRoll server is located here: https://fuchsia-sdk-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=luci.chromium.try:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: If9283073cb46cbdb2b123b13695c1b7dc5b2c4a0 Reviewed-on: https://chromium-review.googlesource.com/1152887Reviewed-by:
Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#578603}
-
Denis Kuznetsov authored
Bug: 864455 Change-Id: Ia71d2bc2360b3db4100643e62210401c9b6c7a2b Reviewed-on: https://chromium-review.googlesource.com/1148451 Commit-Queue: Denis Kuznetsov <antrim@chromium.org> Reviewed-by:
Pavol Marko <pmarko@chromium.org> Cr-Commit-Position: refs/heads/master@{#578602}
-
Mythri Alle authored
We want to decouple the code cache from the HttpCache and recently added a separate code cache [1]. This cl adds required support to fetch the generated code from the code cache. There will be a followup cl that uses this API and fetches from the code cache. [1] chromium-review.googlesource.com/c/chromium/src/+/1120179 Bug: chromium:812168 Change-Id: I4972def83f05d2c6d5810f489704ffd9f1d1d536 Reviewed-on: https://chromium-review.googlesource.com/1150239 Commit-Queue: Mythri Alle <mythria@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#578601}
-
Tim Volodine authored
Rebaseline some interfaces/properties that have been exposed in Chrome and should also be exposed in Android WebView. In particular: - attribute @@toStringTag - method @@iterator - attribute @@unscopables - CSS* related interfaces/attributes BUG=867450 Change-Id: Iee23c9f75fb3031826dfe388920a1ae6b3a9722e Reviewed-on: https://chromium-review.googlesource.com/1150177Reviewed-by:
Richard Coles <torne@chromium.org> Commit-Queue: Tim Volodine <timvolodine@chromium.org> Cr-Commit-Position: refs/heads/master@{#578600}
-
Tatsuhisa Yamaguchi authored
Resolve errors on Chrome OS with opening Drive files whose file name contains a valid URL-encoding like "%41.pdf". Test: unit_tests --gtest_filter=ExternalFileURLUtilTest.*:FileManagerPathUtilConvertUrlTest.* Bug: 850035 Change-Id: If4768ae07f82d722065601192d557eaea53b3f12 Reviewed-on: https://chromium-review.googlesource.com/1151172 Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Cr-Commit-Position: refs/heads/master@{#578599}
-
Mihai Sardarescu authored
This reverts commit bfc33b7c. Reason for revert: The work for UKM is done in a separate CL, so this is no longer needed. Original change's description: > [unified-consent] Add shutdown information for the UrlKeyedDataCollectionConsentHelper. > > This CL adds methods that handle the shutdown state for the > UrlKeyedDataCollectionConsentHelper. This is set when the dependent sync > service is shut down. > > Bug: 867388 > > Change-Id: Id6f948bc0aa1a55ec2943590a5f77cd179b65e3f > Reviewed-on: https://chromium-review.googlesource.com/1148564 > Reviewed-by: Brian White <bcwhite@chromium.org> > Commit-Queue: Mihai Sardarescu <msarda@chromium.org> > Cr-Commit-Position: refs/heads/master@{#578242} TBR=msarda@chromium.org,bcwhite@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 867388 Change-Id: If3978f977afcb9ea1997bdde1b7220f56a271bbb Reviewed-on: https://chromium-review.googlesource.com/1152847Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#578598}
-
Helen Li authored
Rearrange ReceiveMore and SendMore, so that we check MOJO_RESULT_SHOULD_WAIT before !MOJO_RESULT_OK. This CL adds three unit tests that pass with or without the change. Bug: 721401 Change-Id: I6b132b2399f9091374d037d890d7ca98ba358445 Reviewed-on: https://chromium-review.googlesource.com/1150428Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#578597}
-
Oscar Johansson authored
When building using jumbo, files get merged and functions and variables with the same name may end up in the same namespace and conflict. This happens in: media/cast/net/cast_transport_impl.cc media/cast/net/udp_transport_impl.cc Since the functions and variables are identical they are moved to a common util file. Bug: 867350 Change-Id: I08747781d56b78e9a8783bfec870c7084183062a Reviewed-on: https://chromium-review.googlesource.com/1152738 Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org> Reviewed-by:
Chrome Cunningham (In Paris) <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#578596}
-
Pin-chih Lin authored
Currently VideoEncodeAccelerator::Initialize() does not take level into account which is a factor of AVC encoding. Instead, it always set the fixed level in VEA implementation. For upstream Android CTS, it requires AVC level setting and checks the level of output encoded stream is within configured one. To satisfy this we need to pass level on initialization. Since there is a bunch of parameters we decide to refactor VEA::Initialize() to use Config struct bundling all parameters as VideoDecodeAccelerator did. This CL only modifies VEA::Initialize() API to use Config struct as input to cover original parameters and output_level. Once this is approved to change, we will implement the utilization of output_level inside VEA. BUG=b:110811755 BUG=863321 TEST=build and deplot chrome on eve TEST=build and run video_encoder_accelerator_unittest on eve TEST=build media_unittests TEST=build content_unittests 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 Change-Id: I002169634a2b2d23a00178eb5015a14f4ddcf5dc Reviewed-on: https://chromium-review.googlesource.com/1130570 Commit-Queue: Pin-chih Lin <johnylin@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Pawel Osciak <posciak@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Reviewed-by:
Jamie Walch <jamiewalch@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Bill Budge <bbudge@chromium.org> Cr-Commit-Position: refs/heads/master@{#578595}
-
Oscar Johansson authored
When building using jumbo, files get merged and functions with the same name may end up in the same namespace. This happens with the functions InProgressFrameEncode defined in: media/cast/sender/external_video_encoder.cc media/cast/sender/h264_vt_encoder.cc This commit solves the issue by renaming the functions to something more file specific. Bug: 86750 Change-Id: I8f6fb84783539dcec4aae824bb971e982b4db5c2 Reviewed-on: https://chromium-review.googlesource.com/1152745 Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org> Reviewed-by:
Chrome Cunningham (In Paris) <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#578594}
-
Oscar Johansson authored
When building using jumbo, files gets merged. Files and variables with the same name may merge into the same namespace or scope and conflict. This happens in the files: media/filters/aom_video_decoder.cc media/filters/ffmpeg_video_decoder.cc media/filters/vpx_video_decoder.cc This commit solves the issue by renaming the function GetThreadCount to something more file specific. It also moves some constants to that function. Bug: 867350 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 Change-Id: Id57aa6cc6ac10acd0073eee032d0d04592ba87e4 Reviewed-on: https://chromium-review.googlesource.com/1151311 Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org> Reviewed-by:
Chrome Cunningham (In Paris) <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#578593}
-
Oscar Johansson authored
When building using jumbo, files merged and functions with the same name may end up in the same namespace/scope. This happens for the function OnError() that gets called by Bind(). OnError() is defined in: media/filters/audio_video_metadata_extractor.cc media/filters/media_file_checker.cc This commit solves the issue by renaming one of the functions to something more file specific. Bug: 867350 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 Change-Id: I1b762e4f1c4b11751df2302874af53106889d085 Reviewed-on: https://chromium-review.googlesource.com/1151316 Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org> Reviewed-by:
Chrome Cunningham (In Paris) <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#578592}
-
Oscar Johansson authored
In jumbo builds the compiler complain about not knowing about the template specialization of the function OnStartOfCodedFrameGroup, that is defined in source_buffer_stream.cc. To solve this issue a fordward declaration is done before usage. Bug: 867350 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 Change-Id: If090b848bc2ae78121620aba2349eac4c63d8a65 Reviewed-on: https://chromium-review.googlesource.com/1151313 Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org> Reviewed-by:
Chrome Cunningham (In Paris) <chcunningham@chromium.org> Cr-Commit-Position: refs/heads/master@{#578591}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/c4db8708c903..55b1e6168454 git log c4db8708c903..55b1e6168454 --date=short --no-merges --format='%ad %ae %s' 2018-07-27 angle-skia-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com Roll third_party/externals/angle2 dbd1612782f5..18f7e5033040 (1 commits) 2018-07-27 herb@google.com const all the things 2018-07-26 recipe-roller@chromium.org Roll recipe dependencies (trivial). 2018-07-26 ziadb@google.com SkAR Java: refactored project structure 2018-07-26 ziadb@google.com SkAR Java: changes to DrawManager/FingerPainting 2018-07-26 halcanary@google.com site: Documentation & Fiddle links on front page Created with: gclient setdep -r src/third_party/skia@55b1e6168454 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=benjaminwagner@chromium.org Change-Id: Ib2b45c09f9e0f4cdfa425c6591808cbcea5f1115 Reviewed-on: https://chromium-review.googlesource.com/1152613Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#578590}
-
Anita Woodruff authored
This prevents developers updating the channel id of a notification and forgetting to update the channel id that is logged. Bug: 772027 Change-Id: Ic32cead46f91a24d3164592c196b527fc21edcdd Reviewed-on: https://chromium-review.googlesource.com/1148321 Commit-Queue: Anita Woodruff <awdf@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Cr-Commit-Position: refs/heads/master@{#578589}
-
Daniel Bratell authored
Some osmesa code uses the "real" OSMesa API headers and those are not 100% compatible with the wrapper API headers normally used. That incompatibility means that they cannot compile in the same translation unit (jumbo builds). This splits the osmesa files into their own build target to avoid potential conflicts. Also changes the code to be consistent with how osmesa.h is included. Bug: 864986 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 Change-Id: Ifc96cf25f7b69f23f293e297ab59ce5192f1cd20 Reviewed-on: https://chromium-review.googlesource.com/1149874Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Daniel Bratell <bratell@opera.com> Cr-Commit-Position: refs/heads/master@{#578588}
-
Daniel Bratell authored
In jumbo compilation many cc files compile in the same translation unit and if they use the same local variable names, there will be compilation clashes. That happened in the automatically generated code with info about various problems in gpu drivers or the like. This patch makes sure necessary variables have unique names by adding a unique keyword for each set of data. Bug: 864986 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 Change-Id: Ief86248f2d17f123602c7e641c286d0550a85d89 Reviewed-on: https://chromium-review.googlesource.com/1151314 Commit-Queue: Daniel Bratell <bratell@opera.com> Reviewed-by:
Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#578587}
-
Jose Dapena Paz authored
IWYU: it fixes the build with libstdc++ after dc24395b "Retry initializing download db on failure" Bug: 819294 Change-Id: Id8a9bdf56f74d979d4e48747a3a6fbd9c51f59c6 Reviewed-on: https://chromium-review.googlesource.com/1150530 Commit-Queue: José Dapena Paz <jose.dapena@lge.com> Reviewed-by:
Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#578586}
-
Colin Blundell authored
Currently, AccountTracker::GoogleSigninSucceeded() calls AccountTracker::OnRefreshTokenAvailable() for each account that the token service knows about. However, this is conceptually somewhat confusing, as it's not the case that the refresh tokens for those accounts have suddenly become available. Moreover, an upcoming conversion of this class to use IdentityManager will make it even less suitable, as the corresponding observer callback passes information that AccountTracker won't have (and doesn't need) when invoking it manually. This CL changes AccountTracker::GoogleSigninSucceeded() to directly call through to AccountTracker::UpdateSigninState(), better reflecting the actual flow. Bug: 809923 Change-Id: Ic7673659cb9383d55eb8eda5cfb84a22f3a72214 Reviewed-on: https://chromium-review.googlesource.com/1150223 Commit-Queue: Colin Blundell <blundell@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#578585}
-
Takuto Ikuta authored
Currently MSVC builder cannot build v8 and pdfium with cl.exe. TBR=dpranke@chromium.org, thakis@chromium.org Change-Id: Ic7fc3bd57583f33f56a795e1534e5c04774c6612 Reviewed-on: https://chromium-review.googlesource.com/1152645Reviewed-by:
Fumitoshi Ukai <ukai@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#578584}
-
Vasilii Sukhanov authored
The spec specifies max width for labels in the password dropdown. Bug: 868020 Change-Id: I112c0866e7f0f0165b82f72bdc55c21a25c6c09f Reviewed-on: https://chromium-review.googlesource.com/1151637 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Tommy Martino <tmartino@chromium.org> Cr-Commit-Position: refs/heads/master@{#578583}
-
Vadym Doroshenko authored
This Cl copies and adapts creating pending credentials from PasswordFormManager to NewPasswordFormManager. No changes in logic. Creating of pending credentials is the step in implementing of saving in NewPasswordFormManager. Next steps will be to implement Save() and Update() methods. Bug: 831123 Change-Id: I2a7f24c85e46ecd1474f99cf26c9b5136fe2ae9d Reviewed-on: https://chromium-review.googlesource.com/1148393 Commit-Queue: Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Vaclav Brozek <vabr@chromium.org> Cr-Commit-Position: refs/heads/master@{#578582}
-
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/dbd1612782f5..604d873e1df5 git log dbd1612782f5..604d873e1df5 --date=short --no-merges --format='%ad %ae %s' 2018-07-27 oetuaho@nvidia.com Fix multiview combined with flat interpolation 2018-07-27 fjhenigman@chromium.org Vulkan: make DynamicBuffer reusable, always valid. Created with: gclient setdep -r src/third_party/angle@604d873e1df5 The AutoRoll server is located here: https://angle-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=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=geofflang@chromium.org Change-Id: I12373ce7d2a0d223b8dbea76ee55f4ceb65bc32c Reviewed-on: https://chromium-review.googlesource.com/1152614Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#578581}
-
Dan Elphick authored
Gets the Isolate via ToIsolate(LocalFrame*). Change-Id: I8e6fcb1043d5774483cd9533d848ae5d0fcf6179 Reviewed-on: https://chromium-review.googlesource.com/1151195Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#578580}
-
Dan Elphick authored
since they infer the Isolate from the address of the String they're called on. This will cease to work reliably in future V8 releases, so this change replaces all uses in chromium with ones that pass in the Isolate explicitly. v8: :Write, v8::WriteUtf8 and v8::Utf8Length are marked V8_DEPRECATE_SOON Change-Id: I13780586c91de98375adec0ad414d65476ed5170 Reviewed-on: https://chromium-review.googlesource.com/1149879Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Rachel Blum <groby@chromium.org> Commit-Queue: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#578579}
-
Boris Sazonov authored
This CL modifies the intent used by SyncPromoView and Sync customization pages when master sync toggle in Android is turned off. New intent opens main Sync settings page instead of the specific page for the signed in account. This is necessary because master sync toggle is not accessible from the account sync page. Bug: 863370 Change-Id: I5b2b410084bc0945b6190aec6a0c9cbf1277fefd Reviewed-on: https://chromium-review.googlesource.com/1151988Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#578578}
-
Henrik Boström authored
It was added in this CL: https://chromium-review.googlesource.com/c/chromium/src/+/1145872 I wasn't sure whether to revert that CL or disable the test. Disabling. It's flakily not terminated: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8939871163610794448/+/steps/webkit_unit_tests_on_Windows-10-15063/0/logs/WorkerThreadTest.TerminateWorkerWhileChildIsLoading/0 TBR=japhet@chromium.org, nhiroki@chromium.org NOTRY=True Bug: 868253 Change-Id: I286472514666b2d11627a515fdab92e0d73cd155 Reviewed-on: https://chromium-review.googlesource.com/1152743 Commit-Queue: Henrik Boström <hbos@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#578577}
-
Henrik Boström authored
TBR=fdoray@chromium.org, reillyg@chromium.org NOTRY=True Bug: 868254 Change-Id: I5e80699e1d04e51be31a097b2b4deb0ec5e3a7ce Reviewed-on: https://chromium-review.googlesource.com/1152737 Commit-Queue: Henrik Boström <hbos@chromium.org> Reviewed-by:
Henrik Boström <hbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#578576}
-
fuchsia-sdk-chromium-autoroll authored
The AutoRoll server is located here: https://fuchsia-sdk-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=luci.chromium.try:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: Ib683d4e2468e6454d9ac3ee53ec56218f024071e Reviewed-on: https://chromium-review.googlesource.com/1152700Reviewed-by:
Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#578575}
-
Takuto Ikuta authored
Change-Id: Id6ede6f1c9dd441cd54ac7b757ce18c6859b8b71 Reviewed-on: https://chromium-review.googlesource.com/1152721 Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Cr-Commit-Position: refs/heads/master@{#578574}
-
Elad Alon authored
Improve readability in WebRtcEventLogManager by introducing OnFirstBrowserContextLoaded() in it. Bug: 775415 Change-Id: I7b99e891eb816e763edfbd4b8c2cca05ed81f22c Reviewed-on: https://chromium-review.googlesource.com/1150172 Commit-Queue: Elad Alon <eladalon@chromium.org> Reviewed-by:
Max Morin <maxmorin@chromium.org> Cr-Commit-Position: refs/heads/master@{#578573}
-
Alexandr Ilin authored
This CL adds a new feature that allows the app using Chrome Custom Tabs to prefetch resources in parallel with loading the main intent url. The implementation reuses the DetachedResourceRequest API that was used before for the CCTParallelRequest feature (leapfrog v2). Bug: 866460 Change-Id: If22fbc5a430ba9cef3db69da93605eb448afa00a Reviewed-on: https://chromium-review.googlesource.com/1146801Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Commit-Queue: Alexandr Ilin <alexilin@chromium.org> Cr-Commit-Position: refs/heads/master@{#578572}
-
Takuto Ikuta authored
TBR=yosin@chromium.org Bug: 698165 Change-Id: I9c4e7f82ade5b91176df1a6c943665bdfbc8986c Reviewed-on: https://chromium-review.googlesource.com/1152748 Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#578571}
-
Mattias Nissler authored
This reverts commit bcbd1740. Reason for revert: The bug that prompted disabling of the state-preserving firmware update feature has been fixed per https://chromium-review.googlesource.com/1123830 so we can re-enable the feature. Original change's description: > Disable state-preserving TPM firmware update. > > The test team found an issue where cryptohomed fails to deal with the > state the TPM is in for the case when the update installation gets > interrupted and recovery is invoked to retry and complete the update. > Disable device-state preserving updates for now until the issue is > resolved. > > BUG=chromium:854576 > TEST=Triggering TPM firmware update via chrome://chrome will not invoke the device state preserving update flow. > > Change-Id: I4ac4d325bfbf953cfa14a501e871a3f9f59c699c > Reviewed-on: https://chromium-review.googlesource.com/1111998 > Reviewed-by: Steven Bennetts <stevenjb@chromium.org> > Commit-Queue: Mattias Nissler <mnissler@chromium.org> > Cr-Commit-Position: refs/heads/master@{#570020} TBR=stevenjb@chromium.org,mnissler@chromium.org Bug: chromium:854576 Change-Id: Iab59dc64130792b92071d45b82a77ddbd4553aa4 Reviewed-on: https://chromium-review.googlesource.com/1127539Reviewed-by:
Mattias Nissler <mnissler@chromium.org> Commit-Queue: Mattias Nissler <mnissler@chromium.org> Cr-Commit-Position: refs/heads/master@{#578570}
-
Anita Woodruff authored
- This doNothing() is not actually needed for the tests to pass; I suspect it's a hangover from when these tests were instrumentation tests. R=mlamouri@chromium.org Bug: 772027 Change-Id: Ia2b2103d2bdf6962c065ed5e34ecb1333d8300f5 Reviewed-on: https://chromium-review.googlesource.com/1151320Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Anita Woodruff <awdf@chromium.org> Cr-Commit-Position: refs/heads/master@{#578569}
-
Yuki Shiino authored
SecurityOrigin::CanAccess distinguishes file://host/ from file:/// (empty host), however, the security tokens for file: scheme were always "file://" regardless of host. This makes Blink think two origins are cross origin but V8 think the two are the same origin. This patch changes the security tokens in case of file: scheme to null string (will fallback to the default security token and V8 will always ask Blink to perform a security check). Bug: 858990 Change-Id: If305a994d6150984e8dd41b964003ce1eafb8b99 Reviewed-on: https://chromium-review.googlesource.com/1133105Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#578568}
-
David Grogan authored
We now set AvailableSize and PercentageResolutionSize on the ConstraintSpace properly. And also now give the flex layout algorithm more accurate offsets and main axis size. Bug: 845235 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: Ia24778ffd24a6f1122e778919686a082a63d4b06 Reviewed-on: https://chromium-review.googlesource.com/1142729 Commit-Queue: David Grogan <dgrogan@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Cr-Commit-Position: refs/heads/master@{#578567}
-