- 12 Nov, 2018 40 commits
-
-
Patrik Höglund authored
This reverts commit af760c91. Reason for revert: Appears to cause flaky crashes in external/wpt/webusb/idlharness.https.any.worker.html. See https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.13%20Tests%20%28dbg%29/6367. The crash stack trace mentions SSL. If this CL is innocent, then please just reland. Original change's description: > Add more information to |error| of -didFailFavigationWithSSLError:. > > Add error.localizedDescription and error.userInfo[CWVCertStatusKey]. > > Bug: 898037 > Change-Id: I3e552330378992574d7c08f96f343aba6a7b4d29 > Reviewed-on: https://chromium-review.googlesource.com/c/1325578 > Reviewed-by: Carlos IL <carlosil@chromium.org> > Reviewed-by: Eugene But <eugenebut@chromium.org> > Commit-Queue: Hiroshi Ichikawa <ichikawa@chromium.org> > Cr-Commit-Position: refs/heads/master@{#607127} TBR=eugenebut@chromium.org,ichikawa@chromium.org,carlosil@chromium.org Change-Id: Ieffa8431079996675b1d53f058e4059fa74b8d09 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 898037, 904345 Reviewed-on: https://chromium-review.googlesource.com/c/1329972Reviewed-by:
Patrik Höglund <phoglund@chromium.org> Commit-Queue: Patrik Höglund <phoglund@chromium.org> Cr-Commit-Position: refs/heads/master@{#607205}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/cc4ed36d5b76..662c038f3fdd git log cc4ed36d5b76..662c038f3fdd --date=short --no-merges --format='%ad %ae %s' 2018-11-12 eroman@chromium.org Don't pass the --proxy-server flag when launching the browser for ChromeOS (remote). Created with: gclient setdep -r src/third_party/catapult@662c038f3fdd 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:902579,chromium:902418 TBR=sullivan@chromium.org Change-Id: I6f914d7d5118349371d42e1d32e6d2ef9dd84833 Reviewed-on: https://chromium-review.googlesource.com/c/1331186Reviewed-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@{#607204}
-
Mohamed Amir Yosef authored
FCM messages sent to lazy subscriptions are persisted if Chrome is in the background. This CL records UMA histogram for how many messages are stored at the time a message arrives. This is to estimate how large does the queue grow. Bug: 882887 Change-Id: I1f11e60862b2f356d4103faea45f54901597f27f Reviewed-on: https://chromium-review.googlesource.com/c/1296589 Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Reviewed-by:
Peter Conn <peconn@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Cr-Commit-Position: refs/heads/master@{#607203}
-
Marc Treib authored
TestSyncService already supports a good portion of the setters that are needed here, so this allows us to remove a bunch of boilerplate. Bug: 859874 Change-Id: I10a3e0cac0114d1df70b1ca72bec98f95aad0cb1 Reviewed-on: https://chromium-review.googlesource.com/c/1329795Reviewed-by:
Thomas Tangl <tangltom@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#607202}
-
Marc Treib authored
This new class encapsulates all the user-configurable bits of Sync. For now, it is built on top of ProfileSyncService's public API. Further CLs will migrate callers over to the new class, and then remove the public ProfileSyncService API. Bug: 884159 Change-Id: I2c6aa175f46712d408d7b701292390c539399531 Reviewed-on: https://chromium-review.googlesource.com/c/1283689 Commit-Queue: Marc Treib <treib@chromium.org> Reviewed-by:
Mikel Astiz <mastiz@chromium.org> Cr-Commit-Position: refs/heads/master@{#607201}
-
Keishi Hattori authored
Convert HTML*, Document* class allocations from new to MakeGarbageCollected Bug: 757708 Change-Id: I56a9eb7c23f4adaab3b367c76a8a5358d014b2e3 Reviewed-on: https://chromium-review.googlesource.com/c/1331350Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#607200}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/9d6971eb..0c8e54cc Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I221a7dcbbefae1e0bbc850077084fb05647e4b51 Reviewed-on: https://chromium-review.googlesource.com/c/1331182Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#607199}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/8a7d7b28b1ed..7cdd680b3802 git log 8a7d7b28b1ed..7cdd680b3802 --date=short --no-merges --format='%ad %ae %s' 2018-11-12 taylori@google.com Merge "trace_to_text: Store thread names" 2018-11-12 lalitm@google.com Merge "trace_processor: add stats table to expose parsed stats data" Created with: gclient setdep -r src/third_party/perfetto@7cdd680b3802 The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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=perfetto-bugs@google.com Change-Id: Ib69ee02d879cecd866e65aa0699573feaba4b980 Reviewed-on: https://chromium-review.googlesource.com/c/1331217Reviewed-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@{#607198}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/716adb3d87b3..768184982153 Created with: gclient setdep -r src-internal@768184982153 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-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=mmoss@chromium.org Change-Id: I27d979c39067c7279b5afa04734880280e65b51b Reviewed-on: https://chromium-review.googlesource.com/c/1331216Reviewed-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@{#607197}
-
Mihai Sardarescu authored
On Windows, Taskbar and Desktop icons are refreshed every time |OnProfileAvatarChanged| notification is fired. This CL only fires the notification |OnProfileAvatarChanged| when the profile avatar image actually changes and avoids firing it otherwise. Bug: 900374 Change-Id: I9c930a3eab9d59aa91c785715336d520dcc813c5 Reviewed-on: https://chromium-review.googlesource.com/c/1323711 Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#607196}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/f2c3502f7788..cc4ed36d5b76 git log f2c3502f7788..cc4ed36d5b76 --date=short --no-merges --format='%ad %ae %s' 2018-11-12 perezju@chromium.org [pinpoint_cli] Read job config from any config file Created with: gclient setdep -r src/third_party/catapult@cc4ed36d5b76 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:879526 TBR=sullivan@chromium.org Change-Id: I7e688abd73d5166055b53e62e6c81b510f78b3d9 Reviewed-on: https://chromium-review.googlesource.com/c/1331185Reviewed-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@{#607195}
-
Andreas Haas authored
After more than half a year of finch trial we can turn WebAssembly trap handler on Linux on by default. R=titzer@chromium.org, hablich@chromium.org, piman@chromium.org Bug: chromium:722585 Change-Id: Id52754dba6d80ce794e350216b9a0c0d93c76ebd Reviewed-on: https://chromium-review.googlesource.com/c/1322872Reviewed-by:
Michael Hablich <hablich@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Andreas Haas <ahaas@chromium.org> Cr-Commit-Position: refs/heads/master@{#607194}
-
Corentin Wallez authored
The missing dependency would cause the resource_ids to not be regenerated when the predetermined ids file were changed, causing various compilation failures on bots. BUG=904366 TBR=thestig@chromium.org Change-Id: I310af635916e66ad7d35437d7116b2a3f7fae2a9 Reviewed-on: https://chromium-review.googlesource.com/c/1329793Reviewed-by:
Corentin Wallez <cwallez@chromium.org> Commit-Queue: Corentin Wallez <cwallez@chromium.org> Cr-Commit-Position: refs/heads/master@{#607193}
-
Gauthier Ambard authored
This CL adds features available to the collection view root panel to the table view root panel in the settings. More specifically it adds support for the top right button. It also adds a helper to remove items from the model to the ChromeTableViewController. Bug: 894791 Change-Id: I1d0f4836f160379a0d8ab71e91f3a0ad13a38e5f Reviewed-on: https://chromium-review.googlesource.com/c/1304437 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#607192}
-
Friedrich Horschig authored
This small refactoring enables the provides to be used for objects instead of arrays which will come in handy for the AccessoryData type that is introduced with https://crrev.com/c/1320732 TBR=rouslan@chromium.org Change-Id: I237813484c1ffcfe0357a30672b893db9c9b2d32 Reviewed-on: https://chromium-review.googlesource.com/c/1326512Reviewed-by:
Friedrich Horschig [CET] <fhorschig@chromium.org> Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Friedrich Horschig [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#607191}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/1ba4ba601cc4..8a7d7b28b1ed git log 1ba4ba601cc4..8a7d7b28b1ed --date=short --no-merges --format='%ad %ae %s' 2018-11-12 lalitm@google.com Merge "trace_processor: always print out longs in decimal" Created with: gclient setdep -r src/third_party/perfetto@8a7d7b28b1ed The AutoRoll server is located here: https://autoroll.skia.org/r/perfetto-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=perfetto-bugs@google.com Change-Id: I74daebad12e03ce0dbe5e4f74a3040705f38f416 Reviewed-on: https://chromium-review.googlesource.com/c/1331184Reviewed-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@{#607190}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/f75973390723..716adb3d87b3 Created with: gclient setdep -r src-internal@716adb3d87b3 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-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=mmoss@chromium.org Change-Id: I41c9e4c57d1a3ac7396c3f1f3605f36d9f1bf3f0 Reviewed-on: https://chromium-review.googlesource.com/c/1331180Reviewed-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@{#607189}
-
Jérôme Lebel authored
MDCShadowElevations.m has been removed from material_components_ios/src/components/ShadowElevations https://critique.corp.google.com/#review/220518015 TBR: rohitrao Bug: 903150 Change-Id: I07d7150b3b4fa2fca01bcbbb718bc4a9c8081f60 Reviewed-on: https://chromium-review.googlesource.com/c/1329250Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#607188}
-
Carlos Caballero authored
MessageLoop will go away, eventually. BUG=891670 Change-Id: I6992d6773ec3f753fe8c225dee223b522593347b Reviewed-on: https://chromium-review.googlesource.com/c/1323449Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Commit-Queue: Carlos Caballero <carlscab@google.com> Cr-Commit-Position: refs/heads/master@{#607187}
-
A Olsen authored
crbug.com/433840 says we should split the read and write path for cros settings (or device settings), which will simplify the read path and get rid of its dependency on the write path. Calls to CrosSettings->Set(kSystemTimezone) don't actually affect signed device settings (unlike other cros settings), but end up calling TimezoneSettings->SetTimezone. We can simplify the cros settings interface by calling this directly. This CL replaces the last call to CrosSettings->Set(kSystemTimezone) with an equivalent call to timezone_util, which also delegates to TimezoneSettings. Bug: 433840 Change-Id: Ibaac986323cb286eab83ec7c0b29780dc2ef7d87 Reviewed-on: https://chromium-review.googlesource.com/c/1326144Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: A Olsen <olsen@chromium.org> Cr-Commit-Position: refs/heads/master@{#607186}
-
Arthur Sonzogni authored
This test a server redirect, for which the new url is redirected by the extension. The bug caused DumpWithoutCrashing to be reached in: * https://chromium-review.googlesource.com/c/1319710 * https://chromium-review.googlesource.com/c/1316734 The bug is fixed by: https://chromium-review.googlesource.com/c/chromium/src/+/1326481 Bug: 882661, 880741 Change-Id: I2952dd601fba7fff8b8e110d434413e22e751bba Reviewed-on: https://chromium-review.googlesource.com/c/1328983Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#607185}
-
Yutaka Hirano authored
This is preliminary work for reading data pipes in RequestPeer implementations. Bug: 894819 Change-Id: I93ce399fa83796ab6fd095ea7dbf4666fb1458d4 Reviewed-on: https://chromium-review.googlesource.com/c/1312184Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Commit-Queue: Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#607184}
-
Colin Blundell authored
This is a drive-by cleanup: On tonikitoo's chain of CLs converting UserCloudPolicyTokenForwarder to IdentityManager, I noticed the CloudPolicyService::Observer::OnInitializationComplete() method. This method is ambiguously named from a client POV, as their override simply ends up being Foo::OnInitializationComplete(). This was especially weird from the POV of UserCloudPolicyTokenForwarder because its implementation of that method starts its *own* initialization, i.e., invokes Initialize() ;). This CL renames the method to OnCloudPolicyServiceInitializationComplete() and additionally eliminates the CloudPolicyService* argument, which is not needed: its only usage is to have a few clients confirm that it is the same as the instance on which they registered as an observer, but this will always be the case by the nature of observer callbacks. Change-Id: Iaf73c26b9b4ad1305ad6774c23da0828200d499c Reviewed-on: https://chromium-review.googlesource.com/c/1329146Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Commit-Queue: Colin Blundell <blundell@chromium.org> Cr-Commit-Position: refs/heads/master@{#607183}
-
Hirokazu Honda authored
VideoFrameDoneNotifier is unnecessary. It is the helper class to execute a given callback in the destructor. That totally can be done with VideoFrame::AddDestructionObserver. This just removes VideoFrameDoneNotifier. BUG=None TEST=CtsMediaTestCases Change-Id: I30222876159565f2b639d92650d4ab3fa6de385b Reviewed-on: https://chromium-review.googlesource.com/c/1331103 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
Pawel Osciak <posciak@chromium.org> Cr-Commit-Position: refs/heads/master@{#607182}
-
Yuki Shiino authored
EventListener::handleEvent => Invoke JSBasedEventListener::CallListenerFunction => InvokeInternal EventListener::Code => ScriptBody Change-Id: I5d73913f5808eb7767cae4e37eeaa437ba3d7e8c Reviewed-on: https://chromium-review.googlesource.com/c/1329123Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Hitoshi Yoshida <peria@chromium.org> Commit-Queue: Yuki Shiino <yukishiino@chromium.org> Cr-Commit-Position: refs/heads/master@{#607181}
-
Mikel Astiz authored
This cleans up the remaining leftovers of sessions-related logic from interface SyncService, including: 1. The propagation of PROXY_TABS's runtime state directly from the controller to SessionSyncService. 2. The removal of the getter, SyncService::GetOpenTabsUIDelegate(). 3. The removal of SyncServiceObserver::OnForeignSessionUpdated(), now replaced by a new subscription mechanism in SessionSyncService. Besides simplifying core sync code, it also means client code (UI and tests) have less dependencies, since they can now depend exclusively on SessionSyncService. Bug: 883199 Cq-Include-Trybots: luci.chromium.try:ios-simulator-cronet;luci.chromium.try:ios-simulator-full-configs Change-Id: Id89410fa7149ae97db8e707adbab331e68d5cb8b Reviewed-on: https://chromium-review.googlesource.com/c/1271101 Commit-Queue: Mikel Astiz <mastiz@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Reviewed-by:
calamity <calamity@chromium.org> Reviewed-by:
Olivier Robin <olivierrobin@chromium.org> Cr-Commit-Position: refs/heads/master@{#607180}
-
Morten Stenshorne authored
LayoutObject::PropagateStyleToAnonymousChildren() doesn't do the right thing for LayoutMultiColumnSpannerPlaceholder. Spanner placeholders need to copy the margins from the actual spanner object, in addition to inheriting properties from the parent of the actual spanner. Bug: 903699 Change-Id: I4252c546610eca2698efd523f98f20a98627d217 Reviewed-on: https://chromium-review.googlesource.com/c/1328986 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#607179}
-
Keishi Hattori authored
Converts DOM*, AX*, Event* class allocations from new to MakeGarbageCollected Bug: 757708 Change-Id: I872e34038e2ef3d4c3913d7cd94d1e4da7888ff4 Reviewed-on: https://chromium-review.googlesource.com/c/1331348Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#607178}
-
Keishi Hattori authored
Convert SVG* classs allocations from new to MakeGarbageCollected Bug: 757708 Change-Id: Ifb603bcec1bae98d3763738627dd06248e8b6dae Reviewed-on: https://chromium-review.googlesource.com/c/1331104Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#607177}
-
Mihai Sardarescu authored
This reverts commit ec686dc0. Reason for revert: The fix does not work in all cases and it adds a lot of complexity in the way avatar images are now used. There are now 3 sources of truth for the avatar icon index and it is not clear what the relationship between them is: * kAccountIdKey in the profile entry cache * kProfileAvatarIndex which is a synced pref. * kLocalProfileAvatarIndex which is non-syncable pref. Note that kProfileAvatarIndex and kLocalProfileAvatarIndex are both set when the user updates the image icon in chrome://settings/manageProfile. I've just tested and if I start sync, set a local avatar icon, then turn off sync, I still have the same image. So this CL does not fix the bug 863008 for this scenario. The core issue comes from the fact that we try to fix a problem that cannot be fixed in the current architecture of sync - sync preferences do not have a local variant and do not get reverted to a local variant when the user turns sync off. I think doing a custom implementation for the profile avatar (and not for the profile name for example or settings) just adds confusion. The fact that the relationship between these preferences is not very clear, fixing the issue 900374 becomes hard and is risky. Bug: 900374 Original change's description: > Restore avatar after user logs out from sync. > > BUG=863008 > > Change-Id: If083bc0703e251fb6e9c597fe5b7f842e37875c9 > > R=bauerb@chromium.org,sky@chromium.org > > Change-Id: If083bc0703e251fb6e9c597fe5b7f842e37875c9 > Reviewed-on: https://chromium-review.googlesource.com/1135137 > Commit-Queue: Thomas Tangl <tangltom@chromium.org> > Reviewed-by: Bernhard Bauer <bauerb@chromium.org> > Reviewed-by: Thomas Tangl <tangltom@chromium.org> > Reviewed-by: David Roger <droger@chromium.org> > Reviewed-by: Scott Violet <sky@chromium.org> > Cr-Commit-Position: refs/heads/master@{#582541} TBR=sky@chromium.org,droger@chromium.org,bauerb@chromium.org,tangltom@chromium.org,boocmp@yandex-team.ru # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 863008 Change-Id: I46aed023494d94e197828339f6870bf9ba0c9da8 Reviewed-on: https://chromium-review.googlesource.com/c/1329153Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Thomas Tangl <tangltom@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
anthonyvd <anthonyvd@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#607176}
-
Yuta Kitamura authored
This enum is exposed to //content and other non-Blink code, so its name should start with "Web". Bug: 826203 Tbr: creis@chromium.org Change-Id: I9fc9aa50ef276dae7ecd64a3bd299590893b19a8 Reviewed-on: https://chromium-review.googlesource.com/c/1328549 Commit-Queue: Yuta Kitamura <yutak@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#607175}
-
Gauthier Ambard authored
This CL adds animations when the user is tapping the "switch to this tab" indicator in the omnibox suggestions. Bug: 893121 Change-Id: Ia7af97f6175efa959bbb2e84ab5661f00adedf81 Reviewed-on: https://chromium-review.googlesource.com/c/1322871Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#607174}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/8668e959c79c..f75973390723 Created with: gclient setdep -r src-internal@f75973390723 The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-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=mmoss@chromium.org Change-Id: I5e889b1d9d96e6795289f359911c1b4e77cb9268 Reviewed-on: https://chromium-review.googlesource.com/c/1331214Reviewed-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@{#607173}
-
Hajime Hoshi authored
V8 should not be executed on per-thread task runner. Use per-frame (per- execution-context) task runner instead. Bug: 870606 Change-Id: I353c421901f0ff78966df5d197014f8cb650a985 Reviewed-on: https://chromium-review.googlesource.com/c/1331289Reviewed-by:
Taiju Tsuiki <tzik@chromium.org> Commit-Queue: Hajime Hoshi <hajimehoshi@chromium.org> Cr-Commit-Position: refs/heads/master@{#607172}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/7dbd8f071016..cb830a07e835 git log 7dbd8f071016..cb830a07e835 --date=short --no-merges --format='%ad %ae %s' 2018-11-12 yunlian@google.com generic_stages: temporarily ignore all target prebuilts. Created with: gclient setdep -r src/third_party/chromite@cb830a07e835 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. TBR=chrome-os-gardeners@chromium.org Change-Id: I19be62a112191cc9f7a6c3a67e69f9d690350139 Reviewed-on: https://chromium-review.googlesource.com/c/1331175Reviewed-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@{#607171}
-
Chromium WPT Sync authored
Using wpt-import in Chromium d9349484. With Chromium commits locally applied on WPT: 270d4abd "ServiceWorker: Add new WPT tests to make sure to update a registration with different script type and identical script content." 9ff65400 "Add redirected cases to CSP/import tests in wpt/worklets" 687313c5 "Add action_sequence in testdriver" 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 NOAUTOREVERT=true TBR=foolip No-Export: true Change-Id: I740b7ffb027e29aa60d4e9653ffae9f37f3d7b41 Reviewed-on: https://chromium-review.googlesource.com/c/1331176 Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#607170}
-
Yuzu Saijo authored
This CL adds new UMAs that report the amount of memory metrics that's reduced as a result of intervention. Bug: 888935 Change-Id: Ie336559f2046f0929830df7bf3b2d35ce7887423 Reviewed-on: https://chromium-review.googlesource.com/c/1319399Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Yuzu Saijo <yuzus@chromium.org> Cr-Commit-Position: refs/heads/master@{#607169}
-
Yoichi Osato authored
This patch adds test cases for selection crossing blocks. The test checks regression behavior that reverted crrev.com/c/1182710 caused. Bug: 892584 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng Change-Id: I299ca3323c201a71267f50f8bba95269f759a0fe Reviewed-on: https://chromium-review.googlesource.com/c/1272819Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Commit-Queue: Yoichi Osato <yoichio@chromium.org> Cr-Commit-Position: refs/heads/master@{#607168}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/cfd15c4df421..8668e959c79c Created with: gclient setdep -r src-internal@8668e959c79c The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-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=mmoss@chromium.org Change-Id: If1d3756e747d30ef39186a532ecad0989cafc64a Reviewed-on: https://chromium-review.googlesource.com/c/1331213Reviewed-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@{#607167}
-
Alexey Baskakov authored
Instead of extension-based sizes. Bug: 901226 Change-Id: I4729ac0206fea143980db4cf43de67a71d83f7be Reviewed-on: https://chromium-review.googlesource.com/c/1329819Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Alexey Baskakov <loyso@chromium.org> Cr-Commit-Position: refs/heads/master@{#607166}
-