- 05 Jun, 2019 40 commits
-
-
Marijn Kruisselbrink authored
Also update all references to it, and get rid of some other (partial) copies of the file as well. Tbr: ccameron@chromium.org Bug: none Change-Id: I22f5e59d9d5206a71ffcccb7b9c2a6447bb8dbd5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1639659 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#666491}
-
Peter Kasting authored
* Use ?: * NULL -> nullptr * Inline cheap accesses * Remove unnecessary {} * StringPrintf() -> NumberToString() (faster, will handle size_t more easily) * Init members at declaration * Don't alias * Be consistent about prefix ++ * Rewrap * Spaces around operators * Use find_if() where appropriate * Use range-based for * No || on expressions * Replace indexes with iterators (allows down-counting loops to convert to size_t more safely) * One DCHECK per condition Bug: none Change-Id: If9ab39a1b976217257283bf69857ba71127cdff7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1637940Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Reviewed-by:
Tatiana Gornak <melandory@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#666490}
-
Owen Min authored
DefaultBrowserSettingEnabled is only supported on Windows 7. Update the adm/admx templates' "Supported on" field to "Microsoft Windows 7". Adding a new option named 'chrome.win7' for the supported_on field in policy_templates.json. It is treated as Windows platform with an additional flag during template parsing. And this flag will change the text for the Policy. Bug: 900206 Change-Id: Iac8dace3c4f2d88d45ce27478432b992eb177cd9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644032 Commit-Queue: Owen Min <zmin@chromium.org> Reviewed-by:
Lutz Justen <ljusten@chromium.org> Cr-Commit-Position: refs/heads/master@{#666489}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/6788784ac192..d71ff5db15ed git log 6788784ac192..d71ff5db15ed --date=short --no-merges --format='%ad %ae %s' 2019-06-05 rmhasan@google.com [Telemetry] Add isatty implementation to _LoggingOutputStream Created with: gclient setdep -r src/third_party/catapult@d71ff5db15ed The AutoRoll server is located here: https://autoroll.skia.org/r/catapult-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel BUG=chromium:970335 TBR=skyostil@google.com Change-Id: If1c7b973e82ede274d950ab1c6cc9a5f1915d6f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646893Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#666488}
-
Christopher Thompson authored
The old Security.TimeOnPage histogram was not correctly calculating the time spent on the page. This CL updates the description to clarify what it actually measured (as the old and replacement histograms shouldn't have identical descriptions). Bug: 949370 Change-Id: Ib6497d58780e5e342fa796dd594e73ae1934179f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642225 Auto-Submit: Christopher Thompson <cthomp@chromium.org> Reviewed-by:
Mustafa Emre Acer <meacer@chromium.org> Commit-Queue: Mustafa Emre Acer <meacer@chromium.org> Cr-Commit-Position: refs/heads/master@{#666487}
-
Matthew Webb authored
PinAuth calls the static method MakePinAuth, which does not modify and does not need to copy the base::spans it takes. This CL makes PinAuth const correct and passes everything by const-ref. This change was also made in, but is unrelated to, crrev.com/c/1636723 so I brought it out to another CL. Change-Id: I48c4cd470e667766fd544774746de4ed04b9f99c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642238 Commit-Queue: Matthew Webb <noviv@google.com> Commit-Queue: Martin Kreichgauer <martinkr@google.com> Auto-Submit: Matthew Webb <noviv@google.com> Reviewed-by:
Martin Kreichgauer <martinkr@google.com> Cr-Commit-Position: refs/heads/master@{#666486}
-
wutao authored
There are two sizes for embedded Assistant Ui: smaller size for empty Ui opened by hotkey and hotword etc, larger size for Ui with response or opened from search result page. Bug: 957643 Test: manual Change-Id: I275378f1b68f1977fa28de607fa3a7215b778378 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1629435 Commit-Queue: Tao Wu <wutao@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#666485}
-
Istiaque Ahmed authored
Add a flag to RunExtensionTest so that an existing event page based test extension can be loaded as SW based extension. This can be used to run extension tests with Service Worker based extensions. This CL also adds a simple event page based tabs API test at tabs/lazy_background_on_created/ and demonstrates that the extension can be loaded as SW extension and the corresponding test ExtensionApiTabTest.LazyBackgroundTabsOnCreated passes with the flag in ServiceWorkerBasedBackgroundTest.TabsOnCreated. Doc: https://docs.google.com/document/d/1PvXZ7VGRGdmd1s99SFByn9NCGj0GAiBhhPmfjj35ZeI/edit# Bug: 967899 Change-Id: Ie979986e4c9f5bcac644152d8527aeb1c8c365c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1636988 Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#666484}
-
Michael Giuffrida authored
The Kiosk Next Home app should only be loaded in Kiosk Next Home sessions (using the pref as a proxy for this). Also removes unused local_state references. Bug: 937549 Test: kiosk_next_home now only appears in user's Preferences file if they have enabled Kiosk Next Change-Id: I62b8e05fb30579c4d21cdbb54ed0788c783fe69f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1639479 Commit-Queue: Michael Giuffrida <michaelpg@chromium.org> Auto-Submit: Michael Giuffrida <michaelpg@chromium.org> Reviewed-by:
Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#666483}
-
Joshua Bell authored
Reduce visual noise in code by removing std::make_pair and push_back() calls when brace initializers for pairs/vectors can be used instead. No functional changes. Change-Id: I3553eccb670093ef05044ae19ba6a198cd7e1c94 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1641516 Commit-Queue: Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Cr-Commit-Position: refs/heads/master@{#666482}
-
Khushal authored
This reverts commit 959fa419. Reason for revert: Speculative revert for failure: https://ci.chromium.org/p/chromium/builders/ci/Android%20Release%20%28Nexus%205X%29/20525 Original change's description: > Remove the system URLRequestContext in src/chrome as it's not used anymore. > > The only usage remaining was in safe browsing unit_tests, so I've switched those specific tests to enable the network service feature flag (as it's disabled by default still for unit_tests). > > Bug: 934009 > Change-Id: I8eb23503bbb3f3610b958dce1d62f0b7428d2289 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1643082 > Commit-Queue: John Abd-El-Malek <jam@chromium.org> > Reviewed-by: Ken Rockot <rockot@google.com> > Cr-Commit-Position: refs/heads/master@{#666407} TBR=jam@chromium.org,rockot@google.com Change-Id: I27d1c49de54bdac15d9527bc364b57c0a2ebefff No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 934009 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646648Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Khushal <khushalsagar@chromium.org> Cr-Commit-Position: refs/heads/master@{#666481}
-
Chris Harrelson authored
This is because the overlap-testing rects depend on scroll offset. Bug: 950154 Change-Id: Iacb3d373fe1b9be315701a55d604e44edb88f040 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1638620 Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#666480}
-
Chandani Shrestha authored
Network Conditions drawer had following issues: - Form controls like Network Throttling, User Agent combox box and input box was missing labels for screen reader users. This change addresses that issue by adding aria-label to them. Additionally, axe test is added for Network Conditions. Tested keyboard navigation - navigate up/down drop-down items using arrow keys - Checkbox is checked/unchecked using space bar - Tab into/out (no tab traps) Tested screen reader - reads checkbox label - reads items selected from the drop down list Bug: 963183 Change-Id: I7dec31f1c476f7c63ce4fff9a3b73c1b5377bfc4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1614138Reviewed-by:
Joel Einbinder <einbinder@chromium.org> Commit-Queue: Chandani Shrestha <chshrest@microsoft.com> Cr-Commit-Position: refs/heads/master@{#666479}
-
Aaron Leventhal authored
Menu events should fire when a menu is made visible or hidden, whether or not the menu or something within it gets focused, because some authors simply show the menu at first, and only move focus if the user arrows into it. If the menupopupstart doesn't fire when the menu first appears, then the screen reader does not know to change to focus mode (virtual cursor off), and the web page will not receive down arrow keys. This also matches the behavior of Firefox for menupopupstart. See the two bugs listed. One contains a general simplified test case and the other describes how to reproduce the issue with Gmail. Bug: 98981, 964546 Change-Id: I975513ba73f3b23e884a7f1b38e6b5b161028d17 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1637057Reviewed-by:
Ian Prest <iapres@microsoft.com> Commit-Queue: Aaron Leventhal <aleventhal@chromium.org> Cr-Commit-Position: refs/heads/master@{#666478}
-
Etienne Bergeron authored
This CL is removing the function GetPlatformFontHeight since it's no longer needed. It was used to be more complicated to get height of fonts on windows. Since recent migration, the code is straigh forwards and font heights are constants. Address comments left on previous commit https://chromium-review.googlesource.com/c/chromium/src/+/1570653 R=tapted@chromium.org CC=benck@chromium.org Bug: 701241 Change-Id: I2db7c53914aa9105d5c9390dd98d5266998676c7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642735Reviewed-by:
Trent Apted <tapted@chromium.org> Commit-Queue: Etienne Bergeron <etienneb@chromium.org> Cr-Commit-Position: refs/heads/master@{#666477}
-
Ken Rockot authored
Fixes some leaks in a few bindings tests and re-enables them. Bug: 969789 Change-Id: Iee46b8de5559f2f655b70ced6d0ef9f7a831a379 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644320Reviewed-by:
Reilly Grant <reillyg@chromium.org> Commit-Queue: Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#666476}
-
Devon O'Brien authored
Add newly Qualified CT Logs and remove 2018 CT Log shards. Bug: 962537, 859093, 888130, 796333, 801624 Change-Id: I4b0e5b772c1726af2cb7c633de038b0827453b2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644898 Commit-Queue: Devon O'Brien <asymmetric@chromium.org> Commit-Queue: Ryan Sleevi <rsleevi@chromium.org> Reviewed-by:
Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#666475}
-
Chris Palmer authored
jcivelli is no longer working on Chromium, and we need more OWNERS for this security-relevant service. Bug: None TBR: jcivelli, rsesek Change-Id: I77f0ee0eafb4eb3b4709d18dd4ab09d9ecb7a037 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645240 Commit-Queue: Chris Palmer <palmer@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#666474}
-
Matt Mueller authored
Bug: 749276 Change-Id: If49f4fb0d5e550da54e9aa6c66c1ba5d893a8a42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645982Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: Matt Mueller <mattm@chromium.org> Cr-Commit-Position: refs/heads/master@{#666473}
-
Kristi Park authored
The tile's restricted ID is referenced in multiple places as "rid" and "tid". Standardize naming by changing all references of "tid" to "rid". Change-Id: I285d7b21abb404adb0c97cdecad38c38ae3a39d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642235Reviewed-by:
Gayane Petrosyan <gayane@chromium.org> Commit-Queue: Kristi Park <kristipark@chromium.org> Cr-Commit-Position: refs/heads/master@{#666472}
-
Henrique Nakashima authored
Testing Offline Prefetch in reduced mode manually requires a bit more time. Bug: 933358 Change-Id: Iea9e52fda62bbf7fc2251fe3ca8eef2d110f4889 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1626495 Commit-Queue: Carlos Knippschild <carlosk@chromium.org> Auto-Submit: Henrique Nakashima <hnakashima@chromium.org> Reviewed-by:
Carlos Knippschild <carlosk@chromium.org> Cr-Commit-Position: refs/heads/master@{#666471}
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/2f4a8dfd3a59..d3692c701b12 git log 2f4a8dfd3a59..d3692c701b12 --date=short --no-merges --format='%ad %ae %s' 2019-06-05 johnkslang@users.noreply.github.com Merge pull request #1791 from dj2/unused Created with: gclient setdep -r src/third_party/glslang/src@d3692c701b12 The AutoRoll server is located here: https://autoroll.skia.org/r/glslang-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. TBR=radial-bots+chrome-roll@google.com,geofflang@google.com Change-Id: I09a91e60063dc1083e76aac7782a64a80d04ac85 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644848Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#666470}
-
Carlos Knippschild authored
Bug: 960069 Change-Id: I1870678c0fe42311a2c7ea3e30f272d6de89d372 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644301 Commit-Queue: Carlos Knippschild <carlosk@chromium.org> Reviewed-by:
Dan H <harringtond@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Cr-Commit-Position: refs/heads/master@{#666469}
-
Daniel Murphy authored
This change removes the kWipeCorruptV2IDBDatabases feature to now mark databases that have missing blobs (so permantently corrupt databases) as corrupt, so they are cleared appropriately. Since Docs has error handling here, and we are in active communications with them, disable this corruption handling only for Docs. The reporting will stay in place to monitor how small of a problem this is. The numbers on this are pretty small, there are less than 0.005% of users that have these non-docs corrupt databases. R=pwnall@chromium.org Bug: 857159 Change-Id: Icf8feac9ad6b626108e00213447a706bbe264e5e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1633334Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#666468}
-
Robert Ma authored
To include a fix that would suppress some noisy logging from pywebsocket: https://github.com/web-platform-tests/wpt/pull/17124 Fixups made in this roll: * chromium.patch is updated so that it can apply cleanly. There is no substantial change. * A new file is included: tools/lint/rules.py * The fake test file in PRESUBMIT_test.py is changed to accommodate a stricter regex (https://github.com/web-platform-tests/wpt/pull/16605). * wpt_manifest.py no longer uses the removed `--work` flag. Bug: 968904 Change-Id: Id49c50ba4d5da6dda906a21e06a5c24f24dd5a1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1642112 Commit-Queue: Philip Jägenstedt <foolip@chromium.org> Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Auto-Submit: Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#666467}
-
Cammie Smith Barnes authored
We add a method to extract the resource URL from the cache key, so that HttpCacheDataRemover can perform its URL matching to determine which cache entries to remove. This is in accordance with the spec at https://w3c.github.io/webappsec-clear-site-data/ In future, it may be desirable to change the spec so that when there is a clear-site-data request for a given site, all resources keyed with that site as the resource URL, the domain, or the initiator, inclusively, would be removed from the cache. Test 1: net_unittests --gtest_filter=*HttpSplitCacheKeyTest* Test 2: services_unittests --gtest_filter=*HttpCacheDataRemoverSplitCacheTest* Bug: 951812 Change-Id: Id1afd9fff5f766e46d596ce8be340d09e6296bd4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1590406Reviewed-by:
Maks Orlovich <morlovich@chromium.org> Reviewed-by:
Shivani Sharma <shivanisha@chromium.org> Commit-Queue: Cammie Smith Barnes <cammie@chromium.org> Cr-Commit-Position: refs/heads/master@{#666466}
-
John Abd-El-Malek authored
The tests that used to be on this bot are now on the CQ as network service is on by default. Bug: 598073 Change-Id: If1a941917910a8543aab4c2b2a0d893d516aecc0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645993 Commit-Queue: John Abd-El-Malek <jam@chromium.org> Auto-Submit: John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Cr-Commit-Position: refs/heads/master@{#666465}
-
Xiyuan Xia authored
- Replace mojo interface MediaController/Client in cpp; - Current MediaController/Client -> Media{Controller/Client}Impl; Bug: 958207 Change-Id: Ia3b9253a45d431e9d0532adc434e38fb0a97adf9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646532 Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Becca Hughes <beccahughes@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Auto-Submit: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#666464}
-
Jacob DeWitt authored
- Remove TODOs that reference closed crbugs. - Create crbugs and update comments for TODOs that didn't already link to a crbug. Change-Id: Id25cca7ca4ffa6ee8f9a5ac7f7af4ee817e17ca4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1639204Reviewed-by:
Alexander Cooper <alcooper@chromium.org> Reviewed-by:
David Dorwin <ddorwin@chromium.org> Commit-Queue: Jacob DeWitt <jacde@chromium.org> Cr-Commit-Position: refs/heads/master@{#666463}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/928b2997b301..e0d90c84e42f git log 928b2997b301..e0d90c84e42f --date=short --no-merges --format='%ad %ae %s' 2019-06-05 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 2019-06-05 csmartdalton@google.com ccpr: Preserve fill rules for cached paths Created with: gclient setdep -r src/third_party/skia@e0d90c84e42f The AutoRoll server is located here: https://autoroll.skia.org/r/skia-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;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=jcgregorio@google.com Change-Id: Ia5abd88548b6d0b3d8af4f521e622148af60820c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644926Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#666462}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/e77dbf600358..01927c1c5000 git log e77dbf600358..01927c1c5000 --date=short --no-merges --format='%ad %ae %s' 2019-06-05 manojgupta@google.com cros_sdk: Allow using a Frozen SDK for bootstrapping. 2019-06-05 manojgupta@google.com cros_sdk: Format with yapf. 2019-06-05 saklein@chromium.org Build API: Recursive Path Handling 2019-06-05 dburger@chromium.org Fix config updater. Created with: gclient setdep -r src/third_party/chromite@01927c1c5000 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Change-Id: Idb0a76aea82296816ae761361861da50a612f8a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645986Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#666461}
-
Dana Fried authored
Button listener was not scoped and is not owned by the button in question, so was never deleted. Since button listeners are optional and this one was never used, the choice was made to remove it entirely. Bug: 961075 Change-Id: If9600ba17cb2e30a4a10c9bdf4d0449f056b583f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646753 Commit-Queue: Dana Fried <dfried@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#666460}
-
Charlie Reis authored
This also fixes some owner lines with typos which caused issues in processing the file. Bug: None Change-Id: I17a6ec571846da6a7018bd2cfef5d40e61a4c530 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645981Reviewed-by:
Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Commit-Queue: Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#666459}
-
Mei Liang authored
Change-Id: If3d333382233c155e4882e04fb3e9acbc14ea109 Bug: 970982 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644525 Commit-Queue: Mei Liang <meiliang@chromium.org> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#666458}
-
Krishna Govind authored
TBR=meacer@chromium.org Change-Id: If4403c877bc53c97b35a0b36f7cb0de142cdfb03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646373 Commit-Queue: Krishna Govind <govind@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#666457}
-
Yuchen Liu authored
Temporary grants permisson of PROTECTED_MEDIA_IDENTIFIER, to unblock EME dev. Bug: 922833 Bug: 966191 Test: Build and pages with EME Change-Id: I853719d74ee04a1bbc0e7d4768d17aa208147ab7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1646632 Commit-Queue: Yuchen Liu <yucliu@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#666456}
-
Nick Burris authored
Per discussion, it makes sense for TextFragmentAnchor matching to be case sensitive to reduce ambiguity. Case insensitivity is likely not useful since the targetText would be generated by a copy of the text (either by a machine or a human copy-paste). Change-Id: I7b52d2adc4f9474fbf77bbfb45814f5ab91036c4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1644835Reviewed-by:
David Bokan <bokan@chromium.org> Commit-Queue: Nick Burris <nburris@chromium.org> Cr-Commit-Position: refs/heads/master@{#666455}
-
Harry Cutts authored
See go/cros-gesture-properties-dbus-design (Google internal only, sorry) for more details about the feature. The flag should never expire as these are debugging and development tools, so we'd never want to enable them for everybody, but nor would we want to disable them. Bug: 952054 Test: build, then check chrome://flags/#gesture-properties-dbus-service. Change-Id: Ic876d3f9f2fffff084b7c280b4561e013cd31a0d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1589446 Commit-Queue: Harry Cutts <hcutts@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Dan Erat <derat@chromium.org> Cr-Commit-Position: refs/heads/master@{#666454}
-
Joe Mason authored
Make sure the navigation observer is watching the WebContents returned by AddSelectedTabWithURL. GetActiveWebContents can return null when there is no page open. Change-Id: I9de58135e3559df0bfedc673241e44f345cf3cd4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1636598Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Commit-Queue: Joe Mason <joenotcharles@google.com> Cr-Commit-Position: refs/heads/master@{#666453}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/b632bc58ed37..9cd21f1bf945 git log b632bc58ed37..9cd21f1bf945 --date=short --no-merges --format='%ad %ae %s' 2019-06-05 enga@chromium.org Implement CreateBufferMapped for non-mappable buffers 2019-06-05 enga@chromium.org Add Create/ReleaseStagingBuffer to DynamicUploader Created with: gclient setdep -r src/third_party/dawn@9cd21f1bf945 The AutoRoll server is located here: https://autoroll.skia.org/r/dawn-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel TBR=cwallez@google.com Change-Id: I8078513f9b6e70dd17d21bd64c44766169f097c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1645242Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#666452}
-