- 19 Sep, 2018 40 commits
- 
- 
Martin Robinson authoredBug: 885093 Change-Id: I9b6c6f7635a893b10e333bd6e72765fa630f9efb Reviewed-on: https://chromium-review.googlesource.com/1233353Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Martin Robinson <mrobinson@igalia.com> Cr-Commit-Position: refs/heads/master@{#592534} 
- 
chromium-autoroll authoredhttps://chromium.googlesource.com/catapult.git/+log/92ef0386b8b6..582a06eb7835 git log 92ef0386b8b6..582a06eb7835 --date=short --no-merges --format='%ad %ae %s' 2018-09-19 chiniforooshan@chromium.org Telemetry: don't fail when SF cannot find win name Created with: gclient setdep -r src/third_party/catapult@582a06eb7835 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-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 BUG=chromium:884999 TBR=sullivan@chromium.org Change-Id: I7ce1f90275c2422e5cb7a0325379004f8e30cc3d Reviewed-on: https://chromium-review.googlesource.com/1234957Reviewed-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@{#592533} 
- 
Clifford Cheng authoredSometimes the function failed to find the "close-button" because it's closed. This can be due to a recent change in telemetry. See bug for more details. Bug: 884929 Change-Id: I47af2f8e87431120d5dfebba3d25ead91d98f565 Reviewed-on: https://chromium-review.googlesource.com/1229359 Commit-Queue: Clifford Cheng <cliffordcheng@chromium.org> Reviewed-by: Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#592532} 
- 
Nico Weber authoredRan `tools/clang/scripts/upload_revision.py 342523`. Bug: 880827 Change-Id: I8f747ab4a2de74a0f0b4b5a201d37e111f323077 Reviewed-on: https://chromium-review.googlesource.com/1232590Reviewed-by: Reid Kleckner <rnk@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#592531} 
- 
Sky Malice authoredChange-Id: Ib5698fdef987ca82a67c9e2b974a20012e95f508 Reviewed-on: https://chromium-review.googlesource.com/1228298Reviewed-by: Filip Gorski <fgorski@chromium.org> Reviewed-by: Becky Zhou <huayinz@chromium.org> Commit-Queue: Sky Malice <skym@chromium.org> Cr-Commit-Position: refs/heads/master@{#592530} 
- 
Peter Kasting authoredFixes tests broken in Touchable Refresh and changes them from being tested in Touchable to Touchable Refresh. Also does some other cleanups. This does not remove flags added during refresh (e.g. NTB position); those will be added separately. Bug: 814847 Change-Id: I8c31a0421c65c4cd46f287d9e67aa17645089703 Reviewed-on: https://chromium-review.googlesource.com/1232882Reviewed-by: Allen Bauer <kylixrd@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#592529} 
- 
Gabriel Charette authoredThat variant was useful in the early days of TaskScheduler to ensure compatibility. It is now in the way as other variants of MessageLoop are being considered. MessageLoop is independent from TaskScheduler and the integration is now well-covered implicitly by the vast majority of tests in the codebase. R=fdoray@chromium.org Bug: 708584 Change-Id: Id93d8eb4f970e0aa0e1915ffcbc7471190e3d312 Reviewed-on: https://chromium-review.googlesource.com/1232573Reviewed-by: François Doray <fdoray@chromium.org> Commit-Queue: Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#592528} 
- 
Peter Kasting authoredThis removes support for pre-refresh modes. It does not change the design of path computations/drawing or other deeper refactors. Also does some other minor cleanup, and fixes one clear bug with high DPI in PaintChildren(). Bug: 873855 Change-Id: I3496ea60ed670d5f1c30abf33b989162c558fa16 Reviewed-on: https://chromium-review.googlesource.com/1231898Reviewed-by: Allen Bauer <kylixrd@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#592527} 
- 
Steven Bennetts authoredThis removes direct access to KeyboardUI outside of tests while the code is re-factored for Mash. Bug: 843332 Change-Id: Ic6b60032512b36b7ff1ae05bb8d12060db8f8d29 Reviewed-on: https://chromium-review.googlesource.com/1227171Reviewed-by: James Cook <jamescook@chromium.org> Reviewed-by: Darren Shen <shend@chromium.org> Reviewed-by: Shu Chen <shuchen@chromium.org> Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#592526} 
- 
Krishna Govind authoredR=benmason@chromium.org Change-Id: I0b9f1c1b77edbbd8135d20d3088b2663d4e149b5 Reviewed-on: https://chromium-review.googlesource.com/1235053Reviewed-by: Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#592525} 
- 
chromium-autoroll authoredhttps://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/69f640ec0900..647e1e79eb71 git log 69f640ec0900..647e1e79eb71 --date=short --no-merges --format='%ad %ae %s' 2018-09-19 ehmaldonado@chromium.org Reland "gclient: delete unversioned directory before adding cipd dep for the same path" Created with: gclient setdep -r src/third_party/depot_tools@647e1e79eb71 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. BUG=chromium:882611 TBR=agable@chromium.org Change-Id: I7902b0657f1aa899a92b0a55fb9a2d6317343fd6 Reviewed-on: https://chromium-review.googlesource.com/1234368Reviewed-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@{#592524} 
- 
Zach Helfinstein authoredBug: 884297 Change-Id: I4d733650e54b01459154c1c59da5c95246ec3d05 Reviewed-on: https://chromium-review.googlesource.com/1232682 Commit-Queue: Zach Helfinstein <zhelfins@chromium.org> Reviewed-by: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#592523} 
- 
Yuri Wiitala authoredOverrides the default behavior of dropping fullscreen when a tab modal dialog is opened in the FullscreenWithinTab case. This is because, in FWT mode, the browser window is in its normal layout (not fullscreened). Bug: 883535,812769 Change-Id: I1c262954b962d508eb86ef9a8a312bec03ab2332 Reviewed-on: https://chromium-review.googlesource.com/1228976 Commit-Queue: Yuri Wiitala <miu@chromium.org> Reviewed-by: Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#592522} 
- 
Dmitry Gozman authoredThis removes reliance on the callbacks from blink in RenderFrameImpl (e.g. WillSubmitForm) to track transition type. Instead, we can calculate it when needed based on the WebDocumentLoader instance. Also removes NavigationStateImpl::GetTransitionType() to avoid misusing it. Bug: 855189 Change-Id: I5c53f0f576cec0630c8de07b2ac902a1dffc30f9 Reviewed-on: https://chromium-review.googlesource.com/1228953Reviewed-by: Camille Lamy <clamy@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#592521} 
- 
chromium-autoroll authoredhttps://chromium.googlesource.com/angle/angle.git/+log/2c8f0845431d..2ec817bb6bd9 git log 2c8f0845431d..2ec817bb6bd9 --date=short --no-merges --format='%ad %ae %s' 2018-09-19 tobine@google.com Remove writeResource down readPixels path Created with: gclient setdep -r src/third_party/angle@2ec817bb6bd9 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: I895bd1cc63055d543dcd99c8dcb1a3598ccc1f22 Reviewed-on: https://chromium-review.googlesource.com/1234364Reviewed-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@{#592520} 
- 
Christian Biesinger authoredAfter https://chromium-review.googlesource.com/c/chromium/src/+/1232584, layout_block.h no longer needs to include ng_constraint_space.h Change-Id: I9788d6ba0488431facfd3af1eaf6e8f32927ca91 Reviewed-on: https://chromium-review.googlesource.com/1234218 Commit-Queue: Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by: Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#592519} 
- 
Bence Béky authoredOriginal CL landed at https://crrev.com/c/1073192. Revert (with edits) landed at https://crrev.com/c/1230442. This CL relands the original with the modification that |callback_| is reset in DhcpPacFileFetcherChromeos::Cancel() and OnShutdown(). It is believed that the crashes the original CL caused happened when Fetch() was called after Cancel(), and this modification will prevent that from happening. Kudos to eroman@ for pointing this out at https://crrev.com/c/1230442/3/chromeos/network/dhcp_pac_file_fetcher_chromeos.cc#63 Bug: 882996 Change-Id: I4747004c00fa3ac7a698d7a35c0b52999411adb5 Reviewed-on: https://chromium-review.googlesource.com/1232576Reviewed-by: Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#592518} 
- 
Eugene But authoredTaskShutdownBehavior::BLOCK_SHUTDOWN is used to make sure that completion handler is always called on shutdown. Bug: 853774 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Ief6560419f79055ce9a8d0de47806db5f3f9c9d4 Reviewed-on: https://chromium-review.googlesource.com/1226195Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#592517} 
- 
Nektarios Paisios authoredWindows screen readers call IAccessible::get_appName and IAccessible::get_appVersion to determine that Chrome is running and retrieve its version for loading the correct scripts. However, when the content_shell is running instead of Chrome, |GetProduct| returns an empty string. Since screen reader access is not essential while content_shell is running, we simply return E_FAIL for both these API calls rather than DCHECK. R=dmazzoni@chromium.org Change-Id: Iec62c134cd55b204fe460ff0d632b280996297a3 Reviewed-on: https://chromium-review.googlesource.com/1232583Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#592516} 
- 
Matt Menke authoredAlso add a TCPConnectedSocketOptions structure to sent options on socket construction, as it makes state management and error handling in consumers simpler if they want to set multiple options when creating a socket. BUG=848078 TBR=mfoltz@chromium.org Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I2177b907b2a1d75212e27505f8366c608457e2b8 Reviewed-on: https://chromium-review.googlesource.com/1204852 Commit-Queue: Matt Menke <mmenke@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org> Reviewed-by: Reilly Grant <reillyg@chromium.org> Reviewed-by: Maks Orlovich <morlovich@chromium.org> Cr-Commit-Position: refs/heads/master@{#592515} 
- 
chrome://discardsSigurdur Asgeirsson authoredAlso drop some unused styles while I was in there. Bug: 874968 Change-Id: I44881e7b5eefd63db4d9999c37958ba825fe1cbf Reviewed-on: https://chromium-review.googlesource.com/1228775Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by: Chris Hamilton <chrisha@chromium.org> Reviewed-by: Will Harris <wfh@chromium.org> Commit-Queue: Sigurður Ásgeirsson <siggi@chromium.org> Cr-Commit-Position: refs/heads/master@{#592514} 
- 
Kevin Marshall authored* Install the Context's persistent data directory in /data. * Use the existence of /data as the canonical signal for enabling incognito mode. (Tests will still use the --incognito switch override.) ** Add a smoke test suite for running in incognito mode. * Configure disk_cache to use SimpleCacheBackend. The default persistent cache backend will break because it relies on mmap(), which isn't supported by minfs. * Add semantic validation for the |data_directory| handle. ** Add tests to exercise validation. * Remove GetPackageRoot(), since there's no reason to not use PathService(DIR_SOURCE_ROOT, ...) anymore. * Remove unused Fuchsia-specific path DIR_FUCHSIA_RESOURCES. Bug: 840598 Change-Id: I3efdb072cc4ee8f7df21992d6a84a1b548bb7fed Reviewed-on: https://chromium-review.googlesource.com/1232524Reviewed-by: Scott Graham <scottmg@chromium.org> Reviewed-by: Kevin Marshall <kmarshall@chromium.org> Reviewed-by: Maks Orlovich <morlovich@chromium.org> Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#592513} 
- 
Asanka Herath authoredThis reverts commit c9b5d530. Reason for revert: Compile failure: https://ci.chromium.org/buildbot/chromium.chrome/Google%20Chrome%20ChromeOS/56793 Original change's description: > Refresh access token only for auth error. > > Bug: b:114756954 > Test: Manual > Change-Id: I1d58906771ad789232d8c01f6de06ea44ad8e3f8 > Reviewed-on: https://chromium-review.googlesource.com/1226527 > Commit-Queue: Yilei Li <yileili@google.com> > Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org> > Cr-Commit-Position: refs/heads/master@{#592467} TBR=xiaohuic@chromium.org,dmblack@google.com,yileili@google.com Change-Id: I47b836b3b8d1d49e3dde93a8133d80c27a411a5a No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b:114756954 Reviewed-on: https://chromium-review.googlesource.com/1234895Reviewed-by: Asanka Herath <asanka@chromium.org> Commit-Queue: Asanka Herath <asanka@chromium.org> Cr-Commit-Position: refs/heads/master@{#592512} 
- 
David Tseng authoredFollow up to incomplete change https://chromium-review.googlesource.com/c/chromium/src/+/1228790 Change-Id: I552c0896ae903223f659eb647bf3b3bf5f75e3dc Reviewed-on: https://chromium-review.googlesource.com/1232954Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#592511} 
- 
dpapad authoredIn Polymer 2 the following syntax for registering event listeners is no longer supported. listeners: { 'id.eventName': 'onEventName_', }, Bug: 885294 Change-Id: I143aea45aab7ccb237f9034b04be5882ef7b8de0 Reviewed-on: https://chromium-review.googlesource.com/1231926Reviewed-by:Rebekah Potter <rbpotter@chromium.org> Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#592510} 
- 
liberato@chromium.org authoredThis CL addresses a few issues with MCVD unittests: - Some tests were marked 'VP8' when they were supposed to be 'not VP8'. This changes them to H264. Not exactly the same thing, but better than it was. - Some tests were marked 'H264' since they didn't work for VPx for no clear reason. These have been fixed. - VP8 codec could leak, since the drain during destruction wasn't complete. It now completes the drain when needed. Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Change-Id: Id9dab9b93425581fa3f7892ef734fbd3e5c54eaf Reviewed-on: https://chromium-review.googlesource.com/1234360 Commit-Queue: Frank Liberato <liberato@chromium.org> Reviewed-by: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#592509} 
- 
Mike Reed authoredCq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Id97f574f12deffbfe28e943a7735823be1533db9 Reviewed-on: https://chromium-review.googlesource.com/1233941Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Mike Reed <reed@chromium.org> Cr-Commit-Position: refs/heads/master@{#592508} 
- 
chromium-autoroll authoredhttps://skia.googlesource.com/skia.git/+log/c2d0fb17deef..8a0014403563 git log c2d0fb17deef..8a0014403563 --date=short --no-merges --format='%ad %ae %s' 2018-09-19 halcanary@google.com test,tools: whitespace changes for clang-format 2018-09-19 mtklein@google.com update clang_linux to Clang 7 Created with: gclient setdep -r src/third_party/skia@8a0014403563 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: I2cbc67728a544de1248e2434d1334fb5e09f7e03 Reviewed-on: https://chromium-review.googlesource.com/1234366Reviewed-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@{#592507} 
- 
David Tseng authoredTBR=dmazzoni@chromium.org Bug: 795840 Change-Id: I66868a0b735f660e5f0190e7bcc6e4c158393138 Reviewed-on: https://chromium-review.googlesource.com/1234317Reviewed-by: David Tseng <dtseng@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#592506} 
- 
Sergey Poromov authoredFinalizing LoginDisplayHost before AttemptUserExit caused double deletion of LoginDisplayHost and a crash that leaded to not remembering error KioskAppLaunchError in a pref. Bug: 879652 Test: Manual Change-Id: I291872af300d99940cb9199cbdf62b01918ffbdc Reviewed-on: https://chromium-review.googlesource.com/1234476Reviewed-by: Jacob Dufault <jdufault@chromium.org> Commit-Queue: Sergey Poromov <poromov@chromium.org> Cr-Commit-Position: refs/heads/master@{#592505} 
- 
Mathieu Perreault authoredThe feature was overriding values in |upload_request_.profiles|, which turns out was being reused for a subsequent request. We now use a temporary structure to send country-only addresses to Payments for GetUploadDetails call. This change also disables by default the SendOnlyCountryInGetUploadDetails and PaymentsCustomerData features. Bug: 878416 Test: components_unittests Change-Id: If78098e8b494cf4bba2e5049c368d87b2913a84d Reviewed-on: https://chromium-review.googlesource.com/1233815 Commit-Queue: Mathieu Perreault <mathp@chromium.org> Reviewed-by: Jared Saul <jsaul@google.com> Cr-Commit-Position: refs/heads/master@{#592504} 
- 
Mikel Astiz authoredThis speculates with the idea of the proxy dropping some events, e.g. OnSyncStopping(), and later successfully posting other ones, e.g. OnSyncStarting(). To make investigation easier, we also split a previously existing CHECK into separate ones per datatype, to narrow down which ones are affected. Bug: 885019 Change-Id: I1e569eb0868cff4b66d841420f37f15c71f70567 Reviewed-on: https://chromium-review.googlesource.com/1233707 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#592503} 
- 
Clark DuVall authoredThis is a reland of 888677bc Unit tests were failing on Linux CFI bot, needed a couple extra lines to set up the test WebContents. Original change's description: > Switch UserScriptListener ResourceThrottle to NavigationThrottle > > ResourceThrottles do not work with network service enabled, a > NavigationThrottle will work both with NS enabled and disabled. > > Bug: 884006 > Change-Id: Ifea0e59bd67d70b593b46c2a2a2bd4be0e3f98f0 > Reviewed-on: https://chromium-review.googlesource.com/1227358 > Reviewed-by: Devlin <rdevlin.cronin@chromium.org> > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Commit-Queue: Clark DuVall <cduvall@chromium.org> > Cr-Commit-Position: refs/heads/master@{#592182} TBR=jam@chromium.org,rdevlin.cronin@chromium.org Bug: 884006 Change-Id: I146dced8038dff7dfed262748602a87dc1993af8 Reviewed-on: https://chromium-review.googlesource.com/1234353Reviewed-by: Clark DuVall <cduvall@chromium.org> Reviewed-by: John Abd-El-Malek <jam@chromium.org> Commit-Queue: Clark DuVall <cduvall@chromium.org> Cr-Commit-Position: refs/heads/master@{#592502} 
- 
Justin DeWitt authoredThis patch: * Adds APIs to ExploreSitesService that can request either a site image or a category image (which is a composite of multiple site images) as an SkBitmap (which is the format required for Java Bitmaps). * Adds a task to fetch one or more images from the ExploreSitesStore, based on the site ID or category ID. * The task filters based on the existence of an entry in the site_blacklist table (which is not populated yet but will be as a result of user long-press. This DOES NOT implement conversion from byte vectors to SkBitmap (left as a TODO.) Bug: 867488 Change-Id: Id2e6a6abe2781b790f9ce3f597a9e747eddd4715 Reviewed-on: https://chromium-review.googlesource.com/1233176Reviewed-by: Cathy Li <chili@chromium.org> Commit-Queue: Justin DeWitt <dewittj@chromium.org> Cr-Commit-Position: refs/heads/master@{#592501} 
- 
Oystein Eftevaag authoredThis fixes a data race where ~TraceWriter tries finalizing the same message as ~TraceEventDataSource, on a different sequence (TraceWriter is owned by TraceEventDataSource, but needs to be destructed on a different sequence due to ~TraceEventDataSource being called on thread shutdown and could be calling PostTask). BUG=886528 R=ssid@chromium.org Change-Id: I61b498b8b4bcf3b40e43e9db49711e593766df26 Reviewed-on: https://chromium-review.googlesource.com/1232414Reviewed-by: Siddhartha S <ssid@chromium.org> Commit-Queue: oysteine <oysteine@chromium.org> Cr-Commit-Position: refs/heads/master@{#592500} 
- 
chromium-autoroll authoredhttps://chromium.googlesource.com/catapult.git/+log/c968ea0b6538..92ef0386b8b6 git log c968ea0b6538..92ef0386b8b6 --date=short --no-merges --format='%ad %ae %s' 2018-09-19 dtu@chromium.org [pinpoint] Data migration basic error handling. 2018-09-19 dtu@chromium.org [pinpoint] Basic stats page. 2018-09-19 dtu@chromium.org [pinpoint] Update fields in Jobs table. 2018-09-19 cbruni@chromium.org results2.html debounce search 2018-09-19 simonhatch@chromium.org Dashboard - Fix alert and comparison_magnitude. 2018-09-19 simonhatch@chromium.org Dashboard - Replace last_alerted_revision with query 2018-09-19 slobodan@google.com Fixing frames in animations following response. 2018-09-19 pasko@chromium.org androidStartupMetric: remove the FCP 2018-09-18 dtu@chromium.org [pinpoint] Remove unused <job-chart-icon> import. 2018-09-18 achuith@chromium.org oobe: Catch WebSocketException instead. 2018-09-18 dtu@chromium.org [pinpoint] Remove list of differences and reformat arguments list. 2018-09-18 sadrul@chromium.org rendering: Remove some redundant mean_ metrics. 2018-09-18 sadrul@chromium.org rendering: Remove discrepancy metrics. 2018-09-18 chiniforooshan@chromium.org Telemetry: Fix SF events 2018-09-18 mattm@chromium.org netlog_viewer/README.md: Re-order with user-focused sections at top. Created with: gclient setdep -r src/third_party/catapult@92ef0386b8b6 The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-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 BUG=chromium:878390,chromium:868073,chromium:883079,chromium:883715,chromium:886621,chromium:879353,chromium:884950,chromium:884950,chromium:863926 TBR=sullivan@chromium.org Change-Id: I9fa8e95d9aad524c032b0bf637dfc07926ce7795 Reviewed-on: https://chromium-review.googlesource.com/1234576Reviewed-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@{#592499} 
- 
Lucas Gadani authoredThis CL adds a notification via WebContentsDelegate that notifies the embedder that a new portal WebContents was created. The chrome/ embedder then uses this notification in order to attach TabHelpers to the newly created WebContents. Bug: 865565 Change-Id: I3e39883a5869832e6e59be9db7ea9e88ab18cd64 Reviewed-on: https://chromium-review.googlesource.com/1197143Reviewed-by: Avi Drissman <avi@chromium.org> Commit-Queue: Lucas Gadani <lfg@chromium.org> Cr-Commit-Position: refs/heads/master@{#592498} 
- 
Fady Samuel authoredBrowserPlugin is only on on Chrome desktop and and surface sync is always on on desktop. This CL deletes the old code path. Bug: 672962 Change-Id: If0394bd7b8eed4b04d23fede8e31aff1ee3417b4 Reviewed-on: https://chromium-review.googlesource.com/1234323Reviewed-by: Ken Buchanan <kenrb@chromium.org> Reviewed-by: Saman Sami <samans@chromium.org> Reviewed-by: James MacLean <wjmaclean@chromium.org> Commit-Queue: Fady Samuel <fsamuel@chromium.org> Cr-Commit-Position: refs/heads/master@{#592497} 
- 
Bence Béky authoredBug: 886629 Change-Id: Ie0f096552bec89f57041caebcf248da76f3a468d Reviewed-on: https://chromium-review.googlesource.com/1233936 Commit-Queue: Bence Béky <bnc@chromium.org> Reviewed-by: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#592496} 
- 
chromium-autoroll authoredhttps://pdfium.googlesource.com/pdfium.git/+log/c3099d1c6942..174de19776de git log c3099d1c6942..174de19776de --date=short --no-merges --format='%ad %ae %s' 2018-09-19 thestig@chromium.org Encapsulate CPDF_ImageLoader. 2018-09-19 thestig@chromium.org Validate decoder pipelines. Created with: gclient setdep -r src/third_party/pdfium@174de19776de 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. BUG=chromium:880675 TBR=dsinclair@chromium.org Change-Id: I1af59ee6d0a3d3c21cce116d96083947b365ee31 Reviewed-on: https://chromium-review.googlesource.com/1234295Reviewed-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@{#592495} 
 
-