- 19 Oct, 2018 40 commits
-
-
Wez authored
We switched to staging packages into /tmp instead of data, because some devices have surprisingly slow physical storage. However, since /tmp is a RAM filesystem it can be too small for some packages(!), in particular Debug and/or component builds. We revert to staging into /data by default, but --target-staging-path can now be used to override the path to stage packages into. Change-Id: I25adb9563df0236961f9e79e8b01d9536146ba8d Reviewed-on: https://chromium-review.googlesource.com/c/1291841Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#601309}
-
Azeem Arshad authored
This CL fixes issue with settings page crashing in guest profiles. Bug: 896870 Change-Id: I39f4df84843e723b25300fefb8936c7839c3bd8d Reviewed-on: https://chromium-review.googlesource.com/c/1289952Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Azeem Arshad <azeemarshad@chromium.org> Commit-Queue: Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#601308}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/245ff8c6a020..f04272b969b8 Created with: gclient setdep -r src-internal@f04272b969b8 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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=mmoss@chromium.org Change-Id: I427c75f28fb807fb203efb55b4d20d4afb9cf0f9 Reviewed-on: https://chromium-review.googlesource.com/c/1292276Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#601307}
-
Jacob Dufault authored
This repros by - Open login or lock screen, enter tablet mode - Type password using VK - Dismiss VK - Hit the submit button on the password field You can also submit by hitting 'enter' on the VK Bug: 892270 Change-Id: I39b40c6710a3ddca22cd110255f05014a9134798 Reviewed-on: https://chromium-review.googlesource.com/c/1287177Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Jacob Dufault <jdufault@chromium.org> Cr-Commit-Position: refs/heads/master@{#601306}
-
Dominick Ng authored
This CL implements a method on the ARC IntentHelperHost to allow ARC to request that a URL be opened as a web app in Chrome. If that URL does not have a corresponding installed web app, the method falls back to simply opening the URL in a tab. An additional extension app launch source is added for the new launch pathway. BUG=893927 Change-Id: Ifbcc48f7cab65373dc6353d422580d6e9cc43796 Reviewed-on: https://chromium-review.googlesource.com/c/1282546Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Greg Kerr <kerrnel@chromium.org> Reviewed-by:
David Jacobo <djacobo@chromium.org> Commit-Queue: Dominick Ng <dominickn@chromium.org> Cr-Commit-Position: refs/heads/master@{#601305}
-
Nate Fischer authored
This swaps the deprecated 'arraysize' macro for base::size(), in the //android_webview/ folder. No change to logic. R=boliu@chromium.org Bug: 837308 Test: N/A Change-Id: Ia6960f242f78f117eeb831e451f27596583b0cd2 Reviewed-on: https://chromium-review.googlesource.com/c/1292514Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Nate Fischer <ntfschr@chromium.org> Cr-Commit-Position: refs/heads/master@{#601304}
-
Mostyn Bramley-Moore authored
CL 1278961 introduced a second blink::kMaxThreadCount which breaks jumbo builds. Let's rename it to kMaxTaskWorkletThreads. Bug: 879306 Change-Id: I5e27227c9ffd01d862ca0ace66029bc932ccd2b9 Reviewed-on: https://chromium-review.googlesource.com/c/1292471 Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Commit-Queue: Nate Chapin <japhet@chromium.org> Reviewed-by:
Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#601303}
-
Becca Hughes authored
PiP was having issues with hiding the controls because we have a timer that will hide the controls. This timer gets reset every time we interact with it. In one case we actually stop the timer so it does not hide the controls anymore. BUG=897246 Change-Id: I4965b5c6cae0047ee0260131a16c9b346003bcec Reviewed-on: https://chromium-review.googlesource.com/c/1292518Reviewed-by:
apacible <apacible@chromium.org> Commit-Queue: Becca Hughes <beccahughes@chromium.org> Cr-Commit-Position: refs/heads/master@{#601302}
-
Vladimir Levin authored
This patch re-enables isolation nodes for layout views (iframes). It seems that this causes a memory regression on one of the telemetry tests, but it seems that it might be a timing issue. The evidence for this is that the regression happens in free memory (evictable) and the tile sizes seem to be same or similar with or without the flag. For more details, see the referenced bug. R=chrishtr@chromium.org, pdr@chromium.org, wangxianzhu@chromium.org Bug: 896328 Change-Id: I9c399f253f246364b68980350cec3d14e5690b75 Reviewed-on: https://chromium-review.googlesource.com/c/1292511Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#601301}
-
Scott Graham authored
21edfd3c3a20 [fuchsia] Move non-tests out of tests in the crashpad_tests package ce122b644ce2 [util/net] add error message to scheme check 83e37a9ac145 [fuchsia] add commands for testing on Fuchsia 78d081ee4e91 linux: save and restore old signal actions 1b4fdd0fd079 fuchsia: Re-enable HTTPS transport, but disable tests 9554a89ab60a fuchsia: Use zx_task_resume_from_exception() instead of zx_task_resume() 9b2a119dc6d8 elf: don't warn on trailing unread bytes in the elf dynamic array 0fb654142873 [fuchsia][linux] optionally return local report ID on exception handling 1c09361c2cde fuchsia: Let util build in Mac-host builds 237c5ebdf3a9 fuchsia: Fix include_dirs for host Mac build 7682f0f6ebbd Add ThreadSnapshotMinidump 46b329b3690d list headers' includes as public dependencies b6103e157c5e Add SystemSnapshotMinidump 8d17d5b4cf33 Decode Thread Context in minidump 3b68e1278952 Update bot link for new LUCI console c7f1543dd388 Don't auto-generate gyp build files any more 289be79a33f2 Roll mini_chromium to 3577ffd 196897eb0804 Remove win buildbot bots from cq config 2951fbde523f Remove cq_name from cq.cfg, no longer required/allowed fa96a04fc64d Add .vpython spec to add pywintypes to luci runs TBR: mark@chromium.org Change-Id: I8ecc5aa3925467c2e79a2c1eee7e1bf232aa7331 Reviewed-on: https://chromium-review.googlesource.com/c/1292150Reviewed-by:Scott Graham <scottmg@chromium.org> Commit-Queue: Scott Graham <scottmg@chromium.org> Cr-Commit-Position: refs/heads/master@{#601300}
-
Tarun Bansal authored
Report H2 ping latency to network quality estimator where it is used for computing the network quality. Change-Id: Ib25e339ccd3967293ec8be94ef35aaa2a30a0678 Bug: 823322 Reviewed-on: https://chromium-review.googlesource.com/c/1285077Reviewed-by:
Ryan Hamilton <rch@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#601299}
-
Ryan Sturm authored
This CL moves previews triggering from the IO thread (ChromeResourceDispatcherHostDelegate) to the UI thread (ChromeContentBrowserClient). The new call sites occur in NavigationRequest and update the previews_state in CommonNavigationParams held by the NavigationRequest directly. This means PreviewsDeciderImpl is being moved to the UI thread. Significant changes: - PreviewsDeciderImpl is now owned by the PreviewsUIService instead of profile IO data. - PreviewsUserData is now entirely a UI thread notion. For now, we use PreviewsUITabHelper to manage its lifetime, but it could move to NavigationHandle (if NavigationHandle already supported UserData, that solution would be ideal). - ChromeNavigationData is update to remove PreviewsUserData and PreviewsState (the latter has moved into PreviewsUserData and there is a notion of Allowed PreviewsState and committed PreviewsState) - Offline Previews committed checks use the OfflinePreviewsTabHelper instead of PreviewsUserData now that we are on the same thread. - Cleanup on ResourceRequestInfo to remove PreviewsState setter as well as cleanup to remove PreviewsState from being passed back out of the loader code into NavigationRequest. - There is a TODO to re-evaluate PreviewsState during redirects, which will *not* be plumbed into the network service, so DataReductionProxy Previews will not be updated. Bug: 842233 Change-Id: I80a3044a4c84734b877f8ebcbe5a4bb2c2900dbb Reviewed-on: https://chromium-review.googlesource.com/c/1266196 Commit-Queue: Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Jian Li <jianli@chromium.org> Cr-Commit-Position: refs/heads/master@{#601298}
-
Chris Palmer authored
Clear Site Data has shipped. Bug: None Change-Id: If6a15404905a8086fc195ceb1cb2ec6a66a95909 Reviewed-on: https://chromium-review.googlesource.com/c/1292250Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Chris Palmer <palmer@chromium.org> Cr-Commit-Position: refs/heads/master@{#601297}
-
chrome://accessibilityJames Wallace-Lee authored
In chrome://accessibility, the "Chrome Native UI" section behaves like the "Pages" section. Browser windows are listed individually using the .row class the Pages list uses. -AccessibilityUIMessageHandler::RequestNativeUITree takes a session ID as an argument and returns the accessibility tree for a single browser window. -The TargetDescriptor dictionary includes a field for a browser window's session id, and a "type" field that is either "page" for a WebContents or "browser" for a Browser. -In accessibility.js, showTree and other functions handle both browser windows and pages, since they can check |data|'s type field. -Elements in a .row element are given ids based on the row's id + a description of the element: '<id>:title', 'showTree', 'copyTree', 'hideTree', 'tree'. Bug: 842895 Change-Id: I7ef7b4edb884ff097fdaefc41ba97a813a7b49e1 Reviewed-on: https://chromium-review.googlesource.com/c/1260009 Commit-Queue: James Wallace-Lee <jamwalla@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#601296}
-
Joe Downing authored
This reverts commit 64646172. Reason for revert: LoggingTest.LogMessageMarkersOnStack is failing on several builders and this test was introduced in this CL: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win7%20Tests%20%28dbg%29%281%29/72386 https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Linux%20MSan%20Tests/12371 Example error message: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8932209218566904752/+/steps/base_unittests/0/logs/LoggingTest.LogMessageMarkersOnStack/0 Original change's description: > Add start & end markers around |str_stack| for use by Crash service. > > Crash can locate particular entries in a stack by scanning for start & > end markers with unusual values. We add start & end markers around the > |str_stack| copy of the LogMessage string, created when we're about to > crash the process due to a FATAL level log message. > > Bug: 802393 > Change-Id: I8fd462cc515ddbefb8c7b34363eea848171aa776 > Reviewed-on: https://chromium-review.googlesource.com/c/1201344 > Reviewed-by: Wez <wez@chromium.org> > Reviewed-by: Albert J. Wong <ajwong@chromium.org> > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > Commit-Queue: Wez <wez@chromium.org> > Cr-Commit-Position: refs/heads/master@{#601236} TBR=ajwong@chromium.org,dcheng@chromium.org,wez@chromium.org Change-Id: If00343894caa467887191fbb09a4c80233cc94b1 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 802393 Reviewed-on: https://chromium-review.googlesource.com/c/1292595Reviewed-by:
Joe Downing <joedow@chromium.org> Commit-Queue: Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#601295}
-
Will Harris authored
As stated in BrowserChildProcessHostImpl::OnChildDisconnected: "OnChildDisconnected may be called without OnChannelConnected, so stop the early exit watcher so GetTerminationStatus can close the process handle" This test previously did not take this into account and always expected a connected notification before a disconnected notification, so the test can be fixed by just removing this assumption. BUG=879555 Change-Id: I68ce0d27765a6b5387d57bf7af10b84c8fb240ac Reviewed-on: https://chromium-review.googlesource.com/c/1286910Reviewed-by:
Will Harris <wfh@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Commit-Queue: Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#601294}
-
Yi Gu authored
For worklet animations that run via the main thread, the console log from the worklet may not get back to the main thread within 2 frames. To fix this, we can either wait longer, i.e. wait an extra frame, or not rely on the console log from the worklet as what we do in this patch. Bug: 887659 Change-Id: Ifb8619f571f51a7f5abe132a2c65d7a58f59d600 Reviewed-on: https://chromium-review.googlesource.com/c/1289430Reviewed-by:
Majid Valipour <majidvp@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#601293}
-
Marc-Antoine Ruel authored
As part of issue 894048 we'll probably unmap them and have another mean to map the binaries, but in the meantime this is the first step towards migrating to the Go binaries. R=iannucci@chromium.org Bug: 894045 Change-Id: I40c22997e8948a23a2a56e368b1cea55df0d5ab4 Reviewed-on: https://chromium-review.googlesource.com/c/1286748Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Marc-Antoine Ruel <maruel@chromium.org> Cr-Commit-Position: refs/heads/master@{#601292}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/cdebbeb8cca2..af3d6cc8910f git log cdebbeb8cca2..af3d6cc8910f --date=short --no-merges --format='%ad %ae %s' 2018-10-19 thestig@chromium.org Roll v8/ 604c50c84..626844157 (516 commits) Created with: gclient setdep -r src/third_party/pdfium@af3d6cc8910f The AutoRoll server is located here: https://autoroll.skia.org/r/pdfium-autoroll 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: I9bb62c4420cf39885fd1a9b7ed40c7ae897f5938 Reviewed-on: https://chromium-review.googlesource.com/c/1292289Reviewed-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@{#601291}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/SPIRV-Tools.git/+log/715afb0cea5a..8edf3557ca17 git log 715afb0cea5a..8edf3557ca17 --date=short --no-merges --format='%ad %ae %s' 2018-10-19 stevenperron@google.com Revert "Add custom target to wrap around custom commands. (#1986)" (#1996) 2018-10-19 sheredom@gmail.com Add validaton for SPV_KHR_8bit_storage + convert to/from floats. (#1990) Created with: gclient setdep -r src/third_party/SPIRV-Tools/src@8edf3557ca17 The AutoRoll server is located here: https://autoroll.skia.org/r/spirv-tools-chromium-autoroll 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: Ia844635eeb30bb5a5aa5fb961650fb6144e49797 Reviewed-on: https://chromium-review.googlesource.com/c/1291756Reviewed-by:
-
Kevin McNee authored
The code for this was mostly already written, but its use was commented out, so the callbacks weren't being run. Also, we now use Bind{Once,Repeating} for these callbacks. Bug: 892889 Change-Id: I544de97a511286236cc9c4ad7a89e094aed202f5 Reviewed-on: https://chromium-review.googlesource.com/c/1283729Reviewed-by:Paul Meyer <paulmeyer@chromium.org> Commit-Queue: Kevin McNee <mcnee@chromium.org> Cr-Commit-Position: refs/heads/master@{#601289}
-
Christian Biesinger authored
This is a precondition for the next step I want to do, which is initialize fragments by passing the builder to their constructor, like we already do for constraint spaces. R=eae@chromium.org Change-Id: I626fe15da8d9643a3152f7a1cb0d4e556fb0d22a Reviewed-on: https://chromium-review.googlesource.com/c/1289720 Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#601288}
-
Weidong Guo authored
Changes: When an active window exists, clicking on shelf item to activate another window should not run slide down animation. Bug: 896908 Change-Id: Ifd037b139da44bac3d05c3c88dd5c7f044f7d898 Reviewed-on: https://chromium-review.googlesource.com/c/1292050 Commit-Queue: Weidong Guo <weidongg@chromium.org> Reviewed-by:
-
Matthew Denton authored
Fixes the comments on RecvFrom and SendTo in udp_socket_posix.h and udp_socket_win.h to more accurately reflect who is responsible for keeping some of the function parameters alive. Namely, the sockets always take a reference to the provided IOBuffer in the case of asynch IO. And the SendTo function also creates a copy of the provided IPAddress if necessary, while RecvFrom take a raw pointer to the IPAddress and the caller is in charge of keeping it alive. Change-Id: I080557b8c69c9035eace94a0359ec5ce09c4cf0d Reviewed-on: https://chromium-review.googlesource.com/c/1290089Reviewed-by:
-
Dmitry Gozman authored
For window.open('') we start navigation to an empty url, which goes to RenderFrameImpl::DecidePolicyForNavigation. There we could in some cases decide to go to the browser process instead of committing locally: embedder can decide to fork process, chrome app window can use browser_handles_all_top_level_requests preference. This might trigger erroneous beforeunload/unload handlers if the process touches dom window synchronously before the navigation is processed in browser. Bug: 896379 Change-Id: If62c0e470b1ce8bc060f7378ba54d2b5930291ea Reviewed-on: https://chromium-review.googlesource.com/c/1291360Reviewed-by:Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#601285}
-
Joe Downing authored
This reverts commit abc48df4. Reason for revert: Speculative revert due to MacSandbox* test failures in content_unittests: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/29903 failures: MacSandboxTest.FileAccess MacSandboxTest.UrandomAccess MacSandboxTest.OpenSSLAccess MacSandboxTest.ClipboardAccess Failure log: https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8932209461845513232/+/steps/content_unittests_on__none__GPU_on_Mac_on_Mac-10.11/0/logs/MacSandboxTest.ClipboardAccess/0 Original change's description: > macOS V2 Sandbox: Report correct sandbox profile for nacl-loader. > > The nacl loader process, used only on macOS, currently tries to load an > invalid profile type under the V2 sandbox. This loads the correct type. > This also adds necessary IOKit resource access to the nacl sandbox profile. > > Bug: 892554 > Change-Id: I3118c00a17502efdd88545ef35e81e3a480d2f15 > Reviewed-on: https://chromium-review.googlesource.com/c/1289070 > Reviewed-by: Robert Sesek <rsesek@chromium.org> > Reviewed-by: Derek Schuff <dschuff@chromium.org> > Commit-Queue: Greg Kerr <kerrnel@chromium.org> > Cr-Commit-Position: refs/heads/master@{#601244} TBR=dschuff@chromium.org,kerrnel@chromium.org,rsesek@chromium.org Change-Id: I6825d62c07e111781530bfcb45618782a1570799 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 892554 Reviewed-on: https://chromium-review.googlesource.com/c/1292525Reviewed-by:
-
Kyungyoung Heo authored
Add build/build_config.h to using OS macro without presubmit warning. Bug: 810841 Change-Id: I81fc8bfdbcb539983d17b72d6541ee4119ea9410 Reviewed-on: https://chromium-review.googlesource.com/c/1278457 Commit-Queue: Jinho Bang <jinho.bang@samsung.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#601283}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/82d432980cff..1cb20deb746a git log 82d432980cff..1cb20deb746a --date=short --no-merges --format='%ad %ae %s' 2018-10-19 niklas.enbom@webrtc.org Revert "Make sure Chromium will pick the correct field_trial/metric impl." 2018-10-19 nisse@webrtc.org Replace _stricmp with absl::EqualsIgnoreCase 2018-10-19 yvesg@webrtc.org Mute failed tests when no sanitizer defects. 2018-10-19 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision c66210d3..b1cb8571 (601019:601125) 2018-10-19 sakal@webrtc.org Set renderThreadHandler to null on uncaught exception in EglRenderer. 2018-10-19 mbonadei@webrtc.org Prefix flag macros with WEBRTC_. 2018-10-19 mbonadei@webrtc.org Make sure Chromium will pick the correct field_trial/metric impl. 2018-10-19 mbonadei@webrtc.org Export symbols needed by the Chromium component build (part 5). 2018-10-19 nisse@webrtc.org Move the SocketStream class to test target 2018-10-19 mbonadei@webrtc.org Adding missing #include on absl/memory/memory.h. Created with: gclient setdep -r src/third_party/webrtc@1cb20deb746a The AutoRoll server is located here: https://autoroll.skia.org/r/webrtc-chromium-autoroll 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:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng BUG=chromium:None,chromium:b/117400268,chromium:None TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: Ic76c1f589b49e57dcc0d65351edf8bbd3256e1c1 Reviewed-on: https://chromium-review.googlesource.com/c/1291790Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/a31f315c7410..a4f1ca269765 git log a31f315c7410..a4f1ca269765 --date=short --no-merges --format='%ad %ae %s' 2018-10-19 bpastene@chromium.org cros_run_vm_test: Don't raise an exception when a host-cmd exits non-zero. Created with: gclient setdep -r src/third_party/chromite@a4f1ca269765 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll 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: I1c16d668d8fa98be9d16b73431a06db82b2dac28 Reviewed-on: https://chromium-review.googlesource.com/c/1291846Reviewed-by:
-
Xiaoqian Dai authored
Let TabDragController::RevertDrag() to handle the case if the dragged tab is not dragged far enough and needs to be merged back to its source window's tabstrip. This CL also moved the bounds update of source window to SplitViewController::TabDraggedWindowObserver after the drag ends. This CL also reverts the change in https://chromium-review.googlesource.com/c/chromium/src/+/1266495 since after this CL, the reason to split the one function ClearTabDraggingInfo() into two functions no longer exists. Bug: 892517 Change-Id: I412efa82cbba8629d750859865e11ab21d020637 Reviewed-on: https://chromium-review.googlesource.com/c/1290269Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/0f44307feaae..245ff8c6a020 Created with: gclient setdep -r src-internal@245ff8c6a020 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll 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. BUG=chromium:893729,chromium:895202 TBR=mmoss@chromium.org Change-Id: I9ac7b67e2cb019cd823c8ec2fabed1023f808a72 Reviewed-on: https://chromium-review.googlesource.com/c/1292229Reviewed-by:
-
Jared Saul authored
This is a retry/relanding of https://chromium-review.googlesource.com/c/chromium/src/+/1269524. We're having a *very* hard time reproducing the flakes that the trybots experienced, but I've made some improvements to attempt to reduce possible race conditions in the browsertests. Patch set 1 is the original CL; patch set 2 contains the new changes. (When a card accrues 3 strikes, future offers to save for that card should show the omnibox save icon, but NOT pop up the bubble. On Android, no offer to save is shown at all.) TBR=mahmadi@chromium.org Bug: 884817 Change-Id: If641cff300cab6122bfd0a52942376b8b6449fed Reviewed-on: https://chromium-review.googlesource.com/c/1292169 Commit-Queue: Jared Saul <jsaul@google.com> Reviewed-by:
Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#601278}
-
Ovidio Henriquez authored
This change updates the WebUsbAllowDevicesForUrls policy entry in policy_templates.json to have the future property set to true. This prevents the policy from being in documentation and the generated ADMX and linux.json file until the policy is visible in Site settings. Design document: https://docs.google.com/document/d/1MPvsrWiVD_jAC8ELyk8njFpy6j1thfVU5aWT3TCWE8w Bug: 854329 Change-Id: Ib2c9a04fd992d8c6c88a52ec9c954a48ff9a673d Reviewed-on: https://chromium-review.googlesource.com/c/1262082 Commit-Queue: Ovidio Henriquez <odejesush@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Cr-Commit-Position: refs/heads/master@{#601277}
-
Stephen Martinis authored
It appears that some return statements were accidentally omitted during an earlier refactor. This CL adds a few of them back. Bug: 896175 Change-Id: I218d8ae6fa347fea775d606b5aa04064662ff7e6 Reviewed-on: https://chromium-review.googlesource.com/c/1291787Reviewed-by:
Ned Nguyen <nednguyen@google.com> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> Cr-Commit-Position: refs/heads/master@{#601276}
-
chrome://connnection-helpCarlos IL authored
Added the Symantec cert error code to the bundled connection help that shows when clicking the 'Learn More' link in an interstitial triggers another interstitial. In this case the link is in the section with other cert errors instead of separate, since if the Symantec error shows up in the Chrome support site it is likely due to a captive portal or an extension, not due to the support site using Symantec certs. Change-Id: I3a0889e2e5f683d120f2bdf4a4ce76dae75fc225 Reviewed-on: https://chromium-review.googlesource.com/c/1292005 Commit-Queue: Carlos IL <carlosil@chromium.org> Reviewed-by:
Christopher Thompson <cthomp@chromium.org> Cr-Commit-Position: refs/heads/master@{#601275}
-
Alexander Alekseev authored
Bug: 896645 Change-Id: Id12e434ba7b284829d10b3c489e05278beebbb02 Reviewed-on: https://chromium-review.googlesource.com/c/1290200Reviewed-by:
-
Dan Harrington authored
Helper script to construct a gtest_filter. Change-Id: I370a36c96d9f6f4e3beb450dd1ca26a65154e56f Reviewed-on: https://chromium-review.googlesource.com/c/1265025 Commit-Queue: Dan H <harringtond@google.com> Reviewed-by:
Carlos Knippschild <carlosk@chromium.org> Cr-Commit-Position: refs/heads/master@{#601273}
-
Michael Giuffrida authored
Fixes the placeholder to match the version in chromium_strings.grd. Bug: 895201 Change-Id: I31e1aec017661597cad41f9d9a1fff6601000b3a Reviewed-on: https://chromium-review.googlesource.com/c/1288908Reviewed-by:
-
Ned Nguyen authored
This is a reland of 5f74e543. The original reland is in patch set 1, fixes are in subsequent patchset. The fix implements: If --release or --debug is specified and there are both more than one target dirs, it would try to choose the dir that match the configuration on args.gn Original change's description: >Automatically pick a target directory which exists for run_web_tests.py script > > This also updates how we compute the default for 'configuration'. First, we try to look for the > configuration definition (args.gn) & use that value. If that doesn't succeed, we guess the > value of configuration using the target value. > > Reviewers: the main change is in third_party/blink/tools/blinkpy/web_tests/port/base.py, the rest are followed by. > > > Bug: 893618 > Change-Id: I578c977bcaccd6294596f8cf7079748809698db6 > Reviewed-on: https://chromium-review.googlesource.com/c/1281043 > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Reviewed-by: Robert Ma <robertma@chromium.org> > Commit-Queue: Ned Nguyen <nednguyen@google.com> > Cr-Commit-Position: refs/heads/master@{#600760} Bug: 893618 Change-Id: I386b8261fc4abddd51a5b4f290c5eb3ac6ce5361 Reviewed-on: https://chromium-review.googlesource.com/c/1289308 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by:
-
Joe Downing authored
This reverts commit 66a392fa. Reason for revert: Causing component_unittest failures: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Cast%20Audio%20Linux/23527 failing tests: SyncableServiceBasedBridgeTest.ShouldDecryptPreviousDirectoryDataAfterRestart SyncableServiceBasedBridgeTest.ShouldDecryptInitialRemoteData SyncableServiceBasedBridgeTest.ShouldDecryptForRemoteDeletion Original change's description: > Add encryption support to pseudo-USS > > This patch extends SyncableServiceBasedBridge (core to pseudo-USS) > with support for encryption, via a newly introduced interface > (ModelCryptographer), which needs to be usable in the model thread > (where the bridge lives). > > This should unblock the migration of PASSWORDS to pseudo-USS, because > password data must be encrypted at all times. This is a first step of > a proposal described in a dedicated section in the Design Doc: > https://docs.google.com/document/d/14ScYZ0sop921gjBwXuReIEuQJlwftqkuSM1jMK_A_x4/edit#heading=h.ev0xr4j8pkot > > Actual usage of this functionality will be introduced in follow-up > patches. > > Bug: 870624 > Change-Id: I227d429dc952bfe1a3a4fb05cd1ab71cac9ba1c6 > Reviewed-on: https://chromium-review.googlesource.com/c/1288594 > Commit-Queue: Mikel Astiz <mastiz@chromium.org> > Reviewed-by: Marc Treib <treib@chromium.org> > Reviewed-by: Mohamed Amir Yosef <mamir@chromium.org> > Cr-Commit-Position: refs/heads/master@{#601254} TBR=treib@chromium.org,mastiz@chromium.org,mamir@chromium.org Change-Id: Iab6463f9541ca242facdc8c5217f1cf4c114ba49 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 870624 Reviewed-on: https://chromium-review.googlesource.com/c/1292512Reviewed-by:
-
