- 18 Sep, 2018 40 commits
-
-
Brian White authored
There are reports from the field that the "maximum" of a histogram does not match what was configured. The working theory is that it is due to rounding errors in the creation of exponential buckets. Verify this assumption. If it's behaving wrong, it should crash during the creation. Bug: 836238 Change-Id: I4cf5e4bc3986e288c728ae37e2ff8cf841b7598d Reviewed-on: https://chromium-review.googlesource.com/1228840Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#592128}
-
Oystein Eftevaag authored
This is a reland of 32c8ebbe Relanding after https://chromium-review.googlesource.com/c/chromium/src/+/1225306 Original change's description: > Enable Perfetto by default for all telemetry tests > > This will use Perfetto instead of TraceLog for all > trace-based Telemetry tests running on Mac/Windows/Android. > > R=nednguyen@google.com > > Bug: 839084 > Cq-Include-Trybots: master.tryserver.chromium.perf:obbs_fyi > Change-Id: I570060cd7f53ea0310f67370b4abec132c8a1a01 > Reviewed-on: https://chromium-review.googlesource.com/1176119 > Commit-Queue: oysteine <oysteine@chromium.org> > Reviewed-by: Sami Kyöstilä <skyostil@chromium.org> > Reviewed-by: Ned Nguyen <nednguyen@google.com> > Cr-Commit-Position: refs/heads/master@{#585368} TBR=nednguyen@google.com Change-Id: I31b69483281f9fb34c5083888b8f45f00ca95e6e Reviewed-on: https://chromium-review.googlesource.com/1208955Reviewed-by:
oysteine <oysteine@chromium.org> Reviewed-by:
Annie Sullivan <sullivan@chromium.org> Commit-Queue: oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#592127}
-
Michael Spang authored
Since 57a19db6 ("[fuchsia] Whitelist required Fuchsia services in component manifest.") vulkan fails to initialize. Fix it by adding the vulkan loader service to the policy. Bug: 861853 Test: content_shell --ozone-platform=scenic --enable-vulkan --disable-gpu Change-Id: I38c11f32e0a0dd8b914184a16198c023877a0572 Reviewed-on: https://chromium-review.googlesource.com/1228488Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#592126}
-
Christopher Cameron authored
Add a BridgeFactoryHost structure, which wraps a views_bridge_mac::mojom::BridgeFactory interface. This structure has an BridgeFactoryHost::Observer interface to call back into widgets when the bridge factory they are using is destroyed (e.g, the app shim is force-quit). Bug: 859152 Change-Id: I0c34cf200588ac79d281b31ba85764bca0c02d39 Reviewed-on: https://chromium-review.googlesource.com/1229694 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#592125}
-
Wei Li authored
When Chrome's fullscreen toolbar style changes, it needs to re-layout to make sure the UI is updated. This CL fixed a recent regression due to using |is_exiting_fullscreen| to avoid re-layout. Instead, only when Chrome exits fullscreen mode to non-fullscreen mode, we should avoid doing re-layout because exiting tab fullscreen to browser fullscreen still may requires a re-layout. BUG=884029 Change-Id: I7392c2cb000b184c0f54b9cb6bb3de2b81a79dcf Reviewed-on: https://chromium-review.googlesource.com/1226014 Commit-Queue: Wei Li <weili@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Sidney San Martín <sdy@chromium.org> Cr-Commit-Position: refs/heads/master@{#592124}
-
Ken MacKay authored
Use the correct config for the audio decoder, and fix a hang due to AV sync on desktop machines. Change-Id: Icb8f1e73d093e58f81a68dd00d8512654c1140d1 Reviewed-on: https://chromium-review.googlesource.com/1229933Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Kenneth MacKay <kmackay@chromium.org> Cr-Commit-Position: refs/heads/master@{#592123}
-
Tommy C. Li authored
This CL continues implementation of the Query in Omnibox feature for Desktop with two main changes: 1. Shows the Default Search Provider favicon in the Omnibox when we are displaying search terms. 2. Don't (erroneously) try to do unelision for Steady State Elisions when we are displaying search terms. Adds a test for that. This also adds a convenience method to OmniboxEditModel to prevent repeated code, and adds a bit of testing code for the convenience method. Bug: 874592 Change-Id: Iafd791db349df434f616697ecbe6122285b5351b Reviewed-on: https://chromium-review.googlesource.com/1229493 Commit-Queue: Tommy Li <tommycli@chromium.org> Reviewed-by:Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#592122}
-
Moe Ahmadi authored
Fixes the crash in ConfirmInfoBarController and potentially other infobar controllers after the refactoring of infobars in crrev.com/c/1037640. First attempt at fixing this crash: crrev.com/c/1161324 Bug: 880951 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: I14cef2eabf0293221de4ea31ac2ea18d4c1522b0 Reviewed-on: https://chromium-review.googlesource.com/1213546 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Reviewed-by:
Peter Lee <pkl@chromium.org> Cr-Commit-Position: refs/heads/master@{#592121}
-
Brian White authored
This functionality is used by the FileMetricsProvider, currently based on a Feature to be enabled. Bug: 882892 Change-Id: I55262453b58c5ed9473b33e8839bad423ca7ec84 Reviewed-on: https://chromium-review.googlesource.com/1220310 Commit-Queue: Brian White <bcwhite@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#592120}
-
Weidong Guo authored
Changes: 1. Enable blur for suggestion chip and use a mask layer to clip into rounded rect. 2. Enable blur for folder icon and use a mask layer to clip into circle. 3. Since anyway we have to use mask layer for the blur, we don't need rely on Skia to draw clipped folder icon during the folder icon animation. Instead, we use mask layer as well. 4. Opening assistant will recreate layers in app list (bug 879372). Since Layer::Clone() does not copy mask layer (not owned), the mask layer set before will get lost. So override LayerOwner::RecreateLayer() to set mask layer after cloning. Bug: 873288 Change-Id: I7e7ff23cf38bfaece9799810e398c76bd38f8b68 Reviewed-on: https://chromium-review.googlesource.com/1171546 Commit-Queue: Weidong Guo <weidongg@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#592119}
-
Devlin Cronin authored
When the user selects "On specific sites" from the dropdown in the chrome://extensions page, automatically bring up the dialog to add an allowed site. This makes it clear to the user that they need to specify a site for this setting to work, and also fixes an issue where selecting "On specific sites" and then refreshing the page (or having the data model otherwise update) reset the value to "on click". Bug: 884222 Change-Id: Ie89757f7fba2409774e5fcc2f4a8028aff31f7ba Reviewed-on: https://chromium-review.googlesource.com/1226017Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#592118}
-
rbpotter authored
Match the Print Preview elevation shadow styling to the styling used in other Web UI surfaces (extensions, settings). Bug: 883583 Change-Id: Ida1c4427e401de30c540c5109eb5b45e6b67f79f Reviewed-on: https://chromium-review.googlesource.com/1227395Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#592117}
-
Nate Chapin authored
This allows us to hide the HashSet of all current WorkerThreads, which is valuable since the callers outside WorkerThread don't have access to the necessary mutex to access it safely. Convert TimeZoneMonitorClient to use this helper. Change-Id: I5aa743a1fa0c789a8f9496a05d7cfecde9238004 Reviewed-on: https://chromium-review.googlesource.com/1228950Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Reviewed-by:
Mark Mentovai <mark@chromium.org> Commit-Queue: Nate Chapin <japhet@chromium.org> Cr-Commit-Position: refs/heads/master@{#592116}
-
Richard Townsend authored
When colgroups are modified via Javascript, Blink uses a stale computed logical width, resulting in incorrect sizing of auto columns if layout was previously done. This patch forces a fresh layout of the affected cells, which corrects this behaviour. Bug: 870296 Change-Id: I4a12d5a8db689ba1457c6c9960ae8ea9c1dd40d9 Reviewed-on: https://chromium-review.googlesource.com/1162224 Commit-Queue: David Grogan <dgrogan@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#592115}
-
Ehsan Karamad authored
This is a second attempt at fixing the crash linked in the bug. It appears that PluginResourceThrottle's lifetime is tied with WebURLLoaderImpl does not synchronously go away with the container. When an <embed> element is removed during a resource request, the MHVCBase (aka WebAssociatedURLLoaderClient in this context) is destroyed immediately. The WebURLLoaderImpl, OTH, goes away when the cancel timer on the ResourceClient is fired. To avoid any potential UaF, this CL changes the raw pointer refernece to MHVCBase in PluginResourceThrottle to a WeakPtr. This also serves as a potential fix for the crashes in the linked bug. Bug: 878359 Change-Id: If8b21f9366bb57394819946e891f9288028dde2c Reviewed-on: https://chromium-review.googlesource.com/1230576Reviewed-by:
James MacLean <wjmaclean@chromium.org> Reviewed-by:
Ehsan Karamad <ekaramad@chromium.org> Commit-Queue: Ehsan Karamad <ekaramad@chromium.org> Cr-Commit-Position: refs/heads/master@{#592114}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/debba6d2124e..6fc8d877d745 git log debba6d2124e..6fc8d877d745 --date=short --no-merges --format='%ad %ae %s' 2018-09-18 hnakashima@chromium.org Encode mark names when writing to file. Created with: gclient setdep -r src/third_party/pdfium@6fc8d877d745 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: I83b30a33d09b95a8b76523c2a5f455670d30dd66 Reviewed-on: https://chromium-review.googlesource.com/1231413Reviewed-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@{#592113}
-
Nico Weber authored
win: Remove two exit-time destructors from pdf_child_init and enable -Wexit-time-destructors for chrome/child. The PDF setup code patches up two GDI functions. There's no need to undo that patching at process exit, so use a function-local static base::NoDestructor<> instead of a global. Bug: 404525 Change-Id: I9cb06b857a8b95ba5cbac74be60f8225fbb6f72a Reviewed-on: https://chromium-review.googlesource.com/1231073Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#592112}
-
Nico Weber authored
No need to undo 5 IAT patches at process shutdown, and no need to call a ThreadLocalBoolean dtor either. Bug: 404525 Change-Id: I95e4cc24d7b6ca7171457a0da8dd6b3231fe45a8 Reviewed-on: https://chromium-review.googlesource.com/1231153Reviewed-by:
Antoine Labour <piman@chromium.org> Reviewed-by:
Scott Graham <scottmg@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#592111}
-
Michael Spang authored
Fuchsia platform windows are initially 0x0 and this causes vulkan surface creation to crash. Defer creating a renderer until we have a nonempty size. Bug: 861853 Test: run_ozone_demo --ozone-platform=scenic --enable-vulkan Change-Id: I74e14ed07ab47ebc22730b40cf36217e9ae4873a Reviewed-on: https://chromium-review.googlesource.com/1229162 Commit-Queue: Michael Spang <spang@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#592110}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d89bb9de..345f5513 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,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I7f2c4436587c174c0640ad366c7439cc5a5b98ad Reviewed-on: https://chromium-review.googlesource.com/1231293Reviewed-by:
V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#592109}
-
edchin authored
This CL simply adds the flag so that it can be used in a future CL. This flag controls whether the WKWebView snapshotting API is used to take snapshots for web pages (not native pages) on iOS 11+. The crbug to remove the flag is crbug.com/885003. Bug: 739840 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Id00c4da93f5046e4a2b2af179abe3794a99a248e Reviewed-on: https://chromium-review.googlesource.com/1229703 Commit-Queue: edchin <edchin@chromium.org> Reviewed-by:
edchin <edchin@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Cr-Commit-Position: refs/heads/master@{#592108}
-
Mike Wasserman authored
Track in-flight key events separately from others in WindowTree. (client may ack key events out-of-order from others, for async IME) Add a simple test of OnWindowInputEventAck behavior. Bug: 884368 Test: Moving a mouse while pressing an accelerator with KSV window focused. Change-Id: I7d76ce1aec2ef311f5e721903d40e1b3819e2953 Reviewed-on: https://chromium-review.googlesource.com/1229074 Commit-Queue: Michael Wasserman <msw@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#592107}
-
Zhenyao Mo authored
The original sample is outdated so replace with a newer one. BUG=838970 Change-Id: Id257239e3fdb6424bd7bb9b3bdaea563dd1743dc Reviewed-on: https://chromium-review.googlesource.com/1228783 Commit-Queue: Zhenyao Mo <zmo@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#592106}
-
S. Ganesh authored
Since the tests introduced in install_service_work_item_unittest.cc could interfere with one another, I am also changing the setup unittests to tests in series rather than in parallel by switching to LaunchUnitTestsSerially in run_all_unittests.cc. Bug: 833687 Change-Id: I215feac65799c0278bf3503e929f3415e6d0ff36 Reviewed-on: https://chromium-review.googlesource.com/1137532 Commit-Queue: S. Ganesh <ganesh@chromium.org> Reviewed-by:
Sorin Jianu <sorin@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#592105}
-
Tommy Steimel authored
This CL removes the double-tap AnimatedArrows from the overlay play button and adds them into a new MediaControlAnimatedArrowContainer element. This is the first step of a refactoring of the overlay play button to remove non-play-button-related logic and elements. Bug: 884770 Change-Id: I4d247a0178fd353b294d70c510b9b5b230220c68 Reviewed-on: https://chromium-review.googlesource.com/1228984Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#592104}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/0639cbc44de0..804797362e1d git log 0639cbc44de0..804797362e1d --date=short --no-merges --format='%ad %ae %s' 2018-09-18 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@804797362e1d The AutoRoll server is located here: https://autoroll.skia.org/r/depot-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=agable@chromium.org Change-Id: I0894a56fb702f036390a51ba3db0dc8f3fdfabf8 Reviewed-on: https://chromium-review.googlesource.com/1230640Reviewed-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@{#592103}
-
Theresa authored
Default enable the simplified NTP ablation experiment that removes the bookmarks/downlaods sections on the NTP without replacing them with short buttons. BUG=865216 Change-Id: Ia77dd888db8541f0946788c8c4fc6c4763da6874 Reviewed-on: https://chromium-review.googlesource.com/1229313 Commit-Queue: Theresa <twellington@chromium.org> Reviewed-by:
Becky Zhou <huayinz@chromium.org> Cr-Commit-Position: refs/heads/master@{#592102}
-
Saman Sami authored
Resetting the fallback is no longer necessary and might even result in surfaces outside of the desired range being shown. Bug: 857575 Change-Id: Ib66ad50d0297f2c93e3e9815a3ec2349fde4289b Reviewed-on: https://chromium-review.googlesource.com/1231034Reviewed-by:
kylechar <kylechar@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Commit-Queue: Saman Sami <samans@chromium.org> Cr-Commit-Position: refs/heads/master@{#592101}
-
Lei Zhang authored
Rename the static method to CreateUniqueId(). Mark more parts of CupsPrintJob as const as well. Change-Id: Ifd4cd5d27c71245aa9ad251addf915853b48f6a2 Reviewed-on: https://chromium-review.googlesource.com/1229369Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#592100}
-
Robbie McElrath authored
This is a follow up from a post-merge comment on crrev.com/c/1214081 which moves the SetUp code to the constructor. Bug: 868015 Change-Id: Ied0bd568560c49858eeb5261c5d89352f8b1d787 Reviewed-on: https://chromium-review.googlesource.com/1229356Reviewed-by:
David Roger <droger@chromium.org> Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#592099}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/1d6281d4bb47..0191ed8e87dc git log 1d6281d4bb47..0191ed8e87dc --date=short --no-merges --format='%ad %ae %s' 2018-09-18 mtklein@google.com Make listener lists threadsafe with a mutex. 2018-09-18 bsalomon@google.com Fix image_from_yuv_textures with --preAbandonContext Created with: gclient setdep -r src/third_party/skia@0191ed8e87dc The AutoRoll server is located here: https://autoroll.skia.org/r/skia-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=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=reed@chromium.org Change-Id: Ia9c7909443e6e1be2a567f0fb6de73a525569fe2 Reviewed-on: https://chromium-review.googlesource.com/1231275Reviewed-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@{#592098}
-
liberato@chromium.org authored
This addresses the CFI failures that caused the original revert. This reverts commit 45cb1008. Change-Id: I05fa7fbb8628c992639e960d9d3f41c6ad5fec2b Reviewed-on: https://chromium-review.googlesource.com/1228493 Commit-Queue: Frank Liberato <liberato@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#592097}
-
Martin Robinson authored
This allows accessibility clients to have access to the links available in the text provided by the AtkText interface. Implementing this interface also allows us to write a fully-fleshed out text for the contents provided by the AtkText interface. Bug: 866337 Change-Id: I38e200512a835014922b51df677203e853d6ef4b Reviewed-on: https://chromium-review.googlesource.com/1227942 Commit-Queue: Martin Robinson <mrobinson@igalia.com> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#592096}
-
Asanka Herath authored
Test is flaky. Bug: 884140 Change-Id: I41f4f3eb084edd56867ffecc437817be8f1bcc1b Reviewed-on: https://chromium-review.googlesource.com/1231037Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Asanka Herath <asanka@chromium.org> Cr-Commit-Position: refs/heads/master@{#592095}
-
Etienne Pierre-Doray authored
This CL creates a feature kCookieStorePriorityBoost that changes SQLitePersistentCookieStore background_task_runner sequence priority to USER_BLOCKING. A CANARY_DEV_50 experiment will be launch and used to compare SessionRestore.ForegroundTabFirstPaint3. Bug: 878222 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I22bbd5b90cf4d36d3a858adbe94eff96f280a5a0 Reviewed-on: https://chromium-review.googlesource.com/1224671 Commit-Queue: Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
Maks Orlovich <morlovich@chromium.org> Reviewed-by:
Jialiu Lin <jialiul@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#592094}
-
Xiaohui Chen authored
Bug: b:115918259 Test: locally build Change-Id: Ie1da988afddfbe446cb5860ceb799a9814927116 Reviewed-on: https://chromium-review.googlesource.com/1229184 Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Muyuan Li <muyuanli@chromium.org> Cr-Commit-Position: refs/heads/master@{#592093}
-
Elly Fong-Jones authored
This reverts commit 2065e9fc. Reason for revert: hangs Flash context menus in fullscreen, https://crbug.com/884158 Original change's description: > content: don't show context menus for invisible WebContents > > Bug: 855127 > Change-Id: I2942e122373c210bd0bceaa59791e2619186f79b > Reviewed-on: https://chromium-review.googlesource.com/1175935 > Reviewed-by: Avi Drissman <avi@chromium.org> > Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> > Cr-Commit-Position: refs/heads/master@{#583317} TBR=ellyjones@chromium.org,avi@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 855127 Change-Id: Ic1f62597bfbebe43cd5002ec12f05e36562e8c96 Reviewed-on: https://chromium-review.googlesource.com/1230439Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#592092}
-
Andrey Kosyakov authored
This removes DebugDevToolsInterceptor and supports DEBUG_DEVTOOLS layout as part of regular DevToolsDataSource. Change-Id: I06d5a805ff8b6b324127b131bb87801d3595311f Reviewed-on: https://chromium-review.googlesource.com/1229198Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#592091}
-
Oystein Eftevaag authored
This bundles multiple events into the same higher-level message and adds a string table implementation for frequently repeated strings, on a per-thread basis; specifically the event name, category name, and the name of its arguments. Requires https://android-review.googlesource.com/c/platform/external/perfetto/+/755290 to roll into //third_party/perfetto before landing. R=primiano@chromium.org,skyostil@chromium.org Bug: 839116 Change-Id: Id2681970db5b7ea5cadf0ee9bbcbbc1ebbba08c3 Reviewed-on: https://chromium-review.googlesource.com/1225306 Commit-Queue: oysteine <oysteine@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Cr-Commit-Position: refs/heads/master@{#592090}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/95277a300f52..0fdb956d9c24 git log 95277a300f52..0fdb956d9c24 --date=short --no-merges --format='%ad %ae %s' 2018-09-18 jmadill@chromium.org Re-land "Inline and micro-optimize more for perf tests."" Created with: gclient setdep -r src/third_party/angle@0fdb956d9c24 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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: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@chromium.org Change-Id: Iab9b61a4693af205a33d004a1a1a6685f9fb9a61 Reviewed-on: https://chromium-review.googlesource.com/1231213Reviewed-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@{#592089}
-