- 24 Jul, 2018 40 commits
-
-
Derek Cheng authored
It is flaky as part of network_service_browser_tests on Linux Tests (dbg)(1). TBR=dvadym Bug: 347159 Change-Id: I72e204f87334d94723c26b998d4b649e187c78f0 Reviewed-on: https://chromium-review.googlesource.com/1148774Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#577639}
-
tzik authored
ChainedBlobWriterImpl is a ref-counted object, and its first reference used to be taken by base::BindOnce implicitly in its constructor. The reference is posted to another thread, and released after the task has run. However, if the task finished before the constructor finished, the object is destroyed immediately, and `new ChainedBlobWriterImpl` may return a stale pointer. This CL adds a static constructor to ChainedBlobWriterImpl to keep a reference while the task is posted. Bug: 866456 Change-Id: If8fee2e8944ce550e766b5269d47878f357db6c5 Reviewed-on: https://chromium-review.googlesource.com/1148098Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Commit-Queue: Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#577638}
-
John Abd-El-Malek authored
The problem was different timing of loading messages. Normally, DocumentLoaderImpl::DidRead is called very early which calls into DocumentLoaderImpl::SaveBuffer. The latter sees that the request was in |pending_requests_| and then calls OnPendingRequestComplete which loads the information about the document (e.g. number of pages, linearized PDF info). With the network service, OutOfProcessInstance::DidChangeView was being called before the DidRead calls which was causing |pending_requests_| to be reset. The fix is to only reset pending_requests_ in PDFiumEngine::CalculateVisiblePages if we have already loaded the PDF document structure. This was the intended effect of that code; the extra check wasn't needed before the network service because we were always reaching that function after the document info was loaded. Bug: 705114 Change-Id: I4e668ebfdda0c17ede3c6daa951dea0a1b38eb05 Reviewed-on: https://chromium-review.googlesource.com/1147932Reviewed-by:
Lei Zhang <thestig@chromium.org> Commit-Queue: John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#577637}
-
skia-chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/734b2a859784..6a4e60bb8f49 git log 734b2a859784..6a4e60bb8f49 --date=short --no-merges --format='%ad %ae %s' 2018-07-24 robertphillips@google.com Revert "Reduce arbitrary opList splitting when sorting (take 2)" 2018-07-24 egdaniel@google.com Add Metal test bots. 2018-07-24 ethannicholas@google.com Revert "Revert "removed SkSLLayoutLexer"" 2018-07-24 robertphillips@google.com Reduce arbitrary opList splitting when sorting (take 2) Created with: gclient setdep -r src/third_party/skia@6a4e60bb8f49 The AutoRoll server is located here: https://autoroll.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.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=benjaminwagner@chromium.org Change-Id: I892ad89cf41a6ee9525774ce615b479a766eb701 Reviewed-on: https://chromium-review.googlesource.com/1147959Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#577636}
-
Xiangjun Zhang authored
This CL adds implementation to create a network context through Network Service for the Mirroring Service. Bug: 734672 Change-Id: I26422b4f4f5c69bd8398dc3ae3d5ee501c996eea Reviewed-on: https://chromium-review.googlesource.com/1139218Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
oysteine <oysteine@chromium.org> Commit-Queue: Xiangjun Zhang <xjz@chromium.org> Cr-Commit-Position: refs/heads/master@{#577635}
-
Lu Chen authored
After a webpage is loaded, get metrics of anchor elements in the viewport and send them to the browser process. These metrics, together with metrics extracted in the browser process, are used to compute and record anchor element navigation overall score. This navigation score will be used to decide what action (e.g.,preload, prerender) to take, (or choose to do nothing). Bug: 850624 Change-Id: Ieb85bc0b048f5ec6cb174c41e9b3d72639b393c9 Reviewed-on: https://chromium-review.googlesource.com/1119201 Commit-Queue: Lu Chen <chelu@chromium.org> Reviewed-by:
Hayato Ito <hayato@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Robert Kaplow (OOO until 0730) <rkaplow@chromium.org> Reviewed-by:
Ryan Sturm <ryansturm@chromium.org> Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#577634}
-
Mitsuru Oshima authored
IME uses this propery to tell if it's arc windows. With this fix, text input on type type window (e.g. FB ChatHead) works again. BUG=b/111713229 TEST=manual. See bug for repro step. Change-Id: Ifa058f3a4c4c91daf7059311044c4440780f47ad Reviewed-on: https://chromium-review.googlesource.com/1148237Reviewed-by:
Yury Khmel <khmel@chromium.org> Commit-Queue: Mitsuru Oshima <oshima@chromium.org> Cr-Commit-Position: refs/heads/master@{#577633}
-
Ricardo Quesada authored
Fix small typo in a comment. 'pixesl' is replaced by 'pixels' BUG=None Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: Ie916fc278f794eb07b81e71c519972bbcb9d8d16 Reviewed-on: https://chromium-review.googlesource.com/1148599Reviewed-by:
danakj <danakj@chromium.org> Commit-Queue: Ricardo Quesada <ricardoq@chromium.org> Cr-Commit-Position: refs/heads/master@{#577632}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/696e3951..5107ef4f Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: Ifc51ffbc7fd1202a57b98970ce5dc77fa5ab1885 Reviewed-on: https://chromium-review.googlesource.com/1148641Reviewed-by:
V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#577631}
-
Daniel Cheng authored
- GetMemoryAs<T> should be used when the mapping contains one T. - GetMemoryAsSpan<T> should be used when the mapping contains more than one T. The element count can be explicitly specified or autodeduced from the size of the shared memory mapping. Bug: 795291, 866975 Change-Id: Icea4e276a26a1c47688c0a5206b97cd42ff92043 Reviewed-on: https://chromium-review.googlesource.com/1144312Reviewed-by:
danakj <danakj@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Alexandr Ilin <alexilin@chromium.org> Commit-Queue: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#577630}
-
depot-tools-chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/6ec6d27f2d1b..ca7d881540e3 git log 6ec6d27f2d1b..ca7d881540e3 --date=short --no-merges --format='%ad %ae %s' 2018-07-24 ehmaldonado@chromium.org Reland "Reland "gclient_scm: Use cherry-picking instead of rebasing."" Created with: gclient setdep -r src/third_party/depot_tools@ca7d881540e3 The AutoRoll server is located here: https://depot-tools-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. BUG=chromium:850812 TBR=agable@chromium.org Change-Id: I64f7a6ad1c38b3453ef7493fd3789454d58b333b Reviewed-on: https://chromium-review.googlesource.com/1147912Reviewed-by:
depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: depot-tools-chromium-autoroll <depot-tools-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#577629}
-
Marcin Wiacek authored
@IntDef/@StringDef annotation are preferred way for declaring set of String/int values. 1. they need less space in APK than enum, see https://developer.android.com/topic/performance/reduce-apk-size#remove-enums 2. they give more control over allowed values than "static final" values Main goal of patch is writing "static final" values, enum and some classes in one common @IntDef/@StringDef form: 1. with @IntDef/@StringDef first, @Retention second and related @interface third 2. with values inside @interface 3. with NUM_ENTRIES declaring number of entries if necessary 4. with comment about numbering from 0 without gaps when necessary 5. with @Retention(RetentionPolicy.SOURCE) 6. without "static final" in the @interface Change-Id: I2d2f80b5d0b84b1315016903b74f06d10ab6d4e8 Reviewed-on: https://chromium-review.googlesource.com/1146189Reviewed-by:
Egor Pasko (futex_wait(&secret))) <pasko@chromium.org> Commit-Queue: Marcin Wiącek <marcin@mwiacek.com> Cr-Commit-Position: refs/heads/master@{#577628}
-
Derek Cheng authored
TBR=foolip@chromium.org Bug: 866965 Change-Id: If4b3f8c350b4677490f2ad72d47d12437d82c77d Reviewed-on: https://chromium-review.googlesource.com/1148659Reviewed-by:
Derek Cheng <imcheng@chromium.org> Commit-Queue: Derek Cheng <imcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#577627}
-
Jun Mukai authored
BUG=857103 TEST=ash_unittests, unit_tests Change-Id: I46f1b09a2d2ad4a90fe42c75a9d0f587c03ab1cf Reviewed-on: https://chromium-review.googlesource.com/1137232 Commit-Queue: Jun Mukai <mukai@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#577626}
-
Esmael El-Moslimany authored
Bug: 865899 Change-Id: Ic759da99d5dd11f2e08b201b731bab8006da7b38 Reviewed-on: https://chromium-review.googlesource.com/1145625 Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Reviewed-by:
Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#577625}
-
tzik authored
This CL adds a static constructor to ScreenObserverDelegate to avoid a racy ref count operation. As ScreenObserverDelegate is a ref counted class, base::Bind implicitly increments the reference count, and releases it on the callback instance destruction. If PostTask in the SOD ctor fails, or the posted task ran soon before the ctor has completed, the ScreenObserverDelegate instance may be destroyed immediately, and `new ScreenObserverDelegate` may returns a stale pointer. Bug: 866456 Change-Id: I7c64915529786b2f0738731c323aaf2f293dbc20 Reviewed-on: https://chromium-review.googlesource.com/1147896Reviewed-by:
Christian Fremerey <chfremer@chromium.org> Commit-Queue: Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#577624}
-
kristipark authored
Add functionality to add, delete, undo delete, and reset custom links. These can only be called from the Most Visited or the edit custom links iframes. Delete and undo delete will use the existing "deleteMostVisitedItem" and "undoMostVisitedDeletion". Bug: 856394 Change-Id: I737aa72b92225e2e20174351bd431578311a3396 Reviewed-on: https://chromium-review.googlesource.com/1144526 Commit-Queue: Kristi Park <kristipark@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#577623}
-
Samuel Huang authored
This is the last of the series of CLs to add LLD-LTO string literal support to Supersize by parsing BC .o files when LLD-LTO is enabled. Details: - Add ResolveStringPieces() (alongside ResolveStringPiecesIndirect()) to process strings extracted from BC files. - Check each .o files processed to decide if it's a BC file. - nm is still used to process all .o files (including BC files) to extract symbols. - Any BC .o files found are processed by the new code that calls llvm-bcanalyzer. This provides flexibility since some .o files may still be ELF. Bug: 723798 Change-Id: I5b7d1db9f91a2cd6749fe4c57595ac46eee664a8 Reviewed-on: https://chromium-review.googlesource.com/1130006Reviewed-by:
Samuel Huang <huangs@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: Samuel Huang <huangs@chromium.org> Cr-Commit-Position: refs/heads/master@{#577622}
-
Chromium WPT Sync authored
Using wpt-import in Chromium bff8e3ea. With Chromium commits locally applied on WPT: 4bd63c29 "webrtc wpt: close RTCPeerConnection in generateAnswer helper" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/21664 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 TBR=foolip No-Export: true Change-Id: I7360a6b3716d213b4f87c06283e282814b55960a Reviewed-on: https://chromium-review.googlesource.com/1148583 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@{#577621}
-
Ryan Hamilton authored
Merge internal change 205691599. BUG=865809 Change-Id: I0369074ac0c1e754cc350c37a47681101e35cf12 Reviewed-on: https://chromium-review.googlesource.com/1148640Reviewed-by:
Zhongyi Shi <zhongyi@chromium.org> Commit-Queue: Zhongyi Shi <zhongyi@chromium.org> Cr-Commit-Position: refs/heads/master@{#577620}
-
Andrew Luo authored
Bug: 857590 Change-Id: I2e7387d87e0459e331648ddee8e106a0716af8de Reviewed-on: https://chromium-review.googlesource.com/1144522Reviewed-by:
Richard Coles <torne@chromium.org> Commit-Queue: Andrew Luo <aluo@chromium.org> Cr-Commit-Position: refs/heads/master@{#577619}
-
Patrick Monette authored
In some rare circumstances, that function trigger the reconfiguration of a product, which can open the Windows Installer dialog. It is unknown yet why this happens because we have no repro, but the fix was verified with an external user. Instead look for the path of the component directly in the registry. Bug: 860537 Change-Id: Ib1d32a9f8b64f234c20c11d04bb2d825d51f00eb Reviewed-on: https://chromium-review.googlesource.com/1145821Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Commit-Queue: Patrick Monette <pmonette@chromium.org> Cr-Commit-Position: refs/heads/master@{#577618}
-
Scott Violet authored
BUG=842365 TEST=none Change-Id: I4006ea0b9fd59d183620b90aafa182a239e9bfb7 Reviewed-on: https://chromium-review.googlesource.com/1146994Reviewed-by:
James Cook <jamescook@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#577617}
-
Mostyn Bramley-Moore authored
Otherwise gitiles renders these as (broken) relative links. Change-Id: I66fffdf2e9934bfa7ced35f2850d528a2bbfa947 Reviewed-on: https://chromium-review.googlesource.com/1148046Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com> Cr-Commit-Position: refs/heads/master@{#577616}
-
Jan Krcal authored
This CL makes all sync owners own autofill sync bridges. The current owner from the sync team (jkrcal@) is left there to indicate who is the primary POC for this code. Bug: 853688 Change-Id: I0a5d26ccbbd75b86033b853e37fc29a1998050ea Reviewed-on: https://chromium-review.googlesource.com/1148578Reviewed-by:
Sebastien Seguin-Gagnon <sebsg@chromium.org> Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#577615}
-
Daniel Murphy authored
Reviewer: mek@chromium.org Change-Id: Id461e44052dd29538f7f0504ddc9cc342c5bf472 Reviewed-on: https://chromium-review.googlesource.com/1011554Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#577614}
-
tzik authored
ShaderDiskCache's reference count used to be 0 at the beginning of Init(), and be incremented implicitly by base::Bind there. As the reference count is decremented to 0 if CreateCacheBackend finished immediately and the passed callback is destroyed. That causes unexpected destruction of the SharedDiskCache instance. After this CL, the caller of Init() keeps an reference while Init() is running to avoid destroying the ShaderDiskCache instance. Bug: 866456 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: Iddf9daaf09119aed279ffb81f0af1a74efb9dc54 Reviewed-on: https://chromium-review.googlesource.com/1148090Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Taiju Tsuiki <tzik@chromium.org> Cr-Commit-Position: refs/heads/master@{#577613}
-
Dmitry Gozman authored
Only renderer-inititated navigations go through DecidePolicyForNavigation now, so there should be no |pending_navigation_params_| or |extra_data|. This patch removes previous CHECKs and refactors code assuming there is no extra_data and pending_navigation_params_. Bug: 789577 Change-Id: I8ac67b5735e2f4b8976347df593877e0b1d61292 Reviewed-on: https://chromium-review.googlesource.com/1139024 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Cr-Commit-Position: refs/heads/master@{#577612}
-
Mark Pearson authored
This is important for desktop, as >1% of events emitted are hitting the max. By my estimates from looking at the distribution, it's probably safe to change the max to more like 6-10 hours to have fewer than 1% of emits on desktop in the overflow bucket. I decided to use one day as the max simply because it will make it easier to identify possible issues in the future with this metric. (E.g., we should not see two emits to the overflow bucket on a single day, otherwise something is wrong.) While at it, revise Session.TotalDuration description to explain how it changed in M68. BUG=856467 Change-Id: If0fcead52a15af24899618448b9c72bb87fb3e2b Reviewed-on: https://chromium-review.googlesource.com/1147210Reviewed-by:
Chris Hamilton <chrisha@chromium.org> Commit-Queue: Chris Hamilton <chrisha@chromium.org> Cr-Commit-Position: refs/heads/master@{#577611}
-
Daniel Nicoara authored
DRM legacy is never used with overlays due to a lack of synchronization during page flip. Remove the complexity behing trying to page flip overlays on legacy. Also move tests that deal with overlays to use atomic configuration. BUG=822853 TEST=Ran unittests TEST=Ran chrome on Samus to validate legacy path still works TEST=Ran chrome on Eve to ensure atomic path hasn't regressed Change-Id: Ibc82aafd367c8d12106d3a0e835a3b4b38e0161a Reviewed-on: https://chromium-review.googlesource.com/1120692Reviewed-by:
Michael Spang <spang@chromium.org> Commit-Queue: Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#577610}
-
Xiyuan Xia authored
Lock screen via SessionController so that it also works on dev box. Bug: 745006 Change-Id: Ic269926063741be5d876dcbc9188eee7da12604f Reviewed-on: https://chromium-review.googlesource.com/1148654 Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#577609}
-
Anna Malova authored
Bug: 849861 Change-Id: Ia09b1c41f7a631502e14cf29509d6908d8eb7eac Reviewed-on: https://chromium-review.googlesource.com/1092747 Commit-Queue: Anna Malova <amalova@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Cr-Commit-Position: refs/heads/master@{#577608}
-
Daniel Nicoara authored
Video overlays on the Cast platform follow a special path handled by the hardware via a separate API. The Cast overlay strategy was updated in 3737028e to accept the overlay rather than relying on the Ozone platform to make the decision. The logging was moved in OverlayStrategyUnderlayCast and VideoPlaneController. Bug: None Test: Compiled cast_shell Change-Id: Ide14380ad626d7bcd9ba00cb7f74daab292f2f80 Reviewed-on: https://chromium-review.googlesource.com/1134495Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#577607}
-
Andrey Kosyakov authored
Change-Id: Ibbcdcb6f036ee0ab185558bb437542f556658176 Reviewed-on: https://chromium-review.googlesource.com/1148644Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Cr-Commit-Position: refs/heads/master@{#577606}
-
Scott Chen authored
Bug: 855205 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: I424db425db97b90c7374e9bc8c3504d1dbb32b01 Reviewed-on: https://chromium-review.googlesource.com/1147086 Commit-Queue: Scott Chen <scottchen@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#577605}
-
Evan Stade authored
This avoids some recalculations (chances for bugs) in BrowserNonClientFrameViewAsh. The bounds are horizontally inverted (i.e. they are relative to the top right of the frame) to minimize the number of times they need to be updated when resizing the window and the likelihood of ephemeral inconsistencies. Also fix the registered type of kRestoreBoundsOverrideKey. Since it was registered as a primitive, the address itself would get copied across processes. Bug: 854704 Change-Id: Iedd4eb4dfb2172f7a9ba66bbc542db33633aaf77 Reviewed-on: https://chromium-review.googlesource.com/1144058Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#577604}
-
Eric Karl authored
Currently, the OOP-R transfer cache has a 4MB cache on low-end. As the primary use of this cache turned out to be images, with only small additional colorspace and path caches, we should make this more closely match the discardable image cache's low-end size (512KB). Chosing 768KB to allow for small amounts of additional cache items. R=khushalsagar Bug: 866737 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: I2f04ce5bac3d2d94bb73bfd877169947d7aae64d Reviewed-on: https://chromium-review.googlesource.com/1147759Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#577603}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/ca2b32c84463..d269b0e87d1e git log ca2b32c84463..d269b0e87d1e --date=short --no-merges --format='%ad %ae %s' 2018-07-24 anthonyalridge@google.com Initial files for plotting a simple line graph. Created with: gclient setdep -r src/third_party/catapult@d269b0e87d1e 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:866423 TBR=sullivan@chromium.org Change-Id: I6d810da1512eea82945a307c7dc749e8b733e3b5 Reviewed-on: https://chromium-review.googlesource.com/1147958Reviewed-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@{#577602}
-
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/aa5072898003..c9f1234b1982 git log aa5072898003..c9f1234b1982 --date=short --no-merges --format='%ad %ae %s' 2018-07-24 tsepez@chromium.org Remove lpClass argument from FXJSE_RetrieveObjectBinding() Created with: gclient setdep -r src/third_party/pdfium@c9f1234b1982 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: Ib4c1758146d0caf43dc203af81e48afeaf1da5e0 Reviewed-on: https://chromium-review.googlesource.com/1147957Reviewed-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@{#577601}
-
Chris Palmer authored
Bug: None Change-Id: I1d6d5f22162f7ea9bb54fb942a2c72183ad62071 Reviewed-on: https://chromium-review.googlesource.com/1148555 Commit-Queue: Chris Palmer <palmer@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#577600}
-