- 08 Jun, 2018 40 commits
-
-
Wez authored
This is required to allow us to roll to a Fuchsia SDK revision with the new paths, update Chromium dependencies, and then this extra path will be removed. TBR: sergeyu Change-Id: Ib4c7b8f874a8fdc5bbac85f0cd7c06e505002829 Reviewed-on: https://chromium-review.googlesource.com/1092312Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#565579}
-
Rune Lillesveen authored
Change-Id: Iabd0565de1e044290a444e734a6d79ccb1c20faa Reviewed-on: https://chromium-review.googlesource.com/1092490Reviewed-by:
Anders Ruud <andruud@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#565578}
-
Yunfan Chen authored
This change overrides the bounds change reason to be snapped over drag_move. Otherwise, when drag a window into snapped mode, the drag to reposition reason will override the snap reason for bound change. The server will then send the incorrect reason to ARC++ side. With this change, ARC++ N and P will correctly turn into snapped mode when drag a window to side. Bug: 848626 Change-Id: I0174ea36166ad378f2ba6d935d3ab32b5ece98ae Reviewed-on: https://chromium-review.googlesource.com/1084574 Commit-Queue: Yunfan Chen <yunfanc@chromium.org> Reviewed-by:
Stefan Kuhne <skuhne@chromium.org> Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
David Reveman <reveman@chromium.org> Reviewed-by:
Xiaoqian Dai <xdai@chromium.org> Cr-Commit-Position: refs/heads/master@{#565577}
-
Friedrich Horschig authored
This reverts commit 738c6fa6. Reason for revert: Failed Linux Chromium OS ASan LSan Tests (1) every time since being enabled. First bad build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/27807 Original change's description: > Enable NoBackgroundTasksTest.FirstNonEmptyPaintWithoutBackgroundTasks on ChromeOS. > > The test failed because rendering the NTP was blocked on loading > machine statistics, which was done from a TaskPriority::BACKGROUND > task. The priority of that task was changed to > TaskPriority::USER_BLOCKING in > https://chromium-review.googlesource.com/c/chromium/src/+/1082608, so > the test can be enabled. > > The test is still disabled when Mash is enabled, because the > non-empty paint signal isn't fired in this case (even when background > tasks aren't disabled). > > Bug: 831835 > Change-Id: Ie847b152b057883f62bef060c6294a427980c838 > Reviewed-on: https://chromium-review.googlesource.com/1082609 > Reviewed-by: Avi Drissman <avi@chromium.org> > Commit-Queue: François Doray <fdoray@chromium.org> > Cr-Commit-Position: refs/heads/master@{#565421} TBR=avi@chromium.org,fdoray@chromium.org Change-Id: I7f045845726b0075c639f4708ff183a741d60a1e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 831835 Reviewed-on: https://chromium-review.googlesource.com/1092530Reviewed-by:
Friedrich Horschig <fhorschig@chromium.org> Commit-Queue: Friedrich Horschig <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#565576}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/1c06e11..e4d73cb git log 1c06e11..e4d73cb --date=short --no-merges --format='%ad %ae %s' 2018-06-08 rharrison@chromium.org Update comments in SUPPRESSIONS for xfa_specific Created with: gclient setdep -r src/third_party/pdfium@e4d73cb The AutoRoll server is located here: https://pdfium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: Ibb595c2cec9bf8824aa3c1b93f43e67fa9fe0989 Reviewed-on: https://chromium-review.googlesource.com/1092100Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#565575}
-
Maksim Sisov authored
Make sure forward declaration is not used to just avoid declaring dependencies in the common/BUILD.gn. That is, DrmDevice is a drm platform's class, which shouldn't be known by the ozone/common. To fix this, return GbmDeviceLinux instead and add AsGbmDeviceLinux method to the DrmDevice class to return downcasted pointer to self, and move ScanoutBufferGenerator to drm/gpu as long as it's not needed by the Wayland with dmabuf approach at this stage. Bug: 820047 Change-Id: I25703db41234275e0c0cb5aaefe0e61964a50e83 Reviewed-on: https://chromium-review.googlesource.com/1088690Reviewed-by:
Michael Spang <spang@chromium.org> Commit-Queue: Maksim Sisov <msisov@igalia.com> Cr-Commit-Position: refs/heads/master@{#565574}
-
Hitoshi Yoshida authored
Web IDL spec allows hyphen since August 2014. https://heycam.github.io/webidl/#idl-grammar This CL follows the update. Bug: None Change-Id: I695eed71417b524755b980f3efed5107bc7d29bf Reviewed-on: https://chromium-review.googlesource.com/1092219Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#565573}
-
Xiaohan Wang authored
These tests are flaky on this bot. Disable them for now so that we do not block the CQ. Bug: 793426 Test: Disable tests Change-Id: I12cd105bd2c57a44156a04e33f1a456f4492a996 Reviewed-on: https://chromium-review.googlesource.com/1092311Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#565572}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/d993e197..739f5abc Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: I7260a9a26406022cdfd1e4c0908da26996e391f3 Reviewed-on: https://chromium-review.googlesource.com/1091952Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#565571}
-
spqchan authored
To make the BrowserWindowTouchBar available to MacViews, modify the class so that it no longer depends on Cocoa code to receive updates for the touch bar item states. This CL converts BookmarkTabHelperDelegate to BookmarkTabHelperObserver so that BrowserWindowTouchBar can observe the current web contents for starred changes. Overview CL: https://chromium-review.googlesource.com/c/chromium/src/+/1086867 Bug: 841714 Change-Id: Id8944ad085d87288bcda767a53bb1abfe5c0eab9 Reviewed-on: https://chromium-review.googlesource.com/1090315 Commit-Queue: Sarah Chan <spqchan@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#565570}
-
Lei Zhang authored
Change-Id: I022e4dcfa9bf34de7c48241f3c2e225b6b15c7dc Reviewed-on: https://chromium-review.googlesource.com/1087517 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Cr-Commit-Position: refs/heads/master@{#565569}
-
Sasha Morrissey authored
Add OWNERS in file://ui/file_manager/OWNERS to drive_internals page, since this is mainly used for debugging issues with the Drive Sync engine which is primarly used by the file manager. Bug: 845065 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Iff904ccc44c163621f620fa930e838697f4f8f1e Reviewed-on: https://chromium-review.googlesource.com/1075837Reviewed-by:
Satoru Takabayashi <satorux@google.com> Reviewed-by:
Satoru Takabayashi <satorux@chromium.org> Reviewed-by:
Kazuhiro Inaba <kinaba@chromium.org> Reviewed-by:
Ryo Hashimoto <hashimoto@chromium.org> Commit-Queue: Sasha Morrissey <sashab@chromium.org> Cr-Commit-Position: refs/heads/master@{#565568}
-
sangwoo.ko authored
When dragging ends, an animation for tabs runs. If ideal bounds for tabs changes when the animation is running, We should restart animation with newly calculated bounds. Bug: 848016 Change-Id: I1b5fff8d3bcd02fbe2d39f847c77e9c5df3c01c2 Reviewed-on: https://chromium-review.googlesource.com/1084521 Commit-Queue: Sang Woo Ko <sangwoo108@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#565567}
-
Pavel Feldman authored
Change-Id: I4af521482afdd48a2ca1c1f07496e38a297f7770 Reviewed-on: https://chromium-review.googlesource.com/1092104Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#565566}
-
Lei Zhang authored
They have no overrides. Change-Id: I8bb69c0a0d09f45d29db480d23f665fad6c02a4f Reviewed-on: https://chromium-review.googlesource.com/1090232 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#565565}
-
Bence Béky authored
Do not fail gracefully in WebSocketStreamRequestImpl::PerformUpgrade(). This reverts https://crrev.com/c/1070327, which was only landed so that crash is temporarily suppressed on Beta and Stable. This CL will cause crashes in WebSocketStreamRequestImpl::PerformUpgrade() or in other places, which then can be investigated by the usual means (peppering more CHECKs in the code, code inspections, etc.) and the underlying cause can be solved. Bug: 850183 Change-Id: I7b7b8a1d1b37d0881cc68bfac278c45f38bbaae8 Reviewed-on: https://chromium-review.googlesource.com/1091134 Commit-Queue: Adam Rice <ricea@chromium.org> Reviewed-by:
Adam Rice <ricea@chromium.org> Cr-Commit-Position: refs/heads/master@{#565564}
-
Keishi Hattori authored
Calling CompleteSweep from SetGCState can cause confusion as to when a lazy sweep finishes. Refactors CompleteSweep out of SetGCState and moves it to its callers. Change-Id: I0512c642da92800f07a0b8eab9abb0a2668a51aa Reviewed-on: https://chromium-review.googlesource.com/1090526 Commit-Queue: Keishi Hattori <keishi@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#565563}
-
Yoichi Osato authored
This patch rewrites the layout test with selection_test inside contenteditable. Bug: 679977, 708452 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: Iff922bbff0dd0835a9e648c32a1ef6b67e404660 Reviewed-on: https://chromium-review.googlesource.com/1086822 Commit-Queue: Yoichi Osato <yoichio@chromium.org> Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#565562}
-
Amos Lim authored
Group overrides and match function definitions and declartions order. Change QueryPolicy to take an int argument instead of a net::CertStatus. Bug: 840045 Change-Id: I6aa960015ffc2f81ab73957c2e14451143e6f8df Reviewed-on: https://chromium-review.googlesource.com/1065955Reviewed-by:
Richard Coles <torne@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Emily Stark <estark@chromium.org> Commit-Queue: Amos Lim <eui-sang.lim@samsung.com> Cr-Commit-Position: refs/heads/master@{#565561}
-
Amos Lim authored
This CL moves the audio-specific files from //content/renderer/media to //content/renderer/media/audio. No new code, just moving files. $ mkdir audio $ git mv audio_* audio $ git mv mock_audio_* audio $ git mv mojo_audio_* audio $ tools/git/mass-rename.py NOPRESUBMIT=true Bug: 739232 Change-Id: Ifadc4b5c0b2b4518405112d47614832684b8b7a8 Reviewed-on: https://chromium-review.googlesource.com/1036824 Commit-Queue: Amos Lim <eui-sang.lim@samsung.com> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Bill Budge <bbudge@chromium.org> Reviewed-by:
Henrik Grunell <grunell@chromium.org> Cr-Commit-Position: refs/heads/master@{#565560}
-
Sadrul Habib Chowdhury authored
Only SCROLL_UPDATE and FIRST_SCROLL_UPDATE components currently have non-zero component ids. However, these component-ids are never used for any purpose. So use zero as the component-id for these components. This allows for the following cleanups: . RenderWidgetHostLatencyTracker no longer needs latency_component_id_. . Rename RenderWidgetHostImpl::GetLatencyComponentId() to a clearer name: GetFrameSinkIdForSnapshot(). BUG=849729 Change-Id: I7fea87d5cf1b06c580b6738816ffeb893063b1fb Reviewed-on: https://chromium-review.googlesource.com/1089715 Commit-Queue: Sadrul Chowdhury <sadrul@chromium.org> Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Cr-Commit-Position: refs/heads/master@{#565559}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=amineer@chromium.org Change-Id: Ic4063923607a16b369741c1cb43fcdc1aad583d9 Reviewed-on: https://chromium-review.googlesource.com/1092430Reviewed-by:
chrome-release-bot@chromium.org <chrome-release-bot@chromium.org> Cr-Commit-Position: refs/heads/master@{#565558}
-
Michael Lippautz authored
Bug: chromium:840789 Change-Id: Ic73ce15ac2057d4ed6fec89d3fc74c8bf32909ae Reviewed-on: https://chromium-review.googlesource.com/1092210 Commit-Queue: Michael Lippautz <mlippautz@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#565557}
-
Stefan Zager authored
Prior to this patch, a rect-based hit test took as parameters a center point and four padding values for top, right, bottom, and left padding. Theoretically, a rect-based hit test might have asymmetric padding, but in practice, this never happens in the code (i.e., it's always true that top-padding == bottom-padding and left-padding == right-padding). Additionally, point-based hit tests used the same call path as rect-based hit tests (with zero padding), which made the code hard to follow. Finally, there was a bug in the rect-based hit test implementation: point-based hit tests always create a one-pixel rect with the hit test point at top left and width=height=1. Because rect-based hit tests used the same code path, the hit test rect was always erroneously expanded by one pixel to the right and one pixel down. This patch makes an explicit distinction between point-based and rect-based hit tests; it gets rid of center-point-and-padding in favor of a simple rect; and it fixes the one-pixel-padding bug. BUG=827639 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I11e49da328080c03da2e1662cbcc0317ab409740 Reviewed-on: https://chromium-review.googlesource.com/1081331Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#565556}
-
Xiaohan Wang authored
This reverts commit 0c5ac919. The test has been fixed in http://crrev.com/c/1090113. Bug: 608541 Change-Id: I040328033921d8b24018761d19ecd923d44d75bf Reviewed-on: https://chromium-review.googlesource.com/1091883Reviewed-by:
Frank Liberato <liberato@chromium.org> Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#565555}
-
Lei Zhang authored
As is, it calls RenderProcessHost::SetMaxRendererProcessCount(), which overrides a global variable. Since it does not cleanup after itself, any other subsequent test cases are stuck with its override. RenderProcessHostUnitTest, which is sensitive to the override, has to call SetMaxRendererProcessCount(0) to compensate. BUG=850541 Change-Id: I0351d2bd709e2ba850ee47b5ed9a503903b3f5f8 Reviewed-on: https://chromium-review.googlesource.com/1088316Reviewed-by:
Charlie Reis <creis@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#565554}
-
Lei Zhang authored
Currently it takes an array of IDs, an offset into the array, and a count. This CL changes the method to just take a list of IDs, so its implementation is more straight forward. Callers that used to keep track of offsets into a fixed array now does the work of splitting the array into smaller chunks. BUG=769630 Change-Id: I7661bac4a965ce0734ebc5e89828b6b10b11572a Reviewed-on: https://chromium-review.googlesource.com/1080219 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Cr-Commit-Position: refs/heads/master@{#565553}
-
Lei Zhang authored
Remove the |max_size| parameter, which is now always 0. BUG=769630 Change-Id: Id44b41f4c783119aa9069f474e47e567e5cda6a2 Reviewed-on: https://chromium-review.googlesource.com/1090222 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#565552}
-
Katie D authored
Per UI review, we only want to show the relevant languages to the user by default to reduce page clutter. Tested by using a brand new profile and seeing that the English languages show up, and by adding "German" as another preferred language and seeing that both English and German show up in the preferred languages list. Bug: 823359 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I19b264b8b92921135acf0c2064e7a8bd9c8f881f Reviewed-on: https://chromium-review.googlesource.com/1074329 Commit-Queue: Katie Dektar <katie@chromium.org> Reviewed-by:
David Tseng <dtseng@chromium.org> Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Cr-Commit-Position: refs/heads/master@{#565551}
-
Jeremy Roman authored
Use base/sys_byteorder.h instead. Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I904fec77fbf470f72287fb49a993de9013c9b758 Reviewed-on: https://chromium-review.googlesource.com/1090990 Commit-Queue: Yuta Kitamura <yutak@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Yuta Kitamura <yutak@chromium.org> Cr-Commit-Position: refs/heads/master@{#565550}
-
Pedro Amaral authored
This CL does 3 things: 1) Make BottomToolbarController be initialized in ToolbarManager instead of ChromeActivity. 2) Make an MVC component for the tab switcher button 3) Add the omnibox focus, tab switcher button, and menu buttons to the bottom toolbar. I opted to keep everything in one CL for now to give context. For example I moved the BottomToolbarController initialization because the ToolbarManager has the necessary information that is passed down to the TabSwitcherCoordinator. Let me know if you want this split up. A couple things are missing from this CL: 1) No incognito or theming. The bottom toolbar doesn't do these things yet so the buttons also won't. 2) Support for memex. The logic for triggering memex is spread in ToolbarPhone and I didn't want to have to move it to ToolbarManager or to have ToolbarPhone own BottomToolbarController. 3) There is currently no way of switching bottom buttons using the keyboard. 4) The top toolbar tab switcher button and menu button are still being shown. Eventually we'd probably need to do something with a cached feature. 5) When we enter tab switcher mode the omnibox focus button is still present even though it doesn't do anything. Not sure if we want to hide it. 6) Long-pressing the omnibox focus button doesn't show an accessibility toast. Not sure if we want this but the tab switcher button has it. 7) The app menu update badge is missing. Change-Id: I4aa6f6ca19d18cf32b8d6e4771e86971426fa2d7 Reviewed-on: https://chromium-review.googlesource.com/1068079 Commit-Queue: Pedro Amaral <amaralp@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#565549}
-
Justin Cohen authored
Add -Wno-null-pointer-arithmetic and -Wno-unused-lambda-capture. Correct memory order argument to atomic operation error. Bug: 850333, 843234 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I34647764ebc03adfa865c32c298e24a0c1f60359 Reviewed-on: https://chromium-review.googlesource.com/1090151Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#565548}
-
Hiroshige Hayashizaki authored
To match the behavior with the spec. Bug: 686281 Change-Id: I7aef8035083d1d5c12f91be597c580eb90833fdf Reviewed-on: https://chromium-review.googlesource.com/1091535Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Cr-Commit-Position: refs/heads/master@{#565547}
-
Richard authored
To enhance readability of code, put |version| in SWProviderHost:: GetOrCreateSWHandle() under management by scoped_refptr. Change-Id: I15c6a1d706a1b8176a077909b352726cb0813085 Reviewed-on: https://chromium-review.googlesource.com/1086724Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Han Leon <leon.han@intel.com> Commit-Queue: Han Leon <leon.han@intel.com> Cr-Commit-Position: refs/heads/master@{#565546}
-
Siddhartha authored
BUG=843419 Change-Id: I76862b50d57c845a4f4e0674dab579c45504488f Reviewed-on: https://chromium-review.googlesource.com/1087839 Commit-Queue: Siddhartha S <ssid@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Cr-Commit-Position: refs/heads/master@{#565545}
-
Lei Zhang authored
This is what the information the caller really wants to know about directories on MTP devices before deleting them. By implementing this, the caller no longer have to use MTPDeviceTaskHelper::ReadDirectory(). As a result, ReadDirectory() can be simplified in a follow-up CL. BUG=769630 Change-Id: I29c1b40f7682cb68c271ea4817fbf32c776f33ab Reviewed-on: https://chromium-review.googlesource.com/1090221 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by:
Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#565544}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/09b82f4..c971895 git log 09b82f4..c971895 --date=short --no-merges --format='%ad %ae %s' 2018-06-07 dtu@chromium.org [dashboard] Add mojo-linux-perf to Pinpoint. Created with: gclient setdep -r src/third_party/catapult@c971895 The AutoRoll server is located here: https://catapult-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. 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 BUG=chromium:848551 TBR=sullivan@chromium.org Change-Id: Iaf08d38a02eb816bcc880cb602ed2592804fdc2d Reviewed-on: https://chromium-review.googlesource.com/1092250Reviewed-by:
catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: catapult-chromium-autoroll <catapult-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#565543}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/798b832..1c06e11 git log 798b832..1c06e11 --date=short --no-merges --format='%ad %ae %s' 2018-06-07 art-snake@yandex-team.ru Reland "Reland "Simplify CPDF_HintsTable."" Created with: gclient setdep -r src/third_party/pdfium@1c06e11 The AutoRoll server is located here: https://pdfium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=dsinclair@chromium.org Change-Id: Idb42683d703dd8a2864640eed0c743173d4f8c00 Reviewed-on: https://chromium-review.googlesource.com/1092190Reviewed-by:
pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#565542}
-
Yoshifumi Inoue authored
This patch gets rid of unused function |NextBoundary()| and related things to reduce code size for improving code health. Change-Id: I644f444d2946908ea8f7e954bcd6dd31b42f8523 Reviewed-on: https://chromium-review.googlesource.com/1090692Reviewed-by:
Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Yoichi Osato <yoichio@chromium.org> Commit-Queue: Yoichi Osato <yoichio@chromium.org> Cr-Commit-Position: refs/heads/master@{#565541}
-
Scott Violet authored
And makes work in ash. BUG=837715 TEST=covered by tests Change-Id: I65e5b3ee8a39f103c3ef1561a7de9ab27993aa86 Reviewed-on: https://chromium-review.googlesource.com/1090039 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#565540}
-