- 11 Jul, 2018 40 commits
-
-
Mohamed Amir Yosef authored
This CL addresses the remaining work to fix the attached bug which includes: 1. Detecting a cache_guid mismatches in ClientTagBasedModelTypeProcessor::ConnectIfReady(). 2. Ignore sync metadata in that case, and tell the bridge to delete it (treat as DisableSync/EnableSync). Change-Id: Ibf2f4c0b89872884f1c09a3a2e8fec1136014b93 Bug: 820049 Change-Id: Ibf2f4c0b89872884f1c09a3a2e8fec1136014b93 Reviewed-on: https://chromium-review.googlesource.com/1130527 Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#574107}
-
afdo-chromium-autoroll authored
This CL may cause a small binary size increase, roughly proportional to how long it's been since our last AFDO profile roll. For larger increases (around or exceeding 100KB), please file a bug against gbiv@chromium.org. Additional context: https://crbug.com/805539 The AutoRoll server is located here: https://afdo-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=gbiv@chromium.org Change-Id: I37f962245e818a16b31ed9666b5c918233d355e9 Reviewed-on: https://chromium-review.googlesource.com/1132846Reviewed-by:
afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#574106}
-
Kinuko Yasuda authored
We are seeing a crash that happens while the frame's subresource_ factories are null and it's last commited URL is for about: schemes (e.g. about:srcdoc or about:blank). This situation doesn't really make sense (e.g. prefetches shouldn't come to the browser process for about:blank), so we suppose there's probably a race somewhere. This change does two things: - send URLLoaderFactory for prefetch on NavigationCommit (so no race) - remove PrefetchURLLoaderService related code Bug: 849929 Change-Id: If350afa26daaeb0c679091b74b2013ab7f121531 Reviewed-on: https://chromium-review.googlesource.com/1116411 Commit-Queue: Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#574105}
-
Rakina Zata Amni authored
Currently in some cases, ShadowRoot.elementsFromPoint may not return the document element even when Document.elementsFromPoint returns the document element such as when the document element height is shorter than the hit-tested element's position. This CL fixes it by removing a check in TreeScope::ElementsFromHitTestResult. See example live case detailed in this comment: https://bugs.chromium.org/p/chromium/issues/detail?id=843215#c16 Bug: 843215 Change-Id: I5b774d1a091f3c3f72345c55bca188fad309ba58 Reviewed-on: https://chromium-review.googlesource.com/1131023 Commit-Queue: Rakina Zata Amni <rakina@chromium.org> Reviewed-by:
Hayato Ito <hayato@chromium.org> Cr-Commit-Position: refs/heads/master@{#574104}
-
Daniel Bratell authored
Both syslog and the base namespace have LOG_INFO symbols and in certain cases (which include, but is not limited to, jumbo builds) those will clash. This patch undefs the syslog LOG_INFO macro as is already done with LOG_WARNING previously. Change-Id: Iff59a2ef079d3bb38378af76932b21a128507353 Reviewed-on: https://chromium-review.googlesource.com/1027611 Commit-Queue: Daniel Bratell <bratell@opera.com> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#574103}
-
Aleks Totic authored
Static position needs to be adjusted based upon block container padding box, not upon inline container padding box. Bug: 740993 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: Ib6a5fbb27a56f7a724a52366988cfbd5063d2e36 Reviewed-on: https://chromium-review.googlesource.com/1132532Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Aleks Totic <atotic@chromium.org> Cr-Commit-Position: refs/heads/master@{#574102}
-
Kunihiko Sakamoto authored
This completes the htxg -> sxg renaming. Bug: 856530 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Idf5243b0ff08acc761313299415bcb7fa32b333d Reviewed-on: https://chromium-review.googlesource.com/1132835Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#574101}
-
Sam McNally authored
Bug: 854153 Change-Id: I3a7bb8ff09a5fc366ef918dea8e821ed6606fe0e Reviewed-on: https://chromium-review.googlesource.com/1132261 Commit-Queue: Sam McNally <sammc@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#574100}
-
Yutaka Hirano authored
Some entries are missing, some are fixed, some are not specific to virtual/outofblink-cors. This CL removes fixed/missing ones and reorders entries which are not specific to virtual/outofblink-cors. Bug: 736308 Change-Id: Ic39ce52e5ccd730ccf476bc9be7804d04bcc3f19 Reviewed-on: https://chromium-review.googlesource.com/1130558 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
Adam Rice <ricea@chromium.org> Cr-Commit-Position: refs/heads/master@{#574099}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 7a0a490d. With Chromium commits locally applied on WPT: a9045946 "Add a fail-fast for longtask layouttests" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/21163 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: alexander.shalamov@intel.com, kenneth.r.christiansen@intel.com, rijubrata.bhaumik@intel.com: external/wpt/web-nfc TBR=kereliuk No-Export: true Change-Id: Ic94e54953c4c6609ba8a3b5d26878adb23893748 Reviewed-on: https://chromium-review.googlesource.com/1132683 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@{#574098}
-
Tatsuhisa Yamaguchi authored
Test: manually verified Bug: 858957 Change-Id: I31c2b69ca92680ea38ab38ba65f13169e0e11b45 Reviewed-on: https://chromium-review.googlesource.com/1128822Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Reviewed-by:
Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org> Cr-Commit-Position: refs/heads/master@{#574097}
-
Lily Houghton authored
Bug: 821027 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I597a7ee5d5cd423474232dc3d996055b0962dc60 Reviewed-on: https://chromium-review.googlesource.com/1104800 Commit-Queue: Lily Houghton <lilyhoughton@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Reviewed-by:
Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#574096}
-
Tomasz Moniuszko authored
Change-Id: I5558e93ca31e5ab546457b0e07a73e5f1a60c614 Reviewed-on: https://chromium-review.googlesource.com/1125074Reviewed-by:
Kim Paulhamus <kpaulhamus@chromium.org> Commit-Queue: Tomasz Moniuszko <tmoniuszko@opera.com> Cr-Commit-Position: refs/heads/master@{#574095}
-
tanvir.rizvi authored
This patch changes 15381.html to utilize |selection_test()|. Follow up patch will rename it to |justify-center-do-not-modify-rootEditableElement.html| for better readability and information. Bug: 679977 Change-Id: I41a2816aa3dfb19d7068983d39a7bd726e4272eb Reviewed-on: https://chromium-review.googlesource.com/1128899Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Tanvir Rizvi <tanvir.rizvi@samsung.com> Cr-Commit-Position: refs/heads/master@{#574094}
-
Tarun Bansal authored
Bug: 852484 Change-Id: I4b62bc03266d9e754e01cb7983d841635d9a347b Reviewed-on: https://chromium-review.googlesource.com/1132689Reviewed-by:
Tarun Bansal <tbansal@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#574093}
-
Christopher Cameron authored
Avoid checking the full input |text| string, rather, only examine the part that sits inside |range| and its adjacent context. Note that we still need the full string (or at least we need to know the offset into it that we are using), because that will affect the output. Pre-compute the input hash, to avoid re-computing a string's hash at every comparison. Add comments regarding key hash computation and thread use, and mark members as const (requested in previous patch's review). Bug: 860612 Change-Id: I9dd73e0a2d0247a7139131edfb33b3d99138b103 Reviewed-on: https://chromium-review.googlesource.com/1130159 Commit-Queue: ccameron <ccameron@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Cr-Commit-Position: refs/heads/master@{#574092}
-
Tarun Bansal authored
blink-dev approval thread: https://groups.google.com/a/chromium.org/d/msg/blink-dev/G_rkd0yo2AM/WaVhRALcAQAJ Bug: 826950 Change-Id: If511883b7428f30136642302862b51f14ffe4894 Reviewed-on: https://chromium-review.googlesource.com/1132612Reviewed-by:
Yoav Weiss <yoav@yoav.ws> Commit-Queue: Tarun Bansal <tbansal@chromium.org> Cr-Commit-Position: refs/heads/master@{#574091}
-
rbpotter authored
This is currently broken in Polymer2. Bug: 738611 Change-Id: I813b1b8aa6b4b4cfa31d1735f3b3b35f318e4c3a Reviewed-on: https://chromium-review.googlesource.com/1125501 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#574090}
-
Wenzhao Zang authored
Addressed several corner cases including: 1) Handles overflow of long wallpaper descriptions. 2) Add spinner in the center of the current wallpaper thumbnail. 3) Adjust the color of the error message. Bug: 861915 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: Ibd8b4090dec51a1caad76c1872929490e8f042ae Reviewed-on: https://chromium-review.googlesource.com/1125538Reviewed-by:
Alexander Alekseev <alemate@chromium.org> Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org> Cr-Commit-Position: refs/heads/master@{#574089}
-
Emil A Eklund authored
Modify ShapingLineBreaker performance test to include the paragraph text shaping operation in the timing rather than just the time for each line. Change-Id: I3f23333e17251a9640c5337171c8883c964bb9c2 Reviewed-on: https://chromium-review.googlesource.com/1132822 Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
-
Hajime Hoshi authored
When PRINT_UNSYMBOLIZED_STACK_TRACES is on, |object_fd| is created in |SymbolizeAndDemangle| but not used. The handle is not closed when returning the function. This CL fixes this file descriptor leak by using FileDescriptor. Change-Id: I216c00690946671edd802f91fd694573d5783ba0 Reviewed-on: https://chromium-review.googlesource.com/1120087Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#574087}
-
Hiroki Nakagawa authored
FetchContext::CheckResponseNosniff() is called only from ResourceLoader. That should be placed in ResourceLoader for code health. Bug: 845285 Change-Id: I3174ec08c046d3f95eaf304d23bcb29105c0c67f Reviewed-on: https://chromium-review.googlesource.com/1131044Reviewed-by:
Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
-
Yutaka Hirano authored
Currently PreflightLoader doesn't do anything when the preflight request fails by network errors. This CL fixes that. Bug: 736308, 803766 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: Ica8f911e060b185980a653d7ddecb08f5c86c8ac Reviewed-on: https://chromium-review.googlesource.com/1126894 Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Reviewed-by:
-
Hayato Ito authored
A part of http://crbug.com/738794. TBR=tkent Bug: 738794 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I3710ab07b66afc4766cdbe6a64974c1eb676e1a4 Reviewed-on: https://chromium-review.googlesource.com/1126691Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Commit-Queue: Hayato Ito <hayato@chromium.org> Cr-Commit-Position: refs/heads/master@{#574084}
-
Hitoshi Yoshida authored
Before this CL, setting |null| as a member of dictionary had the same meaning with unsetting the member. It is consistent with the intention of "present" state, and we don't require a way to unset members. (We can create a dictionary from scratch and set necessary members, to achieve such a state.) After this CL, setting |null| makes the member present and be |null|. Bug: 855968 Change-Id: Ia1cbb5cede890cc00834205ca618543c043b9fbb Reviewed-on: https://chromium-review.googlesource.com/1128696Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#574083}
-
Alice Boxhall authored
This reverts commit 1afbb0e4. Reason for revert: Causing test failures on Android CFI: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Android%20CFI/2026 https://logs.chromium.org/v/?s=chromium%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8941340307860291344%2F%2B%2Fsteps%2Fcontent_unittests_on_Android_device_Nexus_5X%2F0%2Flogs%2FDevToolsManagerTest.NoUnresponsiveDialogInInspectedContents%2F0 Original change's description: > Migrate //content unit tests off of QuitCurrent*Deprecated(). > > Cq-Include-Trybots: luci.chromium.try:linux_mojo > Change-Id: If5cf7d3dcda8dc411c6c90de31885dca683354d8 > Reviewed-on: https://chromium-review.googlesource.com/1130656 > Reviewed-by: Bo <boliu@chromium.org> > Commit-Queue: Wez <wez@chromium.org> > Cr-Commit-Position: refs/heads/master@{#574006} TBR=wez@chromium.org,boliu@chromium.org Change-Id: Icb42050ab3fcbf283098bb36943a8c9dfbefccf8 No-Presubmit: true No-Tree-Checks: true No-Try: true Cq-Include-Trybots: luci.chromium.try:linux_mojo Reviewed-on: https://chromium-review.googlesource.com/1132636Reviewed-by:
-
Stefan Zager authored
The immediate motivation for this patch is to pave the way to move stuff out of platform/scroll and into core/. That aside, it probably makes more sense to put this kind of code in core/ rather than platform/. BUG=823365 R=skobes@chromium.org,pdr@chromium.org Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I0859fbfa6ddf82c607a36cbbd0bee77e49a3c0b0 Reviewed-on: https://chromium-review.googlesource.com/1131000Reviewed-by:
Steve Kobes <skobes@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Stefan Zager <szager@chromium.org> Cr-Commit-Position: refs/heads/master@{#574081}
-
Ryan Sleevi authored
During shutdown, the P2PSocketDispatcherHost may still be holding on to resources referring back to the URLRequestContext used to create it, but after that URLRequestContext has already shut down. Gracefully clean up these resources when the MessageFilter signals that the channel is closing, so that the correct shutdown sequence order is observed. BUG=860837 Change-Id: Ie4c2ed24cebc619c40468ca9720d521dfcf1962e Reviewed-on: https://chromium-review.googlesource.com/1128404 Commit-Queue: Ryan Sleevi <rsleevi@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Cr-Commit-Position: refs/heads/master@{#574080}
-
rbpotter authored
Focus the search box when the dialog is shown. Also add an interactive UI test to verify the behavior and fix the same issue in the advanced settings dialog. Bug: 860139 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: I925ba10d10a477dfc526d860d5db108534f4254f Reviewed-on: https://chromium-review.googlesource.com/1130659Reviewed-by:
-
Min Qin authored
RenderFrameImpl::DownloadUrl() can be abused to trigger a large number of download to overwhelm the browser process. This CL rate limits the number of download to a maximum of 10 per second. Bug: 860045 Change-Id: I490ae3a99eb59aef709b9f1e6be5eb0f787354f4 Reviewed-on: https://chromium-review.googlesource.com/1132485 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Cr-Commit-Position: refs/heads/master@{#574078}
-
Tatsuhisa Yamaguchi authored
Close the system tray bubble when these actions are taken. - Switch to another user - Sign in another user - Lock screen Bug: 859041 Test: verified manually Change-Id: Id70d71e73a5c261212bbbb13d52f376275da8002 Reviewed-on: https://chromium-review.googlesource.com/1127914Reviewed-by:
-
Victor Costan authored
LifecycleNotifier currently exposes its internal state (observers_, iteration_state_) to subclasses, and relies on the subclasses to correctly update iteration_state_ while iterating over observers_, like PageVisibilityNotifier::NotifyPageVisibilityChanged() does. Sadly, it's easy to get this wrong, as shown by SynchronousMutationNotifier (does not update iteration_state_ at all) and ContextLifecycleNotifier (only updates iteration_state_ in some cases). This CL makes the LifecycleNotifier state private, and adds a ForEachObserver() method that safely iterates over observers_, encapsulating the needed changes to iteration_state_ and a cast from LifecycleObserverBase* to the desired observer type. Change-Id: Ibb23f2c7e2bba9bdf08d49e8569a354934ed4e11 Reviewed-on: https://chromium-review.googlesource.com/1132674Reviewed-by:
-
Zhongyi Shi authored
Close non-migratable streams but migrate the rest of the steams when QUIC connection migration migrates to an alternate network. Bug: 791886 Change-Id: Idf803b0e7453ffc1ef44de3209e7cda74e58f64d Reviewed-on: https://chromium-review.googlesource.com/1125525 Commit-Queue: Zhongyi Shi <zhongyi@chromium.org> Reviewed-by:
Ryan Hamilton <rch@chromium.org> Cr-Commit-Position: refs/heads/master@{#574075}
-
catapult-chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/1230eb961dc4..10041796ae14 git log 1230eb961dc4..10041796ae14 --date=short --no-merges --format='%ad %ae %s' 2018-07-11 cphlipot0@gmail.com [ftrace importer] Use tid instead of pid for kernel threads 2018-07-11 cphlipot0@gmail.com [ftrace importer] Fix recursive binder reply arguments 2018-07-11 eakuefner@chromium.org [TBMv2] Implement deduplication fast path for GenericSet 2018-07-10 dtu@chromium.org [pinpoint] Add stubs for Commit and Change. Created with: gclient setdep -r src/third_party/catapult@10041796ae14 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:857283 TBR=sullivan@chromium.org Change-Id: Ieb82b740e4d4b458f5bc5a80efb511d22ff6d36f Reviewed-on: https://chromium-review.googlesource.com/1132444Reviewed-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@{#574074}
-
Patti authored
An extremely large number of sites may appear in All Sites, so make sure the client doesn't need to render any more site-entries than required at any one time by converting it to use an iron-list. Also introduce a couple other polish fixes, such as scrolling to a site-entry when it's expanded, or scrolling to the same site-entry previously selected (if any) when navigating back to All Sites from a different route. Bug: 835712 Cq-Include-Trybots: luci.chromium.try:closure_compilation Change-Id: Ifca117ac6739aed927ca61efeaadcb9057487f5d Reviewed-on: https://chromium-review.googlesource.com/1116204Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Patti <patricialor@chromium.org> Cr-Commit-Position: refs/heads/master@{#574073}
-
David Dorwin authored
Bug: 860549 Change-Id: I803fa84db79846b52daa6430b5864eccac0c57a2 Reviewed-on: https://chromium-review.googlesource.com/1132539Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Commit-Queue: David Dorwin <ddorwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#574072}
-
Karan Bhatia authored
This CL ensures that the reserved _metadata folder is not included in the packed extension package. This should be safe as: - Webstore already disallows packages with a _metadata folder. - We already disallow loading unpacked extensions with non-reserved files in the _metadata folder (r502478). This should help ensure that unpacked extensions using the Declarative Net Request API don't have the indexed ruleset in the crx package. BUG=858848 Change-Id: I6d175eae16457531a54818c2d801852dc41df8bc Reviewed-on: https://chromium-review.googlesource.com/1132544 Commit-Queue: Karan Bhatia <karandeepb@chromium.org> Reviewed-by:Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#574071}
-
Wez authored
Replace call to ZX_DLOG_IF(), which won't reference |result| in builds in which DCHECKs are disabled. Bug: 852541 TBR: rockot Change-Id: I0b8ff421a690691277a4f73730f4c756faf5a17e Reviewed-on: https://chromium-review.googlesource.com/1132488Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#574070}
-
Ryan Sleevi authored
Now that CRLSets no longer use a bespoke delta update mechanism, reduce the amount of internal state that needs to be tracked by the CRLSet. This avoids a duplicating the issuer key hashes. Bug: None Change-Id: Id6d1ba752972b63ac4109a91d2a9e17794cf3b1e Reviewed-on: https://chromium-review.googlesource.com/1132105 Commit-Queue: Ryan Sleevi <rsleevi@chromium.org> Reviewed-by:
David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#574069}
-
Alice Boxhall authored
This reverts commit d2099edb. Reason for revert: Seems to be causing test failures, e.g. https://logs.chromium.org/v/?s=chromium%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8941335040754288672%2F%2B%2Fsteps%2Fbrowser_tests%2F0%2Flogs%2FCrExtensionsItemsTest.DeveloperState%2F0 Original change's description: > Extensions: remove dependency on iron-icons > > By not including the entire svg sets from iron-icons/ the resulting > md_extensions/vulcanized.html file size reduced from 232.8 kB to > 144.4 kB. > > As a side clean-up thing, the "store" icon is removed from the icons.html > since it's no longer used. > > Bug: 855205 > Cq-Include-Trybots: luci.chromium.try:closure_compilation > Change-Id: Ia19bd8b61425cbff0e3d70cc08a1201856fe0276 > Reviewed-on: https://chromium-review.googlesource.com/1123164 > Commit-Queue: Scott Chen <scottchen@chromium.org> > Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org> > Cr-Commit-Position: refs/heads/master@{#574005} TBR=dpapad@chromium.org,scottchen@chromium.org Change-Id: I8182c9d62e79688c8396852440e6367052fd08e8 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 855205 Cq-Include-Trybots: luci.chromium.try:closure_compilation Reviewed-on: https://chromium-review.googlesource.com/1132734Reviewed-by:
-
