- 19 Oct, 2017 22 commits
-
-
Antoine Labour authored
This is a microbenchmark that measures the raw throughput of the GLES2 decoders. Bug: 775212 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: Ifbb183c51c498dc497e273b59d6c79849969d021 Reviewed-on: https://chromium-review.googlesource.com/723799Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Victor Miura <vmiura@chromium.org> Commit-Queue: Antoine Labour <piman@chromium.org> Cr-Commit-Position: refs/heads/master@{#509954}
-
Will Chen authored
This migrates tests in debugger-pause/, debugger-frameworks/, debugger-console/ to the new integration test framework. Most of it was done through an automated transformation script, with some manual clean-up. Bug: 667560 Change-Id: I558629883707f270a682e000eb9a0742e7e0e8df Reviewed-on: https://chromium-review.googlesource.com/711317 Commit-Queue: Will Chen <chenwilliam@chromium.org> Reviewed-by:
Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Cr-Commit-Position: refs/heads/master@{#509953}
-
Peter Boström authored
Expands test names like dse_ext1 for instance to something easily readable. Bug: chromium:698785 Change-Id: I96a65b377f9d882f2f9a15832bb701cfe24afe4d Reviewed-on: https://chromium-review.googlesource.com/727364Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#509952}
-
James Robinson authored
When trying to send a platform handle represented as a file descriptor cross-process on Fuchsia, we attempt to transfer the FD in-place which is more efficient than the usual clone + send + close pattern. In some cases the transfer operation fails, however, so we have to make a fresh clone to transfer. Bug: 776163 Change-Id: I6d25e19ee7fef194261f5d1e624a4d8de291d6b3 Reviewed-on: https://chromium-review.googlesource.com/727121 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#509951}
-
Dirk Pranke authored
R=iannucci@chromium.org Change-Id: I00401bbd710e2c80fab6571a1a555925259511a9 Reviewed-on: https://chromium-review.googlesource.com/726559Reviewed-by:
Robbie Iannucci <iannucci@chromium.org> Commit-Queue: Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#509950}
-
xzhan96 authored
This CL replaces native struct content::ServiceWorkerObjectInfo with mojom struct blink.mojom.ServiceWorkerObjectInfo. This is an incremental step toward Onion Soup: having less dependence on content types for service worker. BUG=755861 Change-Id: Ic9b59c771c8eb4687e833fb0cc744987dab40aac Reviewed-on: https://chromium-review.googlesource.com/715476Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Matt Falkenhagen <falken@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Xiaofeng Zhang <xiaofeng.zhang@intel.com> Cr-Commit-Position: refs/heads/master@{#509949}
-
Jun Cai authored
This CL fixes potential nullptr deference by getting sensor type from |sensor| pointer before the pointer is moved. Bug: 775874 Change-Id: Iaba148a7117da1c4220eee51e7149db6c632f2cd Reviewed-on: https://chromium-review.googlesource.com/727239 Commit-Queue: Jun Cai <juncai@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#509948}
-
Matt Menke authored
This will let us start adding NetLog APIs to the network service, and using them in production, without breaking logging. Eventually we'll want to get rid of Chrome's NetLog entirely, but doing that all at once would be too much for a single CL. Bug: 767450 Change-Id: I1ce00dded81f83ce3e7d1e498f72d991a68d3e66 Reviewed-on: https://chromium-review.googlesource.com/723819Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#509947}
-
catapult-deps-roller@chromium.org authored
https://chromium.googlesource.com/catapult.git/+log/79f62e6c96d8..2bc7fe7bc1e6 $ git log 79f62e6c9..2bc7fe7bc --date=short --no-merges --format='%ad %ae %s' 2017-10-18 benjhayden Fix DiagnosticMap.fromObject(). Created with: roll-dep src/third_party/catapult Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: Idb1d7593ad019cf40ba585c7017f1a2d4380b777 Reviewed-on: https://chromium-review.googlesource.com/726850 Reviewed-by: <catapult-deps-roller@chromium.org> Commit-Queue: <catapult-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#509946}
-
Will Chen authored
This eliminates a lot of the code duplication between the new and legacy integration test frameworks by making network-test.js a small wrapper to load the new network_test_runner modules. Bug: 667560 Change-Id: I0374e4b70f07b7602919b0fb7803ae8aabdd2ac0 Reviewed-on: https://chromium-review.googlesource.com/724145 Commit-Queue: Will Chen <chenwilliam@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#509945}
-
Henrik Boström authored
getRemoteStreams() does not exist in the spec. This redefines the method on top of the RTP Media API and removes the need to keep track of a remote_streams_ list and some plumbing. This also makes the order of getReceivers() well-defined (receivers are iterated in the same order that they were added) so that the order of getRemoteStreams() remains well-defined and the same as before this CL. Bug: 741618 Change-Id: I78d25abf7e01ce8efd9dd085d304e778c9789bc4 Reviewed-on: https://chromium-review.googlesource.com/707693 Commit-Queue: Henrik Boström <hbos@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Taylor Brandstetter <deadbeef@chromium.org> Cr-Commit-Position: refs/heads/master@{#509944}
-
David Reveman authored
This uses CrPngBehavior to remove any potential animations from the profile image displayed on the 'People' settings page. Bug: 775988 Test: manual Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ib6675b648e30c409f3c53973bed219d0479f1d6f Reviewed-on: https://chromium-review.googlesource.com/726381 Commit-Queue: David Reveman <reveman@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Cr-Commit-Position: refs/heads/master@{#509943}
-
Bin Zhao authored
ProvideSink() is desktop specific and should not be part of MediaRouter interface. Resolve code review comments https://chromium-review.googlesource.com/c/chromium/src/+/708194/1/chrome/browser/media/router/mojo/media_router_desktop.cc#150 Bug: 687377 Change-Id: I943a8fee32575fea92973707892cc60a01986178 Reviewed-on: https://chromium-review.googlesource.com/714819 Commit-Queue: Bin Zhao <zhaobin@chromium.org> Reviewed-by:
Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#509942}
-
nainar authored
This reverts commit b30a6772. Reason for revert: Not a stable enough change as it causes crbug.com/775922 Original change's description: > Only copy parsed properties over when mutating a rule. > > Currently when we copy over a StyleRule we parse all proeprties > greedily. This should only be the already parsed properties. The greedy > method may result in a dangerous state. > > This is a speculative fix for the ClusterFuzz issue. > > Bug: 774061 > Change-Id: I0b7f09018c7cf2d8ca75ea5d705016fbcce6f0ae > Reviewed-on: https://chromium-review.googlesource.com/722579 > Reviewed-by: Darren Shen <shend@chromium.org> > Commit-Queue: nainar <nainar@chromium.org> > Cr-Commit-Position: refs/heads/master@{#509352} TBR=nainar@chromium.org,shend@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 774061, 775922 Change-Id: I49f536965359f14793eba4f952d216a04dd07df6 Reviewed-on: https://chromium-review.googlesource.com/726679Reviewed-by:
nainar <nainar@chromium.org> Commit-Queue: nainar <nainar@chromium.org> Cr-Commit-Position: refs/heads/master@{#509941}
-
Alice Boxhall authored
Add TestExpectation for virtual/longtask-v2/http/tests/performance-timing/longtask-v2/longtask-v8compile.html TBR=maxlg@chromium.org No-Try: True Bug: 776201 Change-Id: I8c3979f6c47af7aa77d8a381e2b8827eb6b242d9 Reviewed-on: https://chromium-review.googlesource.com/727419 Commit-Queue: Alice Boxhall <aboxhall@chromium.org> Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Cr-Commit-Position: refs/heads/master@{#509940}
-
Catherine Mullings authored
See title. Bug: 770846 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Id8faaabc6d8f85717b392b4cfd2c7501904eb149 Reviewed-on: https://chromium-review.googlesource.com/716637 Commit-Queue: catmullings <catmullings@chromium.org> Reviewed-by:
Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#509939}
-
Alice Boxhall authored
This reverts commit 9af30c03. Reason for revert: Looks like this broke SpokenFeedbackTest.ChromeVoxShiftSearch: https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%281%29/builds/45903 Original change's description: > Fire focus if needed in automation api > > Bug: 773866 > > Change-Id: I4e7295137141fa4f1605247cafb3bc4d40a09a59 > Reviewed-on: https://chromium-review.googlesource.com/723606 > Commit-Queue: David Tseng <dtseng@chromium.org> > Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org> > Cr-Commit-Position: refs/heads/master@{#509911} TBR=dmazzoni@chromium.org,dtseng@chromium.org Change-Id: I6f571f16e6d8015098cf5c8df96e51b2f3653580 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 773866 Reviewed-on: https://chromium-review.googlesource.com/727439Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Commit-Queue: Alice Boxhall <aboxhall@chromium.org> Cr-Commit-Position: refs/heads/master@{#509938}
-
Xu Xing authored
So return at start. BUG=None Change-Id: I47dc83ad746788cf769d9308b9920390d37cca65 Reviewed-on: https://chromium-review.googlesource.com/724720Reviewed-by:
enne <enne@chromium.org> Commit-Queue: Xing Xu <xing.xu@intel.com> Cr-Commit-Position: refs/heads/master@{#509937}
-
Xi Cheng authored
This CL directs toast click and close events to NotificationDisplayService for proper actions. The toast notification clicked or closed needs to still be cached in memory (i.e., Chrome is running and staying in the same browsing session) for the callbacks to take effect. Bug: 734095 Change-Id: I88da36bf052062392af110e427a8c64b2687f625 Reviewed-on: https://chromium-review.googlesource.com/720016 Commit-Queue: Xi Cheng <chengx@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#509936}
-
Yoland Yan authored
This fix the attempt to recover device when `--num_retries=0` is specified locally Change-Id: Ie73a62afd13835dbdd3f18716bc3a2ea0ca59434 Reviewed-on: https://chromium-review.googlesource.com/726848 Commit-Queue: Yoland Yan <yolandyan@chromium.org> Reviewed-by:
Benjamin Pastene <bpastene@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#509935}
-
Naina Raisinghani authored
This is a revert of the following 2 issues: 1. No need for forced SetStyleInternal for unchanged ComputedStyle. 2. Reland 'Store new ComputedStyle object if style did not change. Reverts generated via git revert. Done on behalf of rune@ who has now left Opera. TBR=shend@chromium.org Bug: 771294 Change-Id: I450b66124e30978fb731e05a3528395bb8862f3d Reviewed-on: https://chromium-review.googlesource.com/726719 Commit-Queue: nainar <nainar@chromium.org> Reviewed-by:
nainar <nainar@chromium.org> Cr-Commit-Position: refs/heads/master@{#509934}
-
Raymes Khoury authored
If microphone/camera usage are disabled by feature policy, this will also impact usage by Pepper. Currently we only log a deprecation warning for getUserMedia usage, but also log one for Flash explicitly. Bug: 689802, 776167 Change-Id: If8144099909cfbdb40582d360f4f1eb3265db918 Reviewed-on: https://chromium-review.googlesource.com/720583 Commit-Queue: Raymes Khoury <raymes@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#509933}
-
- 18 Oct, 2017 18 commits
-
-
Reilly Grant authored
These tests appear to be passing and should be enabled. TBR=jam@chromium.org Bug: 729849,721408,611935 Change-Id: I163023024f2f94c8b6031f7f2c3a4ec85f2f2ada No-Try: true Reviewed-on: https://chromium-review.googlesource.com/724282Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#509932}
-
Charlie Andrews authored
This logging is to try and track down a bug in which clock syncs are occassionally not being included in the BattOr trace. TBR=rnephew@chromium.org Bug: 776117 Change-Id: Id8f6d2fd07829db889a883f5474b268e20730628 Reviewed-on: https://chromium-review.googlesource.com/726883Reviewed-by:
Charlie Andrews <charliea@chromium.org> Reviewed-by:
rnephew <rnephew@chromium.org> Commit-Queue: Charlie Andrews <charliea@chromium.org> Cr-Commit-Position: refs/heads/master@{#509931}
-
Xiaocheng Hu authored
This patch adds a reference to the canonical text in offset mapping, so that we don't need to make a detour to NGInlineNode when working on both offset mapping and canonical text. This patch is a preparation for adding NG version of LayoutText::ContainsCaretOffset(). Bug: 771398 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I8cf035c44ab90452819d92b91e7853b848781a41 Reviewed-on: https://chromium-review.googlesource.com/724286Reviewed-by:
Emil A Eklund <eae@chromium.org> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#509930}
-
erikchen authored
Reland #1 ran into two problems. * Timeouts on mac ASAN tests were caused by a mojo bug. See Issue 771805 for more details. * An incorrectly appearing views scrollbar was caused by a logic bug in ScrollView. See Issue 771502 for more details. CL description from original CL: > The memory_instrumentation interface on the content_browser service exposes > cross-platform, consistent memory metrics that should also be used by the task > manager. > > This CL adds a new column to the task manager: Memory Footprint. See > https://docs.google.com/document/d/1PZyRzChnvkUNUB85Op46aqkFXuAGUJi751DJuB6O40g/edit# > for more details. > > Bug: 756276 > Reviewed-on: https://chromium-review.googlesource.com/646047 > Reviewed-by: Nick Carter <nick@chromium.org> > Reviewed-by: Primiano Tucci <primiano@chromium.org> > Commit-Queue: Erik Chen <erikchen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#506220} Bug: 756276 TBR: nick@chromium.org Change-Id: Iffae21fed72410de818180351b66003940f53a3e Reviewed-on: https://chromium-review.googlesource.com/710172 Commit-Queue: Erik Chen <erikchen@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#509929}
-
Kentaro Hara authored
Bug: Change-Id: I7ca545ab4e0a3c9e5c4c788df045a92a85f2ce2c Reviewed-on: https://chromium-review.googlesource.com/725081Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#509928}
-
Scott Graham authored
Useful to narrow down to a particular test fixture when flaking. Bug: 771701 Change-Id: Idcdb60739c55e759c7ad98a5597878d3753b39ca Reviewed-on: https://chromium-review.googlesource.com/726276 Commit-Queue: Scott Graham <scottmg@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#509927}
-
Dave Tapuska authored
Mock a WidgetInputHandlerHost interface and use that for the RenderWidget unittests. BUG=722928 Change-Id: I9ea24668fa08bbec84085909b21f3d425b71885f Reviewed-on: https://chromium-review.googlesource.com/725547Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#509926}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/854d71c1420e..68708e4344b9 $ git log 854d71c14..68708e434 --date=short --no-merges --format='%ad %ae %s' 2017-10-18 dsinclair Cleanup nits in cxfa_ffdocview 2017-10-18 dsinclair Fix rounding of colour values Created with: roll-dep src/third_party/pdfium Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, see: http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls TBR=dsinclair@chromium.org Change-Id: I66291bbea87ba00e3a411daca2dea4bb8457d1fb Reviewed-on: https://chromium-review.googlesource.com/726326 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#509925}
-
Henrique Nakashima authored
This metric is already logged but was never added to histograms.xml. Bug: chromium:768986 Change-Id: I77322aa2e4a71469c45a32bb1cc88aac9f42c7db Reviewed-on: https://chromium-review.googlesource.com/714125 Commit-Queue: Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Cr-Commit-Position: refs/heads/master@{#509924}
-
David Reveman authored
This updates the user image URL handler to respect frame arguments for motion stills and not only the default set of user images. Bug: 775988 Test: manual Change-Id: I25b9e5d0d3937c90b128b246ba40c63bb59a4b51 Reviewed-on: https://chromium-review.googlesource.com/726600Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: David Reveman <reveman@chromium.org> Cr-Commit-Position: refs/heads/master@{#509923}
-
Matt Wolenetz authored
Adds unit test coverage of WebMediaPlayerImpl's logging of playback rate changes. BUG=771002,775878 TEST=media_blink_unittests: WebMediaPlayerImplTest.PlaybackRateChangeMediaLogs Change-Id: Idd3d63142f627a79e90c3a122d84eb2053dc8e48 Reviewed-on: https://chromium-review.googlesource.com/726739Reviewed-by:
Dan Sanders <sandersd@chromium.org> Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org> Cr-Commit-Position: refs/heads/master@{#509922}
-
Jon Napper authored
This reverts commit bbf38447. Reason for revert: Possible performance regression, https://bugs.chromium.org/p/chromium/issues/detail?id=775828#c3 Original change's description: > Added missing well-known conflicting language codes > > Added missing well-known conflicting language codes to whitelist of > codes that are often incorrect (see > go/reducing-language-detection-conflicts) > > Bug: 765006 > Change-Id: I159be446d6521c356461187f94ed372a7494cafc > Reviewed-on: https://chromium-review.googlesource.com/720578 > Reviewed-by: Jia Meng <jiameng@chromium.org> > Commit-Queue: Jon Napper <napper@chromium.org> > Cr-Commit-Position: refs/heads/master@{#508983} TBR=napper@chromium.org,jiameng@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 765006 Change-Id: Id466d355e1468bc9784f613af5b94212549c1d18 Reviewed-on: https://chromium-review.googlesource.com/726680Reviewed-by:
Jia Meng <jiameng@chromium.org> Commit-Queue: Jon Napper <napper@chromium.org> Cr-Commit-Position: refs/heads/master@{#509921}
-
Renee Wright authored
Bug: 775331 Change-Id: Ifddb08515ea3de1de20039c5b1e07fedd0018e67 Reviewed-on: https://chromium-review.googlesource.com/721100 Commit-Queue: Renée Wright <rjwright@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
meade_UTC10 <meade@chromium.org> Cr-Commit-Position: refs/heads/master@{#509920}
-
Rohit Rao authored
The TabSwitcher is now displayed as a contained child, while BVCs are presented inside a container VC. This structure allows for BVCs to be presented even before a TabSwitcher is created, and it allows incognito/non-incognito BVCs to be swapped without having to dismiss and re-present view controllers. More details can be found in the design doc at https://docs.google.com/document/d/11kI9oH8NUl0oJU_ZRHt6amblGyzPSwhvB_7JWsKayGY/view BUG=768563 Change-Id: I971cf80c1c2714d801f9885369ce1d0fb95afb28 Reviewed-on: https://chromium-review.googlesource.com/723940Reviewed-by:
edchin <edchin@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Rohit Rao (ping after 24h) <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#509919}
-
Vlad Tsyrklevich authored
This is a reland of 3d8ae2fd Original change's description: > Fix cfi-icall build by expanding the CFI blacklist > > This adds a wide blacklist for failing cfi-icalls that will pass all > unit tests on Linux. This is functionally a to-do list of libraries and > routines that need to be fixed or refactored to work with cfi-icall. > > Bug: 772906 > Change-Id: Ie483a818f95b594d799622b52a13086686522c25 > Reviewed-on: https://chromium-review.googlesource.com/719733 > Commit-Queue: Peter Collingbourne <pcc@chromium.org> > Reviewed-by: Peter Collingbourne <pcc@chromium.org> > Cr-Commit-Position: refs/heads/master@{#509633} Bug: 772906 Change-Id: I17230612c6d690a55426eab3d4e691cd1fb7a298 Reviewed-on: https://chromium-review.googlesource.com/726543Reviewed-by:
Peter Collingbourne <pcc@chromium.org> Commit-Queue: Peter Collingbourne <pcc@chromium.org> Cr-Commit-Position: refs/heads/master@{#509918}
-
Nico Weber authored
Bug: 775328 Change-Id: I96c860f729c58d3dff9893255978f72eb2bb448b Reviewed-on: https://chromium-review.googlesource.com/727039 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#509917}
-
Sunny Sachanandani authored
This reverts commit b7662394. Reason for revert: The underlying issue (crbug.com/760886) has been fixed so reenable Original change's description: > Disable browse:media:tumblr on Mac 10.11 Perf > > TBR=nednguyen@chromium.org,altimin@chromium.org > > Bug: 760966 > Change-Id: If6eb9c9f865925df89ef3df74dd49dd179468b47 > Reviewed-on: https://chromium-review.googlesource.com/648248 > Reviewed-by: Charlie Andrews <charliea@chromium.org> > Reviewed-by: Alexander Timin <altimin@chromium.org> > Commit-Queue: Charlie Andrews <charliea@chromium.org> > Cr-Commit-Position: refs/heads/master@{#499324} TBR=nednguyen@google.com,charliea@chromium.org,altimin@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 760966 Change-Id: I031b6d73da8f31b9fd10263b67671083409dd2d1 Reviewed-on: https://chromium-review.googlesource.com/726839 Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Reviewed-by:
Sunny Sachanandani <sunnyps@chromium.org> Cr-Commit-Position: refs/heads/master@{#509916}
-
Xiaocheng Hu authored
This patch adds a new API to LayoutNG offset mapping, so that we can easily check if a given DOM offset is right after a non-collapsed character. This patch is a preparation for adding an NG version of LayoutText::ContainsCaretOffset(). Bug: 771398 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng Change-Id: I93e8a89089661c6f17da6a7a847f75914cb36f75 Reviewed-on: https://chromium-review.googlesource.com/724230 Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Reviewed-by:
Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#509915}
-