- 31 Oct, 2019 40 commits
-
-
Antonio Gomes authored
This CL moves all but one header from t_p/blink/public/web/modules/peerconnection to t_p/blink/renderer/modules/peerconnection. The only remaining one is peer_connection_dependency_factory.h that is still being included from content/renderer/renderer_blink_platform_impl.cc BUG=787254, 919392 R=guidou@chromium.org, haraken@chromium.org Change-Id: Ia21ecc977363fd03734e63a809cba3187279c633 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1876488Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Cr-Commit-Position: refs/heads/master@{#711215}
-
Andy Paicu authored
Bug: 1014899 Change-Id: Ib93a32e91b1fb6fb4dff5e3501e8289dc9271a69 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1868872 Commit-Queue: Andy Paicu <andypaicu@chromium.org> Reviewed-by:
Evan Stade <estade@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#711214}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/185ae4c0..ac12db7d Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ib8721109747d4d83c1da36e07cb2a78f3684627d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892831Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#711213}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/6946043676e7..133be10311f8 git log 6946043676e7..133be10311f8 --date=short --no-merges --format='%ad %ae %s' 2019-10-31 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/SwiftShader e1cda3916cee..fda994c63075 (2 commits) Created with: gclient setdep -r src/third_party/angle@133be10311f8 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC ynovikov@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=ynovikov@google.com Bug: chromium:None Change-Id: If2f9d9c874323c237764e244c70dec87009f1e1d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892757Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#711212}
-
Friedrich Horschig authored
With this CL, credentials in the autofill dropdown will have an additional, right-aligned G logo if they originate from the account store. Screenshots are in the linked bug. The order of suggestions and the signed-out/no-flags experience are untouched. Bug: 1018699, 1018700 Change-Id: I84894614665ea18546f9c686f5dad477d418b59e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1881562 Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Reviewed-by:
Christos Froussios <cfroussios@chromium.org> Reviewed-by:
Matthew Denton <mpdenton@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#711211}
-
Roman Sorokin [CET] authored
This reverts commit 2de3b7e6. Reason for revert: root cause should be fixed by CL:1868769 Original change's description: > Disable flaky AssistantOptInFlowTest.WAADisabledByPolicy > > TBR=updowndota > > Bug: 1013824 > Change-Id: Ibbeb425fe2511f0dadaa93bb4dd3cbfde476877b > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1857130 > Reviewed-by: David Roger <droger@chromium.org> > Commit-Queue: David Roger <droger@chromium.org> > Cr-Commit-Position: refs/heads/master@{#705491} TBR=droger@chromium.org,updowndota@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1013824 Change-Id: I0cd2c2b8119876ff2ed7b1866236783611b65a2d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892593Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Cr-Commit-Position: refs/heads/master@{#711210}
-
Omer Katz authored
Calling IsGlobalEmpty on a worklist while there are cocurrent markers will result in a data race. Introducing IsLocalViewEmpty to replace it. IsLocalView checks only the local segment and the global pool (i.e. whether the worklist is empty from the context of the current task). It does not access segments of other tasks. Bug: 986235 Change-Id: I94fa5026df580235630acbc59d597ece5a3f5c2c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890441 Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#711209}
-
Mark Cogan authored
Converts the keyboard_commands_egtest for eg2 compatibility. - Ports GetRegisteredKeyCommandsCount to an app interface method. - Makes //net:test_support a public dep of eg_test_support+eg2, to match the test_support target and prevent annoying linker errors from other code that doesn't cleanly specify dependencies. - Factors table view constants into a separate target so eg2 tests can use them safely. Bug: 987646 Change-Id: I5c7dcaa6bd29ee1cb1aa507c5386ec92d1c93a0f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1889977 Commit-Queue: Mark Cogan <marq@chromium.org> Reviewed-by:
Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#711208}
-
Mihai Sardarescu authored
Bug: 1015805 Change-Id: Ibd55ca25e03fd9ab7889fe791f714efcbef6e671 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891194Reviewed-by:
Thomas Tangl <tangltom@chromium.org> Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#711207}
-
Arnaud Mandy authored
Moving kMeanGravity constant to header file. Fixing sign according to constant in other files using it. Bug: 1018705 Change-Id: I08edc8cfbb12faed5d90b2395b97c8a0ea3c587b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1883571Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Jonathan Ross <jonross@chromium.org> Reviewed-by:
Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com> Commit-Queue: Arnaud Mandy <arnaud.mandy@intel.com> Cr-Commit-Position: refs/heads/master@{#711206}
-
Rune Lillesveen authored
If the node needs a LayoutObject on the next lifecycle update, we will trigger the re-attach going from null to non-null ComputedStyle in RecalcStyle. Had to remove the DisallowTransitionScope because it relied on the lifecycle state being kVisualUpdatePending, so that the dirtying below didn't change that. However that might not be true anymore since the SetForceReattachLayoutTree() no longer does that for null ComputedStyle. Bug: 972752 Change-Id: I18a192d6303faccf400beacfa008384051a6f4f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886856 Commit-Queue: Rune Lillesveen <futhark@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#711205}
-
Monica Basta authored
As part of the ProfileInfoInterface deprecation, we remove |SetAuthInfoOfProfileAtIndex| and |GetUserNameOfProfileAtIndex|. Afterwards, setting the auth info and getting the username should be done through |ProfileAttributesEntry::SetAuthInfo| and |ProfileAttributesEntry::GetUserName|. Change-Id: I2ccfc2260127546984f7ed4aa058b5f0cd28e9fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892773Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Monica Basta <msalama@chromium.org> Cr-Commit-Position: refs/heads/master@{#711204}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/64a3bb93dcea..9ec47a0bca27 git log 64a3bb93dcea..9ec47a0bca27 --date=short --no-merges --format='%ad %ae %s' 2019-10-31 jiajie.hu@intel.com Validate texture component type in BGL against its shader module declaration Created with: gclient setdep -r src/third_party/dawn@9ec47a0bca27 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC cwallez@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel TBR=cwallez@google.com Bug: None Change-Id: Ide00eb1c32801012658873c0dc07a9e7dbe14dc3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892755Reviewed-by:
-
Mikhail Khokhlov authored
Bug: 1018248 Change-Id: I82dc9af8d17059f014180e7e8daf08a2f0b72499 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1887691Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Mikhail Khokhlov <khokhlov@google.com> Cr-Commit-Position: refs/heads/master@{#711202}
-
Vlad Tsyrklevich authored
base::ProtectedMemory is being deprecated because it's not widely used enough to make a security impact and justify its maintenance burden. Replace use of base::ProtectedMemory with raw function pointers and add an attribute to disable CFI-icall checking. Bug: 1018834 Change-Id: Id6d956b02758aecaa0710aee56715429f5ffc267 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1884819 Commit-Queue: Primiano Tucci <primiano@chromium.org> Auto-Submit: Vlad Tsyrklevich <vtsyrklevich@chromium.org> Reviewed-by:
Primiano Tucci <primiano@chromium.org> Cr-Commit-Position: refs/heads/master@{#711201}
-
Hajime Hoshi authored
The reasons why the back-forward cache is not used is obtained from BackForwardCacheImpl::CanStoreDocument, and this should be the ground truth. Bug: 1004676 Change-Id: Ibff1eb26665dfbb29635181353657a871bb7e0b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1889890Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#711200}
-
Omer Katz authored
Both types are identical so we can merge them into one. Leaving MarkingItem as an alias for readability. There are a few cases where we had a TraceDescriptor which we had to unbox to create a new MarkingItem. Making MarkingItem and TraceDescriptor the same type avoids this. Bug: 986235 Change-Id: Ie5e596abed358c2663ae32ac1b9e279757f57e42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890436 Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
-
Tanmoy Mollik authored
SyncEncryptionKeysTabHelper::EncryptionKeyApi::SetEncryptionKeys() should pass gaia_id to AddTrustedVaultDecryptionKeys() SyncUserSettings::AddTrustedVaultDecryptionKeys() expects gaia_id as per the implementation https://cs.chromium.org/chromium/src/components/sync/driver/sync_service_crypto.cc?l=324&rcl=6b95460e56dc3cf417134e8eb1baed68daa55a34 It should not be getting a CoreAccountId object. Change-Id: I1e3dddd14539d256e0083c8cb59e81ffeda45d24 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886827Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Cr-Commit-Position: refs/heads/master@{#711198}
-
Caleb Raitto authored
IIUC, the iframe content is fetched from another resource, not written inline. Make it clear that we're presenting a devtools-like view of the frame content. Change-Id: I43adb622bcc03ef1d51429386f11966de0171dc7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890476Reviewed-by:
Christian Dullweber <dullweber@chromium.org> Commit-Queue: Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#711197}
-
Eero Häkkinen authored
This adds no BOM decoding option to TextResourceDecoderOptions which disabled BOM checking in the decoder. Bug: 796192 Change-Id: Id0eb00dca451c4898d9ae00f11bf08e7fbb33a1c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1796366Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Hiroshige Hayashizaki <hiroshige@chromium.org> Commit-Queue: Eero Häkkinen <eero.hakkinen@intel.com> Cr-Commit-Position: refs/heads/master@{#711196}
-
Alex Clarke authored
We're seeing expensive low priority tasks getting in the way of navigation tasks, so we should turn this logic off. It's an oversight that it was ever enabled on the UI and IO thread in the first place. This change has exposed a few latent issues: 1. BackgroundSyncTest is failing on the bots due to an infra issue where Play Services is not up to date. 2. Several extension tests where failing due to unwanted TaskPriority propagation for tasks posted from GetExtensionFileTaskRunner() to the UI thread. These tasks where inheriting the sequence's USER_VISIBLE priority and running much later than expected as a result of removing the anti-starvation code. I've worked around that by explicitly setting the priority to the default which is USER_BLOCKING. 3. Some navigations in the UrlOverridingTest are actually getting canceled by the InterceptNavigationThrottle. Previously the task notifying this didn't run in time to be picked up by the test, now they are we need to update the test expectations. Bug: 1013535 Change-Id: Ie9e5bc38f62e8a451b9287390752d3bd1c3284c3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1863410 Commit-Queue: Alex Clarke <alexclarke@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Reviewed-by:
Rayan Kanso <rayankans@chromium.org> Reviewed-by:
Mugdha Lakhani <nator@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#711195}
-
Henrik Andreasson authored
This reverts commit d7b4f14f. Reason for revert: Discussed with Chrome Sheriff and decided to land the original CL again since the CL only affects logging and no audio-related functions. Original change's description: > Revert "Adds native WebRTC logs to media::InputController." > > This reverts commit 29ab72b2. > > Reason for revert: [sheriff]suspected culprit for https://ci.chromium.org/p/chromium/builders/ci/Mac10.12%20Tests/28504 > > Original change's description: > > Adds native WebRTC logs to media::InputController. > > > > These changes will make it possible to track down issues related to > > audio input streams (create/open/close) using native logs. > > > > Focusing on AudioServiceAudioStreams since old streams are going away. > > > > Tbr: olka > > Bug: 1017219 > > Change-Id: Iffbc1880d9eea1868ce1a5e17e298c03328949e0 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886676 > > Commit-Queue: Henrik Andreasson <henrika@chromium.org> > > Reviewed-by: Henrik Andreasson <henrika@chromium.org> > > Reviewed-by: Guido Urdaneta <guidou@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#711176} > > TBR=guidou@chromium.org,henrika@chromium.org > > Change-Id: Ibe7f92aaa821264e3cfd1cc6353c1d1ecae0e134 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 1017219 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892774 > Reviewed-by: Armando Miraglia <armax@chromium.org> > Commit-Queue: Armando Miraglia <armax@chromium.org> > Cr-Commit-Position: refs/heads/master@{#711191} TBR=guidou@chromium.org,armax@chromium.org,henrika@chromium.org Change-Id: I9bedb45daccd46f0352420688bee1380b48f86ad No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1017219 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891174Reviewed-by:
Henrik Andreasson <henrika@chromium.org> Commit-Queue: Henrik Andreasson <henrika@chromium.org> Cr-Commit-Position: refs/heads/master@{#711194}
-
Vasilii Sukhanov authored
There are two bugs fixed together. - HttpAuthManagerImpl::OnLoginSuccesful should care about the case when the password store isn't done. In such a case the prompt should not be invoked. - ManagePasswordsState should account for the method parameters may depend on the internal memeber |form_manager_|. Bug: 1017481 Change-Id: Ib63d1c7480cc2a5130d943e36cb0a02f9b0af7cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890437 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#711193}
-
Hajime Hoshi authored
Bug: 1004676 Change-Id: I7d6ef345c9abea66d47cc2808ebb8aebe14eef9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892634Reviewed-by:
-
Armando Miraglia authored
This reverts commit 29ab72b2. Reason for revert: [sheriff]suspected culprit for https://ci.chromium.org/p/chromium/builders/ci/Mac10.12%20Tests/28504 Original change's description: > Adds native WebRTC logs to media::InputController. > > These changes will make it possible to track down issues related to > audio input streams (create/open/close) using native logs. > > Focusing on AudioServiceAudioStreams since old streams are going away. > > Tbr: olka > Bug: 1017219 > Change-Id: Iffbc1880d9eea1868ce1a5e17e298c03328949e0 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886676 > Commit-Queue: Henrik Andreasson <henrika@chromium.org> > Reviewed-by: Henrik Andreasson <henrika@chromium.org> > Reviewed-by: Guido Urdaneta <guidou@chromium.org> > Cr-Commit-Position: refs/heads/master@{#711176} TBR=guidou@chromium.org,henrika@chromium.org Change-Id: Ibe7f92aaa821264e3cfd1cc6353c1d1ecae0e134 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1017219 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892774Reviewed-by:
Armando Miraglia <armax@chromium.org> Commit-Queue: Armando Miraglia <armax@chromium.org> Cr-Commit-Position: refs/heads/master@{#711191}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast TBR=cr-fuchsia+bot@chromium.org Change-Id: Id117193a8707693615ca7c9345c3af3f50da2743 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892751Reviewed-by:
-
Christian Dullweber authored
This reverts commit 1205b351. Reason for revert: Accidentally disabled the wrong test. The flaky test seems ok again, so just reverting this CL. Original change's description: > Disable flaky TaskManagerBrowserTest.NoticeHostedAppTabAfterReload on Linux > > Tbr: afakhry@chromium.org > Bug: 1017110 > Change-Id: I3c153b07a4d6c381b7c85684d0e59f09b71712b9 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1875259 > Reviewed-by: Christian Dullweber <dullweber@chromium.org> > Commit-Queue: Christian Dullweber <dullweber@chromium.org> > Cr-Commit-Position: refs/heads/master@{#708555} TBR=afakhry@chromium.org,dullweber@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1017110 Change-Id: Ieda5cfa39e22dd0b965e27d08db6d4962baedde6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892771Reviewed-by:
-
Arthur Hemery authored
PerNavigationMojoInterface is now on by default and the flag has been removed. Bug: 968090 Change-Id: I1eb346319fba02cd7d6f22d8a1db444162ea743e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886675Reviewed-by:
Nik Bhagat <nikunjb@chromium.org> Commit-Queue: Arthur Hemery <ahemery@chromium.org> Cr-Commit-Position: refs/heads/master@{#711188}
-
Johannes Kron authored
Bug: chromium:1007526 Change-Id: I05ca79c5b142737eb0549119278b3725d64613f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886670Reviewed-by:
Jesse Doherty <jwd@chromium.org> Commit-Queue: Johannes Kron <kron@chromium.org> Cr-Commit-Position: refs/heads/master@{#711187}
-
Marina Sakai authored
This CL removes macro-generated fucntions for private properties of CustomElementLifecycle, and replaces them with the new API GetSymbol, which uses an instance address of V8PrivateProperty::SymbolKey as a key to get Symbol. Bug: 715418 Change-Id: If4cd6487c11e4999f2165bf1ea684c573465a248 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892315 Commit-Queue: Marina Sakai <marinasakai@google.com> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#711186}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/7e2cba6a5241..618ee509421f git log 7e2cba6a5241..618ee509421f --date=short --no-merges --format='%ad %ae %s' 2019-10-30 dneto@google.com Fix some clang-tidy issues in graphics_robust_access_pass (#2998) 2019-10-30 stevenperron@google.com Add description of wrap-opkill. (#3000) 2019-10-30 kubak@google.com [opt] Do not compare optimized binary with an invalidated buffer (#2999) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@618ee509421f If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,ynovikov@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 TBR=radial-bots+chrome-roll@google.com,ynovikov@google.com Bug: None Change-Id: Icae7cd22ac49ce55040477cca0802d593a99c607 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892047Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/841b01678c49..6946043676e7 git log 841b01678c49..6946043676e7 --date=short --no-merges --format='%ad %ae %s' 2019-10-31 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/spirv-tools/src 7e2cba6a5241..618ee509421f (3 commits) Created with: gclient setdep -r src/third_party/angle@6946043676e7 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC ynovikov@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md 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=ynovikov@google.com Bug: chromium:None Change-Id: Ifd9bc93f8590d99e5e601ceb1c766feee3bd0903 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1892559Reviewed-by:
-
Marina Sakai authored
This CL removes macro-generated functions for private properties of MessageChannel by using the new API GetSymbol created in CL1877490. Bug: 715418 Change-Id: I913997334d77807bedc5d2cc8022788c0daa80fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888616 Commit-Queue: Marina Sakai <marinasakai@google.com> Reviewed-by:
-
Caleb Rouleau authored
The possible_browser is not used. The skipping is done later when you try to run the story (in story_runner.py), so skipping here is unnecessary. This change saves ~2 seconds per Android test. The tests seem to run in parallel, so it is unclear how much real time this will save. Bug: 944909 Change-Id: I7e89d7692a79afefcb2706fbaa4d9ce6c1af21e4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1887873 Auto-Submit: Caleb Rouleau <crouleau@chromium.org> Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org> Reviewed-by:
-
Mihai Sardarescu authored
Re-enable SigninSyncConfirmationTest and ProfileMenuViewExtensionsTestWithScopedAccountConsistency tests. This CLs removes chrome/test/data/webui/signin_browsertest.{h|cc} as they are now obsolete. And it also re-enables SigninSyncConfirmationTest.Dialog test as it should be working now. Bug: 862573 Change-Id: I2de61f71ad7ba3935a358ea0d4d8a1864199e1bd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890077 Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Reviewed-by: -
Tobias Sargeant authored
Enable Crashpad collection of minidumps on Android. Other platforms are skipped because the concept of the user data directory is OS-specific. On Android specifically crashes are written to [app-data-dir]/cache/Crashpad. Test: manual Test: Check current crash count with: Test: adb shell ls '/data/data/org.chromium.weblayer.shell/cache/Crashpad/pending/*.dmp' | wc -l Test: load chrome://crash or chrome://inducebrowsercrashforrealz Test: Check crash count has increased by 1 Change-Id: I9718754dc4218e7e2f2e7737de420db83d4de14b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1883653 Commit-Queue: Tobias Sargeant <tobiasjs@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#711180}
-
Michael Lippautz authored
This removes - the inline vector arena as they are no different then normal vectors - all but a single vector arena as the heuristics for promptly freeing no longer apply and having multiple arenas causes fragmentation issues Bug: 1014932 Change-Id: I7b5cbd9c2f617bedb921cd5e83cb115bd97bdd9e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1863804Reviewed-by:
Anton Bikineev <bikineev@chromium.org> Reviewed-by:
-
Yi Su authored
Extend the expiration date of 2 metrics for iOS copy image feature: ContextMenu.iOS.GetImageDataByJsResult ContextMenu.iOS.CopyImage Also add gambard@ as backup owner of some metrics. Bug: 1018791 Change-Id: I297f01e1430cb0457b59a11dd22b19de8acd5a95 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1890052Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Yi Su <mrsuyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#711178}
-
Armando Miraglia authored
TBR=mmoskvitin Bug: 1020070 Change-Id: I2c5f2790ec98ed7587f73cc15a88eeb059303809 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1891193Reviewed-by:
-
henrika authored
These changes will make it possible to track down issues related to audio input streams (create/open/close) using native logs. Focusing on AudioServiceAudioStreams since old streams are going away. Tbr: olka Bug: 1017219 Change-Id: Iffbc1880d9eea1868ce1a5e17e298c03328949e0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1886676 Commit-Queue: Henrik Andreasson <henrika@chromium.org> Reviewed-by:
-
![Roman Sorokin [CET]'s avatar](https://secure.gravatar.com/avatar/7b28a287700b8b8659a03c33a38be1a8?s=80&d=identicon "Roman Sorokin [CET]")
