- 11 Feb, 2018 18 commits
-
-
David 'Digit' Turner authored
This CL adds a small unit-test for the ZIP parsing code. To achieve this, the FileDescriptor class is also slightly improved: - It now always wraps a single integer value, instead of using void* when unit-testing. - The Read() method now returns an ssize_t value. - The SeekTo() method now returns an off_t value. - The ReadFull() method was added. - The Map() method now returns nullptr on failure, instead of MAP_FAILED, and the mock implementation was fixed. - More importantly, the GetFileSize() method was added to avoid calling stat() or fstat() directly in the zip parser, making it unit-testable!! + A new script is added to generate manually a set of data tables corresponding to various test zip archives embedded directly into the unit-test source code (see generate_zip_test_tables.sh) BUG=NONE R=pasko@chromium.org,agrieve@chromium.org,rmcilroy@chromium.org Change-Id: I095aa21ba26d924f7b95cf826d606875bd7d4591 Reviewed-on: https://chromium-review.googlesource.com/911408Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: David Turner <digit@chromium.org> Cr-Commit-Position: refs/heads/master@{#536008}
-
Rohit Rao authored
BUG=811075 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I4c6ab0393b1051807d3e9886fd08556e6fdb67a3 Reviewed-on: https://chromium-review.googlesource.com/911736Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#536007}
-
https://skia.googlesource.com/skia.git/+log/c5cecdcaf8ab..78c110e2f0fa $ git log c5cecdcaf..78c110e2f --date=short --no-merges --format='%ad %ae %s' 2018-02-09 caryclark resolve bookmaker refs fixes Created with: roll-dep src/third_party/skia 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;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 TBR=caryclark@chromium.org Change-Id: If19828715f27fedc1bdd488fd65fe4218db29bb7 Reviewed-on: https://chromium-review.googlesource.com/913148 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#536006}
-
Kush Sinha authored
Enable Mirror account reconciliation on Chrome OS if the preference |kAccountConsistencyMirrorRequired| is set. This is a Chrome OS specific preference and was being used to stop sign in from secondary Google accounts and sign out from the primary account. Enabling Mirror here will have the effect of users' cookies getting re-minted if they clear their cookies from the Settings page. Bug: 805519 Test: out/ChromeOS/unit_tests --gtest_filter="*AccountConsistencyModeManagerTest*" Change-Id: Ie8fe9c4be9868d1301f6063dc84764a1c9dea028 Reviewed-on: https://chromium-review.googlesource.com/906430 Commit-Queue: Kush Sinha <sinhak@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#536005}
-
https://skia.googlesource.com/skia.git/+log/954295bc700d..c5cecdcaf8ab $ git log 954295bc7..c5cecdcaf --date=short --no-merges --format='%ad %ae %s' 2018-02-09 brianosman Remove image functions from GrGLInterface, part one Created with: roll-dep src/third_party/skia 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;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 TBR=caryclark@chromium.org Change-Id: Ie300196e258f5c344d6d8aa921a23d77e33751a7 Reviewed-on: https://chromium-review.googlesource.com/911978 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#536004}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ifdb2041f20f777b06180bc981786d1c6777eb778 Reviewed-on: https://chromium-review.googlesource.com/911999 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#536003}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win10_chromium_x64_rel_ng Change-Id: I301857ed92c18e80943ac4b6abab062c839b332f Reviewed-on: https://chromium-review.googlesource.com/911998 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#536002}
-
yiyix authored
In this patch, I added some metrics to measure effectiveness of draw occlusion so that we can get more meaningful data from finch trials. Metrics added includes measuring the number of quads that are removed or resized, total area of drawing that are skipped, and total number draw quads that need to go through complex transforms. Bug:672929 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I354c95a3ac5525ab81694ad67f4f1b183c86f0cb Reviewed-on: https://chromium-review.googlesource.com/898342 Commit-Queue: Yi Xu <yiyix@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
danakj <danakj@chromium.org> Cr-Commit-Position: refs/heads/master@{#536001}
-
https://skia.googlesource.com/skia.git/+log/193d3a051400..954295bc700d $ git log 193d3a051..954295bc7 --date=short --no-merges --format='%ad %ae %s' 2018-02-11 update-docs Update markdown files Created with: roll-dep src/third_party/skia 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;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 TBR=caryclark@chromium.org Change-Id: Ie262a2ca81f4047c12fbbf7168a7e3089c35234e Reviewed-on: https://chromium-review.googlesource.com/912779Reviewed-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@{#536000}
-
Sylvain Defresne authored
Use browsing_data::RecordDeletionForPeriod() to record the action instead of re-implementing it and use the function browsing_data::CalculateEndDeleteTime to correctly compute the end of the time period (fix the "older than 30 days" time period). Remove unnecessary code to clean up delete_end_ as the corresponding issue is long fixed. Bug: none Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Ib1cf0287daad9f4228a1d2c146e0f8f2028e16cd Reviewed-on: https://chromium-review.googlesource.com/911885Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#535999}
-
Ken Rockot authored
and struct_traits => mojom_traits TBR=dcheng@chromium.org TBR=asvitkine@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: Ic201838785b6613659bea1dab6a3f59b070e5e9a Reviewed-on: https://chromium-review.googlesource.com/912733 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535998}
-
Bence Béky authored
HttpBasicState::Initialize() always succeeds, so remove callback argument and change return type to void. Why convert CompletionCallback to CompletionOnceCallback if one can safely remove it instead? :) Bug: 807724 Change-Id: I16244d0600a278139ba2e112373f95933245a1b2 Reviewed-on: https://chromium-review.googlesource.com/908635Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#535997}
-
Bence Béky authored
Bug: 807724 Change-Id: Ieb4afbaf6986261bbd795051b18a8eb148aa0ad1 Reviewed-on: https://chromium-review.googlesource.com/908868Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#535996}
-
Ken Rockot authored
and struct_traits => mojom_traits TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Ia252dcac3d6e0c00ab143871e66b7eec13a66e81 Reviewed-on: https://chromium-review.googlesource.com/912778 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535995}
-
Ken Rockot authored
and struct_traits => mojom_traits TBR=dcheng@chromium.org TBR=sky@chromium.org Bug: 806965 Change-Id: I8f8aca7d28c654378792d02d9ad272d226763cb2 Reviewed-on: https://chromium-review.googlesource.com/912517Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535994}
-
Bence Béky authored
Change logging in StaticSocketDataHelper::VerifyWriteData() such that if writted data does not match mock write, then print actual written data in full length, not only truncated to the length of the mock write data. Remove unused StaticSocketDataProvider:CompleteRead(). The rest of this CL belongs to https://crbug.com/810752: Inline ConstructSpdyGoAway() as ConstructSpdyGoAway(0). Remove unused struct SpdyHeaderInfo. Remove obsolete "The caller takes ownership of the frame." comment from spdy_test_util_common.h, leftover from when methods used to return raw pointers. This is a follow-up to https://crrev.com/2156643002/diff/1/net/spdy/spdy_test_util_common.h. Also remove useless "Returns a SpdySerializedFrame." comments from the line above the definition showing return type SpdySerializedFrame. Bug: 600438, 810752 Change-Id: I1f3471f4e4a9b71603257cf82986d0efe0288c7f Reviewed-on: https://chromium-review.googlesource.com/912013Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Bence Béky <bnc@chromium.org> Cr-Commit-Position: refs/heads/master@{#535993}
-
Kenneth Russell authored
Start by copying nearly all of the bots' descriptions into new names. TBR=dpranke@chromium.org, machenbach@chromium.org, jmadill@chromium.org, ynovikov@chromium.org Bug: 792780 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: I0e9f564e9dc87124578f5b699b64011ff9b952da Reviewed-on: https://chromium-review.googlesource.com/912767 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by:
Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#535992}
-
Noel Gordon authored
Similar to crrev.com/534964, since crc32 can be freely called per the "zlib.h" docs, add comment about zlib crc32 usage conventions viz., calling crc32(0, NULL, 0) before use and how we use that as signal to cache CPU features. Tbr: cavalcantii@chromium.org Bug: 796178 Change-Id: I3c80149ea9c09baeb5a376d54ffeb4e36b12b5ec Reviewed-on: https://chromium-review.googlesource.com/912790Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#535991}
-
- 10 Feb, 2018 22 commits
-
-
Chromium WPT Sync authored
Using wpt-import in Chromium 3b1891a0. Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/12320 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=leon.han@intel.com No-Export: true Change-Id: I919462ad2ead0c81343da55034b9993c4fe5694c Reviewed-on: https://chromium-review.googlesource.com/912114 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@{#535990}
-
Xiaocheng Hu authored
Change-Id: I6be5e1344f831b4b63d522084eace80799398115 Reviewed-on: https://chromium-review.googlesource.com/912462Reviewed-by:
Fredrik Söderquist <fs@opera.com> Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org> Cr-Commit-Position: refs/heads/master@{#535989}
-
Kim Paulhamus authored
Bug: 664630 Change-Id: Ie7fb418978c6d7194f071ead353084cc59ffb677 Reviewed-on: https://chromium-review.googlesource.com/912204Reviewed-by:
Mike West <mkwst@chromium.org> Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org> Cr-Commit-Position: refs/heads/master@{#535988}
-
Steven Holte authored
These will be required for implementing UKMs SourceUrlRecorder for iOS. Bug: 792662 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I110d51ddf79dd1e08e4759721387d21cf280c439 Reviewed-on: https://chromium-review.googlesource.com/887954 Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#535987}
-
https://skia.googlesource.com/skia.git/+log/32750e6fb73f..193d3a051400 $ git log 32750e6fb..193d3a051 --date=short --no-merges --format='%ad %ae %s' 2018-02-08 reed use SkShaper instead of SkTextBox Created with: roll-dep src/third_party/skia 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;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 TBR=bsalomon@chromium.org Change-Id: I74cbbaced485d6a8118bcd0575185f45a8197e02 Reviewed-on: https://chromium-review.googlesource.com/912134Reviewed-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@{#535986}
-
Ken Rockot authored
And any struct_traits => mojom_traits Brought to you by grep and sed TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo;master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I3782b0e0fbfe438678a9aeeaefd2922fbd19d0eb Reviewed-on: https://chromium-review.googlesource.com/912511Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535985}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I0f64f2c2ac39f27fe175f9e71bb627de91fd7b13 Reviewed-on: https://chromium-review.googlesource.com/912775 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535984}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=sky@chromium.org Bug: 806965 Change-Id: Ia1eb1567f60ea4e5e9ac3a3ec98052c575767501 Reviewed-on: https://chromium-review.googlesource.com/912773Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535983}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: I15f64c901705cc2573e2c36b78af4cb6158f0104 Reviewed-on: https://chromium-review.googlesource.com/912772 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535982}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: I6fe6abafbdf5d51b633489440d72ab6073af59aa Reviewed-on: https://chromium-review.googlesource.com/912771 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535981}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: Ie3ae003a0e2aeca38b6ead9aa82e5399e7b0ac86 Reviewed-on: https://chromium-review.googlesource.com/912770 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535980}
-
Ken Rockot authored
TBR=jam@chromium.org TBR=dcheng@chromium.org Bug: 806965 Change-Id: I5ada864d9ce5ab638d23ba5177140b1bb0415b9c Reviewed-on: https://chromium-review.googlesource.com/912769 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535979}
-
Eugene But authored
Rename TestPreClassificationCheckInPageNavigation to TestPreClassificationCheckSameDocumentNavigation. Old name was presumably used to match was_within_same_page IPC, but SameDocument name better reflects the the type of the navigation (navigation did not change the document object). This change will make naming more consistent with the rest of Chromium code. Bug: 695189 Change-Id: Ib0e1229d1730d6ac01038aed112a47d9eea78015 Reviewed-on: https://chromium-review.googlesource.com/912451Reviewed-by:
Nathan Parker <nparker@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#535978}
-
https://skia.googlesource.com/skia.git/+log/77966ea37d0e..32750e6fb73f $ git log 77966ea37..32750e6fb --date=short --no-merges --format='%ad %ae %s' 2018-02-10 angle-skia-autoroll Roll skia/third_party/externals/angle2/ 6108b766b..8f27b0509 (1 commit) Created with: roll-dep src/third_party/skia 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;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 TBR=bsalomon@chromium.org Change-Id: I7e279d8fb701e21e63d94c758786266ab43a3864 Reviewed-on: https://chromium-review.googlesource.com/911975 Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#535977}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: I98f89d62af5defb342eb62ea308b820e55b8401c Reviewed-on: https://chromium-review.googlesource.com/912768 Commit-Queue: Ken Rockot <rockot@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535976}
-
Ken Rockot authored
TBR=dcheng@chromium.org TBR=jam@chromium.org Bug: 806965 Change-Id: I8ffbfcaa998bfdfca4b037848309315324bd8dfd Reviewed-on: https://chromium-review.googlesource.com/911886Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Ken Rockot <rockot@chromium.org> Cr-Commit-Position: refs/heads/master@{#535975}
-
Michael Giuffrida authored
BrowserView::CanClose() hides the browser frame before closing the browser. This causes the active WebContents to log an extraneous TabManager.TabMetrics event, thinking the tab was backgrounded. Unless the window was actually minimized, the active WebContents shouldn't log a backgrounded event when hidden. Bug: 784639 Change-Id: I211e37bf169112982c1130dffb7c502d83c5e9db Reviewed-on: https://chromium-review.googlesource.com/905095 Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#535974}
-
Michael Giuffrida authored
TabMetricsLogger generally shouldn't be invoked to log a UKM for a background tab if the tab is closing, but there are lots of potential edge cases where this could happen. If the browser window is being closed, early exit instead of logging. Bug: 784639 Change-Id: Ic7f39f1f0cbfbba1dd867195c91365afb630dc61 Reviewed-on: https://chromium-review.googlesource.com/905794Reviewed-by:
François Doray <fdoray@chromium.org> Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Cr-Commit-Position: refs/heads/master@{#535973}
-
Michael Giuffrida authored
TabActivityTestBase simulates tab navigation and tab switching in a few unit tests. Split it into its own file, since it's not directly related to UKM. Also make it a standalone class instead of a test base class. It was originally a test itself, then spun out into a base class. Now it uses nothing from ChromeRenderViewHostTestHarness except profile(), which can be gotten from the TabStripModel. This makes the class's purpose clearer and allows the UKM helper in tab_ukm_test_helper.h to be used in browser_tests without including extra test classes. The remaining class, UkmEntryChecker, should probably be moved into a renamed file somewhere else in the future, depending on how useful it is for other teams. Bug: 784639 Change-Id: I62ca7bb1feaa00e06a39241741696d1f217ed26b Reviewed-on: https://chromium-review.googlesource.com/903582 Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#535972}
-
Greg Thompson authored
The two implementations of this class each had their own member representing the time that an update was detected. This is good to have in the base class so that consumers of UpgradeDetector can make use of it. BUG=511300 Change-Id: Ib0e9a36f309fb630b5ec34e317eed6388873239b Reviewed-on: https://chromium-review.googlesource.com/908449Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#535971}
-
Alexei Filippov authored
The interface allows various clients to collect extra data on samples. E.g. DevTools heap profilers may collect JS stacks. BUG=803276 Change-Id: I34e90a7df59d11f726564671b54f31e1a6f8cfa4 Reviewed-on: https://chromium-review.googlesource.com/911970 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#535970}
-
Alexei Filippov authored
The Memory.getBrowserSamplingProfile command is introduced. BUG=803276 Change-Id: I4cc1ede9ace572971102b681f5d9e9bc693e08f0 Reviewed-on: https://chromium-review.googlesource.com/912323 Commit-Queue: Alexei Filippov <alph@chromium.org> Reviewed-by:
Pavel Feldman <pfeldman@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#535969}
-