- 15 Dec, 2020 40 commits
-
-
Nick Czajka authored
Updates the text on Settings > Device > Displays > Arrangement to include keyboard instructions, when keyboard-based display alignment is enabled. Screenshot: http://screen/6iBtZF5bP7YoJTY Bug: 1077365 Change-Id: I6a1599cc2185fda0487e184c68b0c50625318b4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593173Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Commit-Queue: Nick Czajka <czajka@google.com> Cr-Commit-Position: refs/heads/master@{#837291}
-
Michael Crouse authored
This is a reland of ffac4483 Original change's description: > [LanguageDetection] SimpleKeyedService and optimization guide. > > > This change converts the translate model service to a > SimpleKeyedService and moves it to components which will enable > easier support beyond just browser use cases. > > This change also registers the translate model service to > the optimization guide for provided necessary model files. > > A future change will add loading and service the model files > to consumers of the translate model service. > > Bug: 1151407 > Change-Id: I4e903115c82a30757d46d75ed8d733874726d830 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580339 > Commit-Queue: Michael Crouse <mcrouse@chromium.org> > Reviewed-by: Colin Blundell <blundell@chromium.org> > Reviewed-by: Trevor Perrier <perrier@chromium.org> > Reviewed-by: Scott Little <sclittle@chromium.org> > Reviewed-by: Tarun Bansal <tbansal@chromium.org> > Reviewed-by: Sophie Chang <sophiechang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#836656} Bug: 1151407 Change-Id: I3606aa9a1292a4a61828e9be9ec91714bb8d7575 TBR: blundell@chromium.org, sclittle@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592167 Commit-Queue: Michael Crouse <mcrouse@chromium.org> Reviewed-by:
Michael Crouse <mcrouse@chromium.org> Cr-Commit-Position: refs/heads/master@{#837290}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/b1d06a4d0549..f1c7d0dd2ddc 2020-12-15 vapier@chromium.org gerrit_util: move changes: enable keep_all_votes If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC ehmaldonado@google.com,sokcevic@google.com,ajp@google.com,apolito@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 Bug: None Tbr: ehmaldonado@google.com,sokcevic@google.com,ajp@google.com,apolito@google.com Change-Id: I56e5f28ce65f0b0172b5b40c3e6b1217ac317110 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593996Reviewed-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@{#837289}
-
Ben Mason authored
TBR=govind@chromium.org Change-Id: I20328d6027e0c870437f786fea2bea7aa3a75388 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593754Reviewed-by:
Ben Mason <benmason@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Commit-Queue: Ben Mason <benmason@chromium.org> Cr-Commit-Position: refs/heads/master@{#837288}
-
Yao Xiao authored
Bug: 1156543 Change-Id: Ie148e7f191d4148c697059ce3bdfd430c84b1411 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2578478Reviewed-by:
Josh Karlin <jkarlin@chromium.org> Commit-Queue: Yao Xiao <yaoxia@chromium.org> Cr-Commit-Position: refs/heads/master@{#837287}
-
Toni Barzic authored
Makes the previews animation better match the speced behavior. More fundamental change is the way the list of items to be shown in the icon is calculated - instead of having HoldingSpaceTrayIcon track the holding space model changes and update itself on each change, the HoldingSpaceTrayIcon now does not depend directly on the holding space state. Instead, HoldingSpaceTray calculates the list of items to display in the icon, and passes it on to HoldingSpaceTrayIcon. The icon calculates and runs transitions needed to update the icon to match the new state. Unifying the API to update the set of items in the icon makes batching the holding space items easier - it's sufficient to schedule an async task to calculate the list of items in the model, and pass it on to the icon when the time is right. Other changes: HoldingSpaceTrayIconPreview now tracks its current and pending index in the item list, which simplifies calculating transitions, and simplifies logic for determining whether the preview needs a layout. Also, the indices can be used to distinguish new from existing items when updating the icon (new items will not have the current index set). Unifies AnimateShift and AnimateUnshift into a single method - the method now uses pending preview index to calculate the target transform, which makes the distinction between shift and unshift irrelevant. BUG=1142572 Change-Id: Id614c39838d0aa10dc2652d4a23b8e934d67bdbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2582431 Commit-Queue: Toni Baržić <tbarzic@chromium.org> Reviewed-by:
David Black <dmblack@google.com> Cr-Commit-Position: refs/heads/master@{#837286}
-
Anne Lim authored
Changed return type of GetExpiryTimeMicros() to base::Optional<int_64>. Bug: 1061639 Change-Id: Ic5dd406a6dbbf7dea4725f546cdfd9dc129bf163 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587525Reviewed-by:
Jared Saul <jsaul@google.com> Commit-Queue: Anne Lim <annelim@google.com> Cr-Commit-Position: refs/heads/master@{#837285}
-
Mike Dougherty authored
CRWJSInjectionManager is deprecated and language_detection is one of the few remaining features using it. Migrating it off of this dependency will help to cleanup existing JavaScript feature code in preparation for new JavaScript features. Note that although language_detection_controller_unittest is removed here, it was previously testing private API sand the JS still has tests in language_detection_javascript_unittest. Bug: 949178 Change-Id: I9834ca60f8e8d4d75c324a5f93337ed04b997da5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580902 Commit-Queue: Mike Dougherty <michaeldo@chromium.org> Reviewed-by:
John Wu <jzw@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#837284}
-
Angus L. M. McLean IV authored
Bug: 1135270 Change-Id: I7573ec1adf7b0623f3d60716a2d0e575cd31cac9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590015 Commit-Queue: Angus McLean <angusmclean@chromium.org> Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Cr-Commit-Position: refs/heads/master@{#837283}
-
Will Harris authored
reddit.com has migrated to a mobile design but old.reddit.com retains the old design which does not play well with DOM distiller. BUG=957667 Change-Id: Idc892fe13ff57d05422521c6c4dbb0db3f45e98d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594208Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#837282}
-
Vladimir Levin authored
This patch ensures that we unlock the auto locks when printing, since that makes the whole page essentially visible. R=chrishtr@chromium.org Change-Id: Ia49d20d6885e1391d4db633df08db8f41caf1441 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593873Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: vmpstr <vmpstr@chromium.org> Cr-Commit-Position: refs/heads/master@{#837281}
-
ckitagawa authored
It looks like the PaintPreviewCompositorClientImpl very rarely gets deleted before the mojo channel is established on a different sequence (for bitmap IO reasons). The mojo remote is already deleted on this other sequence (see its initialization using a TaskRunnerDeleter in PaintPreviewCompositorClientImpl) so instead of passing PaintPreviewCompositorClientImpl* into the post task (which could be deleted at any time), we should pass the mojo remote. The remote is guaranteed to be deleted on the same sequence as the task binding it and we know it exists before the bind task is posted so by definition of a sequence it will now be deleted only after the binding task is executed thus avoiding the race. Before this fix it was possible the PaintPreviewCompositorClientImpl* would be deleted so calling the BindPipeAndPassReceiver proxy method on it was unsafe. I was able to repro locally with a hack that deleted the PaintPreviewCompositorClientImpl right after the PostTask and based on that I'm reasonably confident this fix resolves the issue. Bug: 1157019 Change-Id: I85b2daf36260cc8a9dbd8515e45624b4effe1379 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593875 Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> Commit-Queue: Mehran Mahmoudi <mahmoudi@chromium.org> Reviewed-by:
Mehran Mahmoudi <mahmoudi@chromium.org> Cr-Commit-Position: refs/heads/master@{#837280}
-
James Cook authored
For lacros, both browser and OS share a single consent for metrics reporting. The main pref is in ash. It is policy controlled by the DeviceMetricsReportingEnabled policy. For policy-controlled devices, lacros always has the right value for metrics reporting (it dynamically tracks the value in ash). However, we need to tell lacros whether or not the ash-side pref is managed, so lacros can show the correct "managed" icon in browser settings. Device policy is synced before signin, and is therefore available before lacros starts, so pass the managed state as part of LacrosInitParams. Long-term we'll need a more sophisticated solution. This device policy is "dynamic", meaning it could change from managed to unmanaged during a user session, so we'll need some kind of observer mechanism for whether it is managed or not. However, there's some general purpose policy work underway for lacros, so this should hold us until that work is done. Test: enroll device in managedchrome.com, disable metrics via device policy, ensure managed icon shows up in browser prefs Bug: 1148604 Change-Id: Ifd6818b6bf2565e987d62b28f9826d612b4b04a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2591774Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Commit-Queue: James Cook <jamescook@chromium.org> Cr-Commit-Position: refs/heads/master@{#837279}
-
Yulun Wu authored
Bug: 1134433 Change-Id: I69ba2d06d2d99d3295b71449a33db1adc863c429 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594114Reviewed-by:
Toni Baržić <tbarzic@chromium.org> Reviewed-by:
Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Yulun Wu <yulunwu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Yulun Wu <yulunwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#837278}
-
Kelvin Jiang authored
This CL replaces the int UserScript id with string. This is necessary for dynamic content scripts which have ids that are specified by the extension. To distinguish generated ids from future user defined ones, generated ids are prefixed with "_". Bug: 1054624 Change-Id: I3d439d6f52b1ec2edb0d847847684ed19046a7e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2570856Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Commit-Queue: Kelvin Jiang <kelvinjiang@chromium.org> Cr-Commit-Position: refs/heads/master@{#837277}
-
Meilin Wang authored
This CL adds two histograms: * Nearby.Connections.Bluetooth.Socket.Read.Result * Nearby.Connections.Bluetooth.Socket.Write.Result to track the result of socket operations. Bug: b/163064897 Change-Id: If5a4cce7133d1aee41d6d5c33a3581bbe3fcca39 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587580 Commit-Queue: Meilin Wang <meilinw@chromium.org> Reviewed-by:
Jesse Doherty <jwd@chromium.org> Reviewed-by:
Ryan Hansberry <hansberry@chromium.org> Cr-Commit-Position: refs/heads/master@{#837276}
-
Yi Gu authored
Bug: None Change-Id: I0584cbdbf6b0dc47e3b7a0dc6a98c4839a2105d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593556Reviewed-by:
Sam Goto <goto@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#837275}
-
Sophie Chang authored
This is a reland of 84f1c409 TBR=droger@chromium.org,mcrouse@chromium.org Original change's description: > Implement read-only mode for Optimization Guide when in incognito > > Bug: 1151087 > Change-Id: I4a6074a01793c5a97de7f07e14ccbd133ef6cb51 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586246 > Reviewed-by: David Roger <droger@chromium.org> > Reviewed-by: Tarun Bansal <tbansal@chromium.org> > Reviewed-by: Michael Crouse <mcrouse@chromium.org> > Commit-Queue: Sophie Chang <sophiechang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#836442} Bug: 1151087 Change-Id: I0586433a365540bdc99a82a22061f8c9ffc65064 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593811 Commit-Queue: Sophie Chang <sophiechang@chromium.org> Reviewed-by:
Josh Simmons <jds@google.com> Cr-Commit-Position: refs/heads/master@{#837274}
-
Brian Sheedy authored
Fixes the unexpected pass finder not working with the power suite due to a mismatch between the suite name and the expectation file name. Bug: 998329 Change-Id: I7133347683870c6db15dc81ae66d809d5c8beba4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592388 Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#837273}
-
Keith Lee authored
All functionality related to InputMethodKeyboard* were designed to handle the onscreen / virtual keyboard. The class is not directly related to InputMethods. Hence, should be renamed to VirtualKeyboard*. Bug: 1158633 Change-Id: I06569bcae62755826bfc7dcf479a29dd90db7175 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592207Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Commit-Queue: Keith Lee <keithlee@chromium.org> Cr-Commit-Position: refs/heads/master@{#837272}
-
Michael Thiessen authored
Other ContextMenuParams getters to follow. Bug: 783819 Change-Id: I6612d31f3c62d5434ce29420d6b36b3718b7b2e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592010 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Yaron Friedman <yfriedman@chromium.org> Cr-Commit-Position: refs/heads/master@{#837271}
-
Hongbo Song authored
Change-Id: I5e67d5c916b46a6dda381c3bf05d37dcc2f7172e Bug: 1152552 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594008Reviewed-by:
Steve Kobes <skobes@chromium.org> Commit-Queue: Hongbo Song <hbsong@google.com> Cr-Commit-Position: refs/heads/master@{#837270}
-
Aaron Krajeski authored
Part of the greater effort to migrate cc and the paint_op_buffer away from SkMatrix, which is 3x3 and relatively slow, compared to SkM44. SetMatrix(SkM44) was added. Once concat(SkM44) is added we can make the PaintCanvas version of these functions that use SkMatrixs just call these functions and remove the SkMatrix versions within the paint op buffer. Bug: 1155544 Change-Id: Id0a866cba2ea8eb44822d9632bd890980ab707c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2577704Reviewed-by:
Caleb Raitto <caraitto@chromium.org> Reviewed-by:
Aaron Krajeski <aaronhk@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Aaron Krajeski <aaronhk@chromium.org> Cr-Commit-Position: refs/heads/master@{#837269}
-
Elly Fong-Jones authored
They are no longer on the project and email to them bounces. Bug: 1148873 Change-Id: Ib235c178f61a58ae34176558ad99acc56b9452a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593932Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#837268}
-
Zhiyuan Cai authored
Make the menu item "Track prices on tabs" invisible if user isn't signed in or is in incognito mode. Bug: 1144742, 1155688 Change-Id: Id124ed77289246d1b2e4ddcbe245c3be84fcebb7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2575578 Commit-Queue: Zhiyuan Cai <zhiyuancai@google.com> Reviewed-by:
David Maunder <davidjm@chromium.org> Reviewed-by:
Yue Zhang <yuezhanggg@chromium.org> Reviewed-by:
Yusuf Ozuysal <yusufo@chromium.org> Cr-Commit-Position: refs/heads/master@{#837267}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/41545839b323..ae8efc5c4e6a 2020-12-15 chrche@microsoft.com [Loc] Migrate persistence to localization V2 2020-12-15 jacktfranklin@chromium.org Tests and fix for data grid resizing with hidden column If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@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 Bug: chromium:1136655 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I35a5335478b7701873bc17c7ba320b1ba1617539 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594109Reviewed-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@{#837266}
-
Nicolás Peña Moreno authored
Currently ChromeClient uses the presentation timestamp from the corresponding presentation callback. This CL renames the users of such methods to reduce confusion around what timestamps are being used. Change-Id: I84729f50b67e1c2ae0bc46f891af3889cc382292 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2561524 Commit-Queue: Nicolás Peña Moreno <npm@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
Steve Kobes <skobes@chromium.org> Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Cr-Commit-Position: refs/heads/master@{#837265}
-
Jiahe Zhang authored
In this CL, a feature flag is added to do power experiment on throttling the timers of those visible but not focused pages to 1s interval. Local test shows this can save ~15% (CrOS) to ~50% (Win) of package power for web pages like weather.com. Bug: 1155467 Change-Id: I79ca20ed2bd9ca246591e7ea328b14a28071da34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2569450 Commit-Queue: François Doray <fdoray@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#837264}
-
Gil Dekel authored
This CL modifies GetModesetBuffer() to skip the copying or clearing of a freshly acquired buffer when we mode test, since it will not be used in the next page flip. This reduced the average traced duration of GetModesetBuffer() from 1.8ms (min:0.008ms to max:5.36ms) to 0.006ms (min:0.004ms to max:0.029ms). This is a significant reduction since GetModesetBuffer() is called num_of_displays x num_of_mode_tests times per display configuration. The average traced duration of a single TestModeset() was reduced from 8.9ms (min:5.04ms to max:16.9ms) to 0.059ms (min:0.04ms to max:0.199ms). Tracing was collected on a Drallion, with two 4K 13.3" monitors connected via MST and running 1,000 consecutive TestModeset(). Test: Trace on a Drallion Change-Id: I0673f421dab597d45ab8e5666e17b2612bbca55a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2590393Reviewed-by:
Mark Yacoub <markyacoub@google.com> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Commit-Queue: Gil Dekel <gildekel@chromium.org> Cr-Commit-Position: refs/heads/master@{#837263}
-
Daniel Hosseinian authored
In most (if not all) RTL languages, numbers and fractions are more naturally read LTR. Fixed: 1158896 Change-Id: I1b94d10479982576b9a551f28774ad50a5d79071 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593515 Commit-Queue: Daniel Hosseinian <dhoss@chromium.org> Commit-Queue: dpapad <dpapad@chromium.org> Auto-Submit: Daniel Hosseinian <dhoss@chromium.org> Reviewed-by:
dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#837262}
-
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-aemu-chromium-autoroll Please CC chrome-fuchsia-gardener@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 Tbr: chrome-fuchsia-gardener@grotations.appspotmail.com Change-Id: I33ba14febfd54de55facc0249feebc7caeb35b1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593714Reviewed-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@{#837261}
-
Edward Lesmes authored
Bug: 1113033 Change-Id: I5773ff4c9a5afc50cf13d519e58f6d0a3fc3796f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593971 Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org> Commit-Queue: sebsg <sebsg@chromium.org> Reviewed-by:
sebsg <sebsg@chromium.org> Cr-Commit-Position: refs/heads/master@{#837260}
-
Sebastien Marchand authored
Bug: 1144025 Change-Id: Ieb15a25b91dfeba39fd44a9777c949a21f2a5f67 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2561765 Auto-Submit: Sébastien Marchand <sebmarchand@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
François Doray <fdoray@chromium.org> Commit-Queue: Sébastien Marchand <sebmarchand@chromium.org> Cr-Commit-Position: refs/heads/master@{#837259}
-
Alexander Cooper authored
In preparation for removing the WMR runtime, the tests need to be removed as well. Note that this does leave an interesting situation for the macros, as they are now only needed when we wish to run both an incognito test and a normal test. However, there is a significant amount of chrun when converting away from the macros (primarily the removal of the t-> references). Because of this, a "dummy" macro is currently added which will allow the code to still compile. A future change will remove the outdated multi-class macros. Bug: 1121632 Change-Id: I993357e87b6cf03b2260a14861289d838846ef7d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2587556 Commit-Queue: Alexander Cooper <alcooper@chromium.org> Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#837258}
-
Evan Stade authored
Skip-Translation-Screenshots-Check: True Bug: 1147276 Change-Id: Ia2e75df45c94f9ec4fe8d1d7b4986dedc2d4eecd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586963Reviewed-by:
Ted Choc <tedchoc@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Auto-Submit: Evan Stade <estade@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#837257}
-
Victor Costan authored
Change-Id: I9e62fcf1a06a8c1513c8c64662bfb4e7ac6f9419 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592927 Auto-Submit: Victor Costan <pwnall@chromium.org> Commit-Queue: Ayu Ishii <ayui@chromium.org> Reviewed-by:
Ayu Ishii <ayui@chromium.org> Cr-Commit-Position: refs/heads/master@{#837256}
-
Liquan (Max) Gu authored
This CL inlines initAndValidate() in init(). This CL causes no behavioural change. Other changes: * The result of getValidatedMethodData() is null-check before used. Bug: 1157847 Change-Id: Ie2fcc98f4089cb3cdf196794c5440cf232af115c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2592192 Commit-Queue: Liquan (Max) Gu <maxlg@chromium.org> Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Cr-Commit-Position: refs/heads/master@{#837255}
-
chromium-autoroll authored
Roll Chrome Mac PGO profile from chrome-mac-master-1608033365-b773312111f7e1aa196b8e797089c32200e6b39a.profdata to chrome-mac-master-1608054952-4392c501b8d700164ac6acf89580f5ab35749e86.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-mac-chromium Please CC pgo-profile-sheriffs@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 Cq-Include-Trybots: luci.chrome.try:mac-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: Ibd7ba34300bf4b9d5169b36a80b8eb739253208a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2593712Reviewed-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@{#837254}
-
chromium-autoroll authored
Roll ChromeOS Bigcore AFDO profile from 89-4324.22-1607342008-benchmark-89.0.4351.0-r1 to 89-4342.0-1607944823-benchmark-89.0.4351.0-r1 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/cros-afdo-bigcore-chromium Please CC c-compiler-chrome@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: c-compiler-chrome@google.com Change-Id: Ic0627d56c7d661208003a38e2b39aa100e67ee07 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594187Reviewed-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@{#837253}
-
Thomas Tellier authored
Bug: 1136487 Change-Id: I893d3e43bad10f4cdbca7d22e320739c5dce317a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2582119Reviewed-by:
Steven Holte <holte@chromium.org> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Commit-Queue: Thomas Tellier <tellier@google.com> Cr-Commit-Position: refs/heads/master@{#837252}
-