- 21 Oct, 2020 40 commits
-
-
Mihai Sardarescu authored
Use std::make_unique to create unique pointers for BookmarkBubbleSignInDelegate. Bug: None Change-Id: Iabffb64eed63a554f4def7a0f2c1f68de38ec765 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490043 Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#819422}
-
Peng Huang authored
SkiaOutputSurfaceImpl::SubmitPaint() not always submits painting tasks to GPU. So sync token returned by it is not always valid. Fix the problem by renaming it to EndPaint() which will not submit tasks to GPU anymore and not return sync token. It also adds a new method Flush() which will submit pending tasks to GPU thread and provide a sync token. Bug: 1139697 Change-Id: I34fdd5e325ac2425399a9fc98ffcaa6052cddc3a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488302 Commit-Queue: Peng Huang <penghuang@chromium.org> Reviewed-by:
Jonathan Backer <backer@chromium.org> Cr-Commit-Position: refs/heads/master@{#819421}
-
chromium-internal-autoroll authored
Release_Notes: http://go/help_app-x20/relnotes/Main/help_app_nightly_202010210700_RC00.html https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/help_app/app/+/FPS7GXWrnoUpSm7SUEXrX5w9y54l81RKV4hoZXUW3xEC If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/help-app-chromium-autoroll Please CC help-app@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: b/170910313,b/170910490 Tbr: help-app@grotations.appspotmail.com Change-Id: I5dcc1e8d8acd7ca42c54ce19d330a68dd2c01910 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489751Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#819420}
-
Alex Gough authored
Input zip files do not need to be written, and are opened read only, e.g. https://source.chromium.org/chromium/chromium/src/+/master:components/services/unzip/public/cpp/unzip.cc;l=109 Bug: 1130762 Change-Id: I161ea1debe3b6ba7bf3a9f002bdb135be739eb34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488399Reviewed-by:
Joshua Pawlicki <waffles@chromium.org> Commit-Queue: Alex Gough <ajgo@chromium.org> Cr-Commit-Position: refs/heads/master@{#819419}
-
Alex Gough authored
Input files do not need to be written and are opened readonly. Bug: 1130762 Change-Id: I23a6a610ae34c6eb0aa76f54f727458c55fd5e07 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488667Reviewed-by:
-
Sylvain Defresne authored
In preparation of supporting arm64 simulator build (for Apple Silicon macs), we need to decouple the fact that a build is a simulator build from the `target_cpu`. This was done by introducing a `target_environment` variable that can be set by the user (the default value is set using the same heuristic based on value of `target_cpu` if unset). Change the ios/web_view build script to explicitly set the variable. Also change gn_args from a string to a list of string and join them when creating the command-line to make the code easier to read. Bug: 1138425 Change-Id: I540ad628921dba9cd37a7599b68fd1d27bf59cda Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489866 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#819417}
-
Austin Sullivan authored
Modifies CreateIncognitoFileSystemContextForTesting method Bug: N/A Change-Id: I28a86cb4edb35d312899ac438e283bd502fd1722 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2487723Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Commit-Queue: Austin Sullivan <asully@chromium.org> Cr-Commit-Position: refs/heads/master@{#819416}
-
Yusuf Sengul authored
Bug: 1139587 Change-Id: I5e0166850dbccd5cc936d2e59417da085fa62632 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2482545 Commit-Queue: Yusuf Sengul <yusufsn@google.com> Reviewed-by:
Rakesh Soma <rakeshsoma@google.com> Cr-Commit-Position: refs/heads/master@{#819415}
-
Jan Wilken Dörrie authored
This change replaces existing usages of autofill::PasswordForm with password_manager::PasswordForm in //components. TBR=dvadym,dullweber,arbesser,treib Bug: 1067347 Change-Id: Ide786bd54570dc99867dbf6e93057a95af1719f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2487324Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Vadym Doroshenko <dvadym@chromium.org> Reviewed-by:
Mohamed Amir Yosef <mamir@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#819414}
-
Chris Lu authored
InfobarContainerCoordinator is not used in the Overlay implementation. User taps to badges in the overflow menu should just insert an overlay request into the OverlayRequestQueue. Bug: 1138106 Change-Id: Id20bfec51be0894a3a430ed90069fd8f668349e3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488905 Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#819413}
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: I532524fbd4ddf1ed699abf717de06cf56a1462ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490301Reviewed-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@{#819412}
-
Roberto Moura authored
Add an opacity transformation to the tab grid's top toolbar when transitioning from thumb strip to tab grid. Bug: 1127604 Change-Id: I38c43eb556f2f02e7da2ff81a0d30dda7c60fea4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2485163 Auto-Submit: Roberto Moura <mouraroberto@google.com> Reviewed-by:
Robbie Gibson <rkgibson@google.com> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Commit-Queue: Roberto Moura <mouraroberto@google.com> Cr-Commit-Position: refs/heads/master@{#819411}
-
Mohamed Amir Yosef authored
This is preparation for adding the functionality of moving multiple passwords from settings view. Mocks are in the linked bug. Bug: 1139263 Change-Id: Ib173b0625f2ee82542164b703c97a53cee997030 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2480162Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Cr-Commit-Position: refs/heads/master@{#819410}
-
Rohit Agarwal authored
This CL refactors the test code on profile and adds Incognito CCT tests. Bug: 1060940 Change-Id: I1e340b9dc34d080c1b2a191720c37d7c6db17cda Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489903Reviewed-by:
-
Tommy Li authored
For HistoryQuickProvider, we use an in-memory URL database called URLIndexPrivateData. This in-memory database is rebuilt from History if its own cache file is missing or corrupt. Previously, Android devices were limited to 1000 entries during the rebuild step, whereas other platforms had no limit. There are some pathological cases where unlimited is bad, such as when users are stuck in a redirect loop with autogenerated URLs that spam their history. This CL adds a 20,000 entry cap for all platforms, and that should be safe, as that will affect only 0.01% of Windows users. I plan to add further per-eTLD+1 caps and per-URL length caps as well in followup CLs. NOTE: This only affects the case where the HQP cache file is missing or corrupt, and the HQP index has to be rebuilt on startup from the History DB. This is an unusual case, so further work will be needed to address the typical present cache file case. Bug: 750845, 1032679 Change-Id: Ia9e309dd43e3272823d69c707cceb8abe1a192ff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488384Reviewed-by:
Mark Pearson <mpearson@chromium.org> Commit-Queue: Tommy Li <tommycli@chromium.org> Cr-Commit-Position: refs/heads/master@{#819408}
-
Andrew Grieve authored
Abstracts the pattern of looping over weak references and removing stale ones. * Converts UnownedUserDataKey to use WeakSet since order does not matter. TBR=agrieve # Trivial refactor in AutofillManagerWrapper.java Bug: 1131047 Change-Id: Idf226faecb31de69f7b7bbae32d6c795dca3dca2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2477050 Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Cr-Commit-Position: refs/heads/master@{#819407}
-
Achuith Bhandarkar authored
BUG=chromium:1115622 TEST=chromeos-betty-arc-r-chrome Change-Id: I002720f117ca6bad652fe44d42222da4b584ac16 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488900Reviewed-by:
Ben Pastene <bpastene@chromium.org> Commit-Queue: Achuith Bhandarkar <achuith@chromium.org> Cr-Commit-Position: refs/heads/master@{#819406}
-
Hai Bi authored
The ScopedService* publishing templates infer the service name from the supplied Interface class. It is sometimes desirable to publish a service under a manually-specified name, e.g. if the service isn't actually defined as [Discoverable], or it's needed to publish it under a different name. The templates now accept an explicit |name| parameter which defaults to the name of the Interface, if available. Bug: b/171256428 Test: locally tested and pass unit tests Change-Id: If206fea8a7555baea5806b8526be2affc1e0dc74 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2486620 Commit-Queue: Hai Bi <bihai@google.com> Reviewed-by:
Kevin Marshall <kmarshall@chromium.org> Reviewed-by:
Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#819405}
-
rbpotter authored
Bug: 987318 Change-Id: I7c8943f15724e6387c5a1a9fc1666c1940422020 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2486459Reviewed-by:
John Lee <johntlee@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#819404}
-
David Grogan authored
Just a test, no behavior change. Fixed: 965672 Change-Id: Iaf1f86a92aa6c65e33736850ea2fbd5411db45d6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488984Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: David Grogan <dgrogan@chromium.org> Cr-Commit-Position: refs/heads/master@{#819403}
-
Lukasz Anforowicz authored
Similarly to r778581, after this CL, the rewriter will no longer unconditionally skip fields in the following scenarios: - Field pointing to a struct/class with deleted operator new - Fields in generated code Instead, the rewriter will emit such fields as candidates for the --field-filter-file blocklist - in the following section of the output: ==== BEGIN FIELD FILTERS ==== GeneratedStruct::ptr_field # generated-code MyStruct::ptr_to_stack # pointee-has-no-operator-new NoNewOperator::field # embedder-has-no-operator-new ==== END FIELD FILTERS ==== Skipping via filter list is desirable, because it helps with auditing and counting the filtered fields. In particular: - operator-new-based cases should be vetted to ensure the heuristic is sound and check what kind of cases are present outside of Blink - generated code should eventually be fixed (to use CheckedPtr as appropriate) after the rewrite and explicit auditing will help scope this work Note that the "embedder-has-no-operator-new" case was not present (i.e. was not unconditionally skipped) before this CL - this case is new. Bug: 1069567 Change-Id: Ic67ac7a94225d0e11e1e405e75dc41a7248ab67f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2488744Reviewed-by:Daniel Cheng <dcheng@chromium.org> Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Cr-Commit-Position: refs/heads/master@{#819402}
-
Marc Treib authored
This method was always called synchronously from ModelAssociationManager::Associate(), so no need for a delegate method - instead, the one implementation is inlined at the relevant place. Bug: 1102837 Change-Id: Ic69b0f93baa5780c76ab95f45904fd2ca9e08b98 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489642Reviewed-by:
Jan Krcal <jkrcal@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#819401}
-
Yuly Novikov authored
Prefer to configure directly when possible. TBR=bsheedy Bug: 1140626 Change-Id: I083b48a520f2dc2def0986793096b40e898b9bad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490384Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#819400}
-
Ghazale Hosseinabadi authored
This CL removes WINDOWS_GETCURRENT API test coverage from ServiceWorkerBasedBackgroundTest. The reason is 1)The test is flaky, 2)The API test coverage is already added in NonPersistentExtensionTabsTest.TabCurrentWindow. Bug: 1140254 Change-Id: I90d93a27d3c3f420b2a12f17a38503850f33bf79 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2487868Reviewed-by:
David Bertoni <dbertoni@chromium.org> Commit-Queue: Ghazale Hosseinabadi <ghazale@chromium.org> Cr-Commit-Position: refs/heads/master@{#819399}
-
Idries Hamadi authored
When visibility metrics are written to histograms only the currently tracked visibility bucket is logged. This can lead to some data loss. This patch changes the behaviour of the record implemeantation to write any outstanding data for all buckets. Also fixed a couple of spelling errors in comments. R=tobiasjs@chromium.org Bug: 1139372 Change-Id: Idad1e18c9f581dab2579bce315b7b5f0750c736e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2480565Reviewed-by:
Tobias Sargeant <tobiasjs@chromium.org> Commit-Queue: Idries Hamadi <idries@google.com> Auto-Submit: Idries Hamadi <idries@google.com> Cr-Commit-Position: refs/heads/master@{#819398}
-
Paul Moy authored
Now that the battery health and capacity routines have good default parameter values, and can look up board-specific parameters from cros_config, we no longer need to expose these parameters to the end user. This will make the routine easier to run effectively, because the user doesn't need to know what good values for the parameters are. Bug: chromium:1131609 Change-Id: Ie4a11fefbce13b4777144c06e67139b91f53f839 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2477287 Commit-Queue: Paul Moy <pmoy@chromium.org> Reviewed-by:
Oleh Lamzin <lamzin@google.com> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#819397}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/be6747613d78..d139f2661939 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I97907b4f90025157ec3dce017124022b3700f404 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490068Reviewed-by:
-
chromium-internal-autoroll authored
Release_Notes: http://go/media_app-x20/relnotes/Main/media_app_202010210700_RC00.html https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/media_app/app/+/ce3CRtV0VP2yelnNkFkhuX0bCzB4BFzKPeYxeIRHf70C If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/media-app-chromium-autoroll Please CC media-app@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: b/159867576,b/165720635,b/169021817,b/171275470 Tbr: media-app@grotations.appspotmail.com Change-Id: I7ff7d995e1b91f85571af41112f59fbfe811a898 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490300Reviewed-by:
-
Chris Lu authored
Bug: 1138603 Change-Id: I72ecd463778caa7adac8a0ef041ec01dbd0c5f80 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2486071 Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#819394}
-
cfredric authored
Change-Id: I4edb4c31afbd3d54b2bbe673778a783810312dc4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2477041 Commit-Queue: Chris Fredrickson <cfredric@google.com> Reviewed-by:
Asanka Herath <asanka@chromium.org> Cr-Commit-Position: refs/heads/master@{#819393}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/be6747613d78..d139f2661939 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-linux-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: Ib23dbfb3bac151ac4abcc2d66881dd326a482f11 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489786Reviewed-by:
-
Wez authored
This reverts three auto-roller commmits: https://chromium-review.googlesource.com/c/chromium/src/+/2488154 https://chromium-review.googlesource.com/c/chromium/src/+/2487918 https://chromium-review.googlesource.com/c/chromium/src/+/2488154 Reason for rollback: Broke the Fuchsia ARM64 build (crbug.com/1140854). TBR=chromium-autoroll@skia-public.iam.gserviceaccount.com,swiftshader-team+autoroll@google.com Change-Id: I11c4365476e0d827e76e6ca2f7d2e8f97424152c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1140854 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_chromium_msan_rel_ng;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:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490106Reviewed-by:
-
Koji Ishii authored
Legacy does not use hyphenation points that are beyond the line end, but LayoutNG does. The LayoutNG behavior is more correct. This patch updates the expected file to the LayoutNG behavior, and marks the test to fail in `disable-layout-ng`. Note, while crbug.com/870219 was fixed in LayoutNG, the test from the reporter still has a few undesired results, due to crbug.com/1022415 (sometimes it hyphenates though, which is unexpected) and crbug.com/1140738. By adding one more word and using the build the latter issue was fixed, the result looks correct. TBR=ikilpatrick@chromium.org, tkent@chromium.org, yosin@chromium.org Bug: 870219 Change-Id: I6ec6262b2a79e3a5815f4ffd4cd7a34507b2735d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489777Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#819390}
-
Aya Elgendy authored
Fix a typo in the pin dialog error message which read "1 attempts left" instead of "1 attempt left" when the user has attempted incorrect pins and there's only 1 attempt remaining. Fixed: 1060695 Change-Id: Ic9f51d03f87e8d4090cb6f93a5968e0436fdcdf8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2487106Reviewed-by:
Fabian Sommer <fabiansommer@chromium.org> Reviewed-by:
-
Fergus Dall authored
We translate this folder name when it's initially created, but if the folder already exists in the sync service we don't touch it. Since this folder name is not set by the user, we should re-translate the name whenever this might be necessary. Language changes aren't required to take effect without restarting chrome, so doing this on startup is sufficient. Bug: 1133151 Change-Id: I91f656029e71c88a40641447d1e001f1a0751ad3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489923 Auto-Submit: Fergus Dall <sidereal@google.com> Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#819388}
-
Jesse Schettler authored
Replace the temporary app icon with the finalized app icon. Bug: 1059779 Change-Id: I6105da8922cb0ccc0ac40c9fe061ebdb8ec9a44a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2486480Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#819387}
-
Elly Fong-Jones authored
It flakily hangs, which is being investigated but is not a blocker since the hang watcher isn't shipped on Mac currently. Bug: 1140207 Change-Id: I47ab3bfc8770c85b0b3de608295ade53a43e7669 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2486078Reviewed-by:
Oliver Li <olivierli@chromium.org> Reviewed-by:
-
Finnur Thorarinsson authored
It is a much simpler and easier way to load native in the PWA Install test for IPH. Bug: 1135551 Change-Id: I4d20f9ac5a4cfd1cdc00f81bae713c4b514c54d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2489902Reviewed-by:
-
Ramin Halavati authored
Unnecessary code is removed from GuestProfileWindow browser test. Bug: 1125474 Change-Id: Ica812265c273a8cfe041046a44ce65e8c8b0f83c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2490101Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#819384}
-
Wei Li authored
Add support for unique_ptr type of properties. Set their type converter to be non-serializable. Convert Combobox's 'model', 'owned_model', and 'callback' into properties. And apply them into the example builder to show the usages. Bug: 938501 Change-Id: I065f35420e266a0641754edf8edfebd8188f4144 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2477484 Commit-Queue: Wei Li <weili@chromium.org> Reviewed-by:
Peter Boström <pbos@chromium.org> Cr-Commit-Position: refs/heads/master@{#819383}
-
