- 06 Mar, 2018 40 commits
-
-
Dmitry Gozman authored
WebContents::GetDelegate may return null in some cases (see issue for repro cases). Bug: 756527 Change-Id: Iad30cccd1246c59fd035b86305c21e46bd57f404 Reviewed-on: https://chromium-review.googlesource.com/951903Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Andrey Kosyakov <caseq@chromium.org> Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#541220}
-
Ovidio Henriquez authored
Moves a single Bluetooth test under the wpt/http/ directory to its rightful place under wpt/bluetooth. BUG=509038 Change-Id: I9eaa21858c74fedba2a91c508b4097609f8c69a6 Reviewed-on: https://chromium-review.googlesource.com/951882 Commit-Queue: Ovidio Henriquez <odejesush@chromium.org> Reviewed-by:
Conley Owens <cco3@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Cr-Commit-Position: refs/heads/master@{#541219}
-
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/5dc7db08f808..7224dea74d49 $ git log 5dc7db08f..7224dea74 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src-internal The AutoRoll server is located here: https://src-internal-chromium-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=mmoss@chromium.org Change-Id: Ifcb05d4b1eca7a5b271dfc3a1141fb51ca3345ca Reviewed-on: https://chromium-review.googlesource.com/952083Reviewed-by:
src-internal-chromium-autoroll <src-internal-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: src-internal-chromium-autoroll <src-internal-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#541218}
-
Esmael El-Moslimany authored
Bug: 819123 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I35712fb750e941ff14801c41441289b2591911dd Reviewed-on: https://chromium-review.googlesource.com/951898Reviewed-by:
Scott Chen <scottchen@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#541217}
-
Qiang Xu authored
changes: Move IconDecodeRequest from inner class of ArcPlayStoreSearchResult to a separate class, since this class is also used for in app search. Bug: 818902 Test: tested that play store search still works Change-Id: Iaa53007b363d82d496115abb5fac64eba5109c1d Reviewed-on: https://chromium-review.googlesource.com/950006Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Qiang Xu <warx@google.com> Cr-Commit-Position: refs/heads/master@{#541216}
-
Jao-ke Chin-Lee authored
Bug: 731556 Change-Id: Id416d395958f0b3529816f17ca0d35203f863065 Reviewed-on: https://chromium-review.googlesource.com/950023Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#541215}
-
Xianzhu Wang authored
- Guard DCHECK-only code with DCHECK_IS_ON() to reduce several instructions in official build. - Inline GrowCurrentBeginItemVisualRect(). Bug: 803867 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I259049465bf0505a4098feac29d73405c922861d Reviewed-on: https://chromium-review.googlesource.com/951897Reviewed-by:
vmpstr <vmpstr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#541214}
-
Shakti Sahu authored
Added basic support to download home to be able to fetch thumbnails from the offline pages backend. It doesn't have any caching which will be added in future CLs. Bug: 817853 Change-Id: I2885f649fb240fa9a28328b6af83a7e49897b092 Reviewed-on: https://chromium-review.googlesource.com/946904Reviewed-by:
Xing Liu <xingliu@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Commit-Queue: Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#541213}
-
David McAllister authored
On devices which already match the media requirements, the test fails as handleMediaChange is never called. Instead, set the pointer and hover types to None so that updating the types later is guaranteed to trigger the matchMedia handler. Bug: None Change-Id: I8fa91ecef9c1f372d1b04abfbc540a7b9c0fc3c2 Reviewed-on: https://chromium-review.googlesource.com/951860Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Cr-Commit-Position: refs/heads/master@{#541212}
-
Charlie Andrews authored
Because we do a slow flush when we reopen a connection, this prevents a ~50ms startup cost if a connection is already open, which is important especially when recording clock sync markers. I've verified locally that this fixes a ~50ms clock sync latency regression. TBR=nednguyen@google.com Bug: 816470 Change-Id: Icab77977faa61755945883cf50613bfe1e575182 Reviewed-on: https://chromium-review.googlesource.com/946674 Commit-Queue: Charlie Andrews <charliea@chromium.org> Reviewed-by:
Charlie Andrews <charliea@chromium.org> Cr-Commit-Position: refs/heads/master@{#541211}
-
Esmael El-Moslimany authored
Bug: 796650 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I1dc50ba6eb8e7b2f01992fdcee5ade2d7c6f35e1 Reviewed-on: https://chromium-review.googlesource.com/940432 Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#541210}
-
Chris Kuiper authored
This allows an app (Chromecast, e.g.) to change the multiplier and better control ducking behavior. Bug: internal b/69630398 Test: Ran on Android Things speaker with different multipliers, observed proper ducking volume. Change-Id: I5078fd11606a5d648aca9b17938c3c8caefd3826 Reviewed-on: https://chromium-review.googlesource.com/942055Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Stephen Lanham <slan@chromium.org> Reviewed-by:
Mounir Lamouri <mlamouri@chromium.org> Commit-Queue: Chris Kuiper <ckuiper@chromium.org> Cr-Commit-Position: refs/heads/master@{#541209}
-
Jani Hautakangas authored
This CL fixes issue with anonymous function calling a member function without object in GCC version < 6.4. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61636 Bug: 819294 Change-Id: I29a9e14a5160a3c9c658dc76f7f167c74e1c2b41 Reviewed-on: https://chromium-review.googlesource.com/945995 Commit-Queue: Jani Hautakangas <jani.hautakangas@lge.com> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#541208}
-
https://pdfium.googlesource.com/pdfium.git/+log/9600a771999d..a32145f1f16b $ git log 9600a7719..a32145f1f --date=short --no-merges --format='%ad %ae %s' 2018-03-06 dsinclair Rename core/fxcrt IFX files to Iface Created with: roll-dep src/third_party/pdfium 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: I50775e5dd706eb1bb41406688324858086da365a Reviewed-on: https://chromium-review.googlesource.com/951825Reviewed-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@{#541207}
-
https://webrtc.googlesource.com/src.git/+log/8d9dcb1c8935..12eb85881c86 $ git log 8d9dcb1c8..12eb85881 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/webrtc BUG=None,None,None The AutoRoll server is located here: https://webrtc-chromium-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=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg TBR=webrtc-chromium-sheriffs-robots@google.com Change-Id: I4a9593a5506beb7a1e89cb67da61c089a69405db Reviewed-on: https://chromium-review.googlesource.com/951891Reviewed-by:
webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#541206}
-
Fredrik Söderquist authored
This switches SVGResource away from inheriting IdTargetObserver, and makes it have one as a member instead. This should make future subclassing of SVGResource easier (avoiding GC mixins.) It also has the side-effect of allowing the SVGURIReference framework to be reused. Bug: 769774 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I77fed36a06b846f3cd4cbb7722faa3de1b081f59 Reviewed-on: https://chromium-review.googlesource.com/947956Reviewed-by:
Stephen Chenney <schenney@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#541205}
-
Peng Huang authored
Bug: 819267 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I14ef4bab4b43a64f959d12bb82ceb82095e5a261 Reviewed-on: https://chromium-review.googlesource.com/951896Reviewed-by:
weiliangc <weiliangc@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#541204}
-
Brandon Jones authored
This covers the basic feature set needed for the simplified input API. Minor details like naming may shift as we continue discussing the API shape, but the majority of this code is likely to remain relevant. Bug: 812287 Change-Id: I41c452f015c2dec08f123e6970920058fb2a728a Reviewed-on: https://chromium-review.googlesource.com/919150 Commit-Queue: Brandon Jones <bajones@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Reviewed-by:
Bill Orr <billorr@chromium.org> Cr-Commit-Position: refs/heads/master@{#541203}
-
Aaron Leventhal authored
Bug: 818216 Change-Id: If959c872723ce79ba03adee13c247cf8382f8551 Reviewed-on: https://chromium-review.googlesource.com/951365 Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#541202}
-
Chris Palmer authored
This reverts commit 2b0c7b40. Reason for revert: https://chromium-review.googlesource.com/950466 re-fixed the crashpad bug, so we should be good to land this again. Original change's description: > Revert "Add range checks for `StringPiece`." > > This reverts commit c8bccdf1. > > Reason for revert: This change has r540894 as dependency. However, r540894 was reverted in r541006, just before this patch landed. Since this currently causes failures on Mac due to crashpad errors, I am reverting this change for now. > > Original change's description: > > Add range checks for `StringPiece`. > > > > Range check `operator[]`, `front`, `back`, and `remove_{pre,suf}fix`. > > > > Bug: 817982,818376,818844 > > Change-Id: I3a0b560af273c9b04237a18a1bc0b56283d3d824 > > Reviewed-on: https://chromium-review.googlesource.com/945049 > > Commit-Queue: Chris Palmer <palmer@chromium.org> > > Reviewed-by: Daniel Cheng <dcheng@chromium.org> > > Reviewed-by: Mark Mentovai <mark@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#541012} > > TBR=palmer@chromium.org,dcheng@chromium.org,mark@chromium.org > > Change-Id: I7d5c2ca050869aff8dca4ba6b724761172aa1d83 > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 817982, 818376, 818844 > Reviewed-on: https://chromium-review.googlesource.com/950822 > Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org> > Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> > Cr-Commit-Position: refs/heads/master@{#541060} TBR=palmer@chromium.org,dcheng@chromium.org,mark@chromium.org,jdoerrie@chromium.org Change-Id: I50637027781401fc96b75bb749c5baa80dcc14a8 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 817982, 818376, 818844 Reviewed-on: https://chromium-review.googlesource.com/951868Reviewed-by:
Chris Palmer <palmer@chromium.org> Commit-Queue: Chris Palmer <palmer@chromium.org> Cr-Commit-Position: refs/heads/master@{#541201}
-
Ivan Afanasyev authored
Subresource filter may skip the first page due to race condition. In order to reproduce the issue add a |sleep| function call inside |CreateOrOpenHelper::RunWork| for an indexed ruleset file. A sequence of events for ruleset file setup is: 1. |RulesetService::OpenAndPublishRuleset| posts a |CreateOrOpen| task from UI thread to task schedulers blocking pool (BP) on browser start. 2. |RulesetService| receives a result of a |CreateOrOpen| task in |RulesetService::OnOpenedRuleset| on UI thread and calls (with a long callstack) |VerifiedRulesetDealer::Handle::SetRulesetFile| 3. |VerifiedRulesetDealer::Handle::SetRulesetFile| is called on UI thread. It posts |VerifiedRulesetDealer::SetRulesetFile| task to BP. i.e. UI (start) -> BP (open file) -> UI (delegating) -> BP (setup dealer) A sequence of events for checking of ruleset file is: 1. |AsyncDocumentSubresourceFilter| is created on navigation response processing on UI thread. It posts |Core::Initialize| task to BP. 2. |AsyncDocumentSubresourceFilter::Core::Initialize| checks dealer the verified dealer data. i.e. UI (nav. response) -> BP (check dealer) Navigation response on UI is always happened after the start. If navigation response happens before the open file result delivered to UI thread the first page will not get subresource filter feature. Otherwise subresource filter works properly. The proposed fix is to eliminate "UI (delegating)" stage and perform file opening and dealer setup together using |VerifiedRulesetDealer::Handle::TryOpenAndSetRulesetFile|. The fix shouldn't affect the first page open time so as |ActivationStateComputingNavigationThrottle| already waits for ruleset file opening due to operations ordering on BP. Please let me know if I should fail a bug. I haven't found this issue in the tracker. Change-Id: If54b9cdfddd8adb9b17cb6da80d8199721ff669b Bug: 817308 Reviewed-on: https://chromium-review.googlesource.com/939161Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#541200}
-
Artur Khachatryan authored
Change-Id: I76f7fe718913f3e7e2c8bef1d1e5463d0d31c240 Reviewed-on: https://chromium-review.googlesource.com/951962Reviewed-by:
Jonathon Kereliuk <kereliuk@chromium.org> Commit-Queue: Artur Khachatryan <khachatryan@chromium.org> Cr-Commit-Position: refs/heads/master@{#541199}
-
Adam Goode authored
Subpixel positioning test overrides are inconsistent between platforms. While not a perfect solution, this change makes it possible to at least explicitly enable/disable subpixel positioning in tests. This is necessary for getting the unit tests to work on all platforms in crrev.com/c/933805. Bug: 740385 Change-Id: I336231170306bb44c5feaae6daf9439e89049f47 Reviewed-on: https://chromium-review.googlesource.com/944012Reviewed-by:
Emil A Eklund <eae@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Commit-Queue: Emil A Eklund <eae@chromium.org> Cr-Commit-Position: refs/heads/master@{#541198}
-
Dmitry Gozman authored
... by removing newly created tabs in test cases to not affect other test cases. TBR=rdevlin.cronin@chromium.org Bug: 818899 Change-Id: I76178410e47153bb660d4dcaeea0e27bf77ffdbb Reviewed-on: https://chromium-review.googlesource.com/951630 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#541197}
-
Eugene Ostroukhov authored
Bug: 817970 Change-Id: Ia7f3ccacf3231011e7955cee4766035e481b72be Reviewed-on: https://chromium-review.googlesource.com/947255 Commit-Queue: Eugene Ostroukhov <eostroukhov@chromium.org> Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#541196}
-
Jiaquan He authored
Preflight for app list migration. Changes the test order to mactch future logic. Hides search model classes in model updater classes. Bug: 733662 Change-Id: I91a4ca70cb93be08728ec726b357b44eb9c8500d Reviewed-on: https://chromium-review.googlesource.com/950411 Commit-Queue: Jiaquan He <hejq@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#541195}
-
Devlin Cronin authored
We should (finally) remove support for manifest v1. One of the last significant dependencies is that a lot of our tests aren't manifest v2 yet. This CL updates the majority of manifest files under extensions/test/data/manifest_tests (which are not usually named manifest.json). All these manifests updated cleanly. Bug: 816679 Change-Id: I5d36f66ac19541b413915a027297ca7cfd359604 Reviewed-on: https://chromium-review.googlesource.com/950231Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#541194}
-
Joel Hockey authored
Bug: 801311 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: Ibf86a107fe86e95f49b15e54668223ecf0b5df10 Reviewed-on: https://chromium-review.googlesource.com/948163 Commit-Queue: Joel Hockey <joelhockey@chromium.org> Reviewed-by:
Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Cr-Commit-Position: refs/heads/master@{#541193}
-
Alexey Kozyatinskiy authored
This CL is preparation step before attempt to fix existing issues. Bug: chromium:812438 Change-Id: I0afa7c3a146d43d0178ea81d5c9aa77e60fcaa2a Reviewed-on: https://chromium-review.googlesource.com/951627 Commit-Queue: Aleksey Kozyatinskiy <kozyatinskiy@chromium.org> Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Cr-Commit-Position: refs/heads/master@{#541192}
-
Scott Chen authored
Bug: 817713 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I8c2723eb80a9301e74048df4e2b1e65a03b8ea61 Reviewed-on: https://chromium-review.googlesource.com/943523Reviewed-by:
Hector Carmona <hcarmona@chromium.org> Commit-Queue: Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#541191}
-
Darren Shen authored
There are still a couple of direct calls to InputMethodManager in ash/login/ui. Under mash, InputMethodManager will not be available from ash/login/ui, so we'll need to use Mojo instead. We replace calls to InputMethodManager with calls to the existing ImeController::IsCapsLockEnabled() and ImeController::Observer code. Bug: 819087 Change-Id: I7d0b351cf4fac1b11acb1b15f3f4d804f3d581c4 Reviewed-on: https://chromium-review.googlesource.com/950582Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Darren Shen <shend@chromium.org> Cr-Commit-Position: refs/heads/master@{#541190}
-
Charles Harrison authored
The WebContentsImpl is the only consumer of this information, and it only cares about the host. URLs can be huge (up to 2MB), while hosts are much much smaller. It isn't a good idea to copy the URL for every outgoing request to the UI thread every 250ms. Note: previously this code did some grouping by RVid (I think) before thread hopping. This would further alleviate copying issues when there are lots of requests. Probably we could just use two sets: 1. Global frame routing id -> LoadInfo for subresources 2. All navigation requests Bug: 813445 Change-Id: Ied3709fac7bba079d5e31694dc8e477d1c7f2f68 Reviewed-on: https://chromium-review.googlesource.com/951364Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#541189}
-
Khushal authored
Animating images in the compositor is now enabled by default. This change removes the feature flag and removes dead code in blink for driving animations in blink. As a part of this, tests have been updated as follows: 1) If the test verifies logic specifically pertaining to invalidation for non-svg animated image formats, they have been removed. Since invalidations for these images is driven by the compositor, these are unnecessary. 2) For tests which verify blink's paint invalidation for offscreen animated images, the logic under test is agnostic to the image format and also applies to animated svgs. If the test only used GIFs, it has been updated to use animated SVGs instead. 3) For tests, namely drawing an animated image on canvas and updating the animation policy, they have been switched to use a timeout instead of advancing the animation using the Internals API. This ensures they still serve as integration tests for these use-cases. R=chrishtr@chromium.org, piman@chromium.org Bug: 776216 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ic0c5ea080e166aaa5732af81ce497cff6b2b6e4e Reviewed-on: https://chromium-review.googlesource.com/944968Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Khushal <khushalsagar@chromium.org> Cr-Commit-Position: refs/heads/master@{#541188}
-
Xianzhu Wang authored
The function took 15% of total PaintChunksToCcLayer::ConvertInto() time. There were two function calls: FloatRect::EnclosingIntRect() which in turn called IntRect::IntRect(const FloatRect&). Cluster-telemetry showed about 1~2.5% improvement of record times (https://ct.skia.org/results/cluster-telemetry/tasks/chromium_perf_runs/wangxianzhu-20180306062045/html/index.html). Bug: 803867 Change-Id: If69f2c4c207ccc1025035f018c63e14c3b078462 Reviewed-on: https://chromium-review.googlesource.com/950104 Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Cr-Commit-Position: refs/heads/master@{#541187}
-
Chromium WPT Sync authored
Using wpt-import in Chromium d718df0c. With Chromium commits locally applied on WPT: 14350ac7 "[PE] Don't truncate table cell's padding" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/13409 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: jinho.bang@samsung.com, mathp@chromium.org, rouslan@chromium.org: external/wpt/payment-handler TBR=jsbell No-Export: true Change-Id: I17e5752e8977b22e534e0cc83ac93487c8d864b0 Reviewed-on: https://chromium-review.googlesource.com/951822 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by:
Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#541186}
-
Andrew Grieve authored
It now lives at //third_party/android_ndk Bug: 755920 Change-Id: I8ba6ee6cfb97fdca3aed3e43005f26ba540b3d61 Reviewed-on: https://chromium-review.googlesource.com/949098Reviewed-by:
Shenghua Zhang <shenghuazhang@chromium.org> Commit-Queue: agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#541185}
-
iOS Autoroller authored
TBR=smut Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Id50cf5499e8185f84b4f3d47231f21d37b8599bf Reviewed-on: https://chromium-review.googlesource.com/951802 Commit-Queue: iOS Autoroller <ios-autoroller@chromium.org> Reviewed-by:
iOS Autoroller <ios-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#541184}
-
Joshua Peraza authored
This change was included in https://chromium-review.googlesource.com/c/chromium/src/+/947745 and was reverted by https://chromium-review.googlesource.com/c/chromium/src/+/950343 but is still needed by https://chromium-review.googlesource.com/c/chromium/src/+/945049 Bug: chromium:817982, chromium:818376 Change-Id: I4f97d632ba9f86c17d3df55c404c0ae14e1d39c9 Reviewed-on: https://chromium-review.googlesource.com/950466Reviewed-by:
Chris Palmer <palmer@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Joshua Peraza <jperaza@chromium.org> Cr-Commit-Position: refs/heads/master@{#541183}
-
Stephen Martinis authored
Bug: 818832 Change-Id: Icad3529d4b143eb1798250d3e96f36a23dbd23c8 Reviewed-on: https://chromium-review.googlesource.com/950322Reviewed-by:
Nodir Turakulov <nodir@chromium.org> Commit-Queue: Stephen Martinis <martiniss@chromium.org> Cr-Commit-Position: refs/heads/master@{#541182}
-
Lily Houghton authored
This is a follow up to https://chromium-review.googlesource.com/c/chromium/src/+/857421 that renames the PROXY_SERVICE_* event types. Bug: 613637 Change-Id: Iabe3c5706952b45c9a3c253feed290d5a872f13b Reviewed-on: https://chromium-review.googlesource.com/944001Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Lily Houghton <lilyhoughton@chromium.org> Cr-Commit-Position: refs/heads/master@{#541181}
-