- 20 Apr, 2019 24 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/5cc5a8e61949..d8ce95982c94 git log 5cc5a8e61949..d8ce95982c94 --date=short --no-merges --format='%ad %ae %s' 2019-04-19 benjhayden@chromium.org Migrate alerts-table to es6 module Created with: gclient setdep -r src/third_party/catapult@d8ce95982c94 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:950011 TBR=charliea@chromium.org Change-Id: I378e1838dd38f166a8b72840fc3b02f033c946d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575291Reviewed-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@{#652723}
-
Yuwei Huang authored
We will introduce a new heartbeat sender that connects to the new directory server. This CL renames the old HeartbeatSender to XmppHeartbeatSender to prevent confusion. Bug: 954695 Change-Id: Ie29caedb55de0db2dd3d2e8801971c9ea015b164 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574653 Commit-Queue: Yuwei Huang <yuweih@chromium.org> Commit-Queue: Joe Downing <joedow@chromium.org> Reviewed-by:
Joe Downing <joedow@chromium.org> Cr-Commit-Position: refs/heads/master@{#652722}
-
Michael Thiessen authored
No functionality changes intended here, this should just be a no-op refactor. This refactor is to make it easier to land followup changes for spatial navigation here: https://chromium-review.googlesource.com/c/chromium/src/+/1562714/ Bug: 938534 Change-Id: Ia3c2b993db9e2055f7c42f209c3806dd08c34daf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1567081 Commit-Queue: Michael Thiessen <mthiesse@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#652721}
-
Martin Kreichgauer authored
There is a specialization in SetPinRequestHandler, but nothing ever calls ProvidePIN on the base class. Bug: 870892 Change-Id: I86564915b1ae2a1d0fe7996a5e23ec8e5d10ad39 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574643 Auto-Submit: Martin Kreichgauer <martinkr@google.com> Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Adam Langley <agl@chromium.org> Cr-Commit-Position: refs/heads/master@{#652720}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/0ab4b3dfabf6..34a4af2494cc git log 0ab4b3dfabf6..34a4af2494cc --date=short --no-merges --format='%ad %ae %s' 2019-04-20 derat@chromium.org chromeos_config: Run Tast tests on VM PFQ builders. Created with: gclient setdep -r src/third_party/chromite@34a4af2494cc The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@chromium.org Change-Id: Ifae5ebe7da41fe53a0197fa56d80d2cfcb6eb798 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575960Reviewed-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@{#652719}
-
Harry Cutts authored
The copies in `EventConverterEvdevImplTest` and `InputInjectorEvdevTest` were exact duplicates, and I'm working on a CL that also needs it. Also renamed it to `FakeCursorDelegateEvdev`, since it doesn't log calls to its methods and so isn't really a mock. No behaviour changes should be caused by this CL. Bug: none Test: run `EventConverterEvdevImplTest` and `InputInjectorEvdevTest`. Change-Id: Iad54935e675bb2b253dcf6aa5c2bb0d9d9cae1be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1559207Reviewed-by:
Michael Spang <spang@chromium.org> Commit-Queue: Harry Cutts <hcutts@chromium.org> Cr-Commit-Position: refs/heads/master@{#652718}
-
Vidal Diazleal authored
Tab titles in the devtools are not localizable, this PR makes the title function to actually use l10n tagged templates and fixes the following comment on the code: third_party\blink\renderer\devtools\front_end\Runtime.js // FIXME: should be Common.UIString() but runtime is not l10n aware yet. Change-Id: I4a651e2f1b4ef0e0e6d07b54d52e44c17aeda057 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1554648 Commit-Queue: Vidal Diazleal <vidorteg@microsoft.com> Reviewed-by:
Joel Einbinder <einbinder@chromium.org> Cr-Commit-Position: refs/heads/master@{#652717}
-
Hwanseung Lee authored
Use enum class instead of enum at MenuController enum class is more type safety. Bug: 940736 Change-Id: I9302a3d7cc5c4d4c8ffdfdbf758141ad34a417b1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1569370Reviewed-by:
Mitsuru Oshima <oshima@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Commit-Queue: Hwanseung Lee <hs1217.lee@samsung.com> Cr-Commit-Position: refs/heads/master@{#652716}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/990de7c7fcb5..80140adf0944 Created with: gclient setdep -r src-internal@80140adf0944 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. CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=fdoray@chromium.org,bsep@chromium.org,keishi@chromium.org,chanli@chromium.org Change-Id: Icaf2cdb67c9fde554d39c84c8ae9524cb4e0cd5b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576009Reviewed-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@{#652715}
-
Dan Beam authored
Note: though I'm adding a new module, the additions to npm_exclude.txt shrink npm_modules.tar.gz by 200KB+ (4436810 -> 4216240). Bug: 954683 Change-Id: Ibbab9a954c4aa45f10fa5a3c3a1f3e694747d1ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1552415 Commit-Queue: Dan Beam <dbeam@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#652714}
-
John Rummell authored
If the per-device provisioning permission request happens multiple times within a 15 minute window, use the same response as the user choose before if the origin matches. This avoids problems where the user is prompted multiple times for the same site. BUG=950200 TEST=tested manually Change-Id: I8fcb73ebeed5e7ab60b38ab6b7213889f991bbb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1559491Reviewed-by:
Xiaohan Wang <xhwang@chromium.org> Commit-Queue: John Rummell <jrummell@chromium.org> Cr-Commit-Position: refs/heads/master@{#652713}
-
Peter Kasting authored
* Shorten code * Use helpers for repeated code * Fix case where we checked one child's classname, then cast a different child * Use base::size() * Use std::make_unique() * Use base::nullopt * Use View::Views * constexpr kConstant * Make GridLayoutTest.FixedSize consistent about loop nesting order * Make ViewTest.ChildViewZOrderChanged consistent about checking children, then layers Bug: none Change-Id: I871af00669e629176aa8fd2cc4cac2736629b3f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575166 Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#652712}
-
Tom Anderson authored
Real failures: chrome/browser/notifications/scheduler/impression_history_tracker.cc <= should be <. This could cause std::sort() to not work. False positives: chrome/browser/ui/cocoa/applescript/apple_event_util_unittest.mm chrome/utility/safe_browsing/mac/dmg_analyzer.cc components/viz/service/display/skia_renderer.cc: storage/browser/fileapi/obfuscated_file_util_memory_delegate.cc Taking the address of the first element of an empty vector. This would cause problems if we actually tried to dereference the pointer, but in all cases it is paired with a length variable that's 0. services/device/hid/hid_connection_impl.cc Taking the address of one-past the last element of a vector. This would cause problems if we actually tried to dereference the pointer, but it's just used as an end iterator. BUG=923166 TBR=sky Change-Id: I19a9ef1910fb5e26925cd957d7f0edaaaa4d8c1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575727 Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Thomas Anderson <thomasanderson@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#652711}
-
Tim Song authored
The top notification may be clipped even though there is sufficient space to render the message center taller. BUG=954395 Change-Id: I5aade1b772d80640a2094bcb42365a75f75db837 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574956 Commit-Queue: Tim Song <tengs@chromium.org> Reviewed-by:
Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#652710}
-
Eric Karl authored
Ensures that Android WebView works with SurfaceSynchronization enabled. For WebView, SurfaceSynchronization is nearly a no-op, as only one surface is ever in use. This change only does two real things: - Ensures that WebView uses RenderFrameMetadata rather than CompositorFrameMetadata where appropriate. - Ensures that SurfaceIds are correctly sent to the renderer for WebView, allowing for rendering to occur. Also removes previous disabling of WebView in tests. Bug: 881469 Change-Id: If01ebc3be54eb90ca3aa196a464de693ad69bed5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1570136Reviewed-by:
Stefan Zager <szager@chromium.org> Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Reviewed-by:
Saman Sami <samans@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#652709}
-
Hiroshige Hayashizaki authored
This CL adds the following methods that are used in upgrade-insecure-requests processing to FetchClientSettingsObject, and use them in FrameLoader::UpgradeInsecureRequest(): - GetInsecureRequestsPolicy() - GetUpgradeInsecureNavigationsSet() - GetMixedAutoUpgradeOptOut() Before this CL, upgrade-insecure-requests works based on ExecutionContext's SecurityContext, and thus uses the insideSettings' CSP list during off-the-main-thread worker script fetch. After this CL, upgrade-insecure-requests works via FetchClientSettingsObject and thus uses the outside settings object's CSP list correctly in off-the-main-thread worker script fetch. This CL doesn't cause behavior changes in dedicated workers because dedicated workers inherits the parent's CSP list. This CL probably affect behaviors in off-the-main-thread fetch for shared and service workers, but these are not covered by tests. Anyway OTMT for shared and service workers is still under development and will be tested and fixed in the future. Bug: 878274 Change-Id: I412845c9c7a866d22b97c501936d640fab99b71f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1570669 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Carlos IL <carlosil@chromium.org> Reviewed-by:
Hiroki Nakagawa <nhiroki@chromium.org> Cr-Commit-Position: refs/heads/master@{#652708}
-
David Benjamin authored
Bug: none Change-Id: I9c4a35aa6797230840f3436cbf66f35904dbf5cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576010 Commit-Queue: Matt Mueller <mattm@chromium.org> Reviewed-by:
Matt Mueller <mattm@chromium.org> Auto-Submit: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#652707}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/f777fd10f59a..5cc5a8e61949 git log f777fd10f59a..5cc5a8e61949 --date=short --no-merges --format='%ad %ae %s' 2019-04-19 benjhayden@chromium.org Migrate scalar-span to es6 module 2019-04-19 benjhayden@chromium.org Migrate raised-button to es6 module. Created with: gclient setdep -r src/third_party/catapult@5cc5a8e61949 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:950011,chromium:950011 TBR=charliea@chromium.org Change-Id: Idee0eb527cd3f1e3af0a5dd12fce64fafd0b3d30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576139Reviewed-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@{#652706}
-
Bret Sepulveda authored
Attempting to deflake Windows 10 bot. The test virtual/bidi-caret-affinity/editing/selection/modify_move/move-by-word-visually-crash-test-2.html is passing on some platforms and crashing on others. This patch adds Pass to the test's expectation. TBR=xiaochengh@chromium.org Bug: 894651, 954586 Change-Id: Ic0b0068269016cbd3de611f2059cf5364ff918e5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576159Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#652705}
-
Marcin Wiącek authored
Before: file contains many similar methods and variable definitions. After: there are created tables for integer and boolean values indexed by @IntDef, which are handled by two methods. It makes code and APK shorter. Change-Id: I549b288f53a7644601179b1477e8a05d2e9b2de7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1565311 Commit-Queue: Marcin Wiącek <marcin@mwiacek.com> Reviewed-by:
Donn Denman <donnd@chromium.org> Cr-Commit-Position: refs/heads/master@{#652704}
-
Wez authored
Exiting the Runner when it is not hosting any components saves memory but increases the time taken to launch new components, since the web.Context must be re-created from scratch each time. Fuchsia's Components v2 APIs will allow the system to teardown Runners to save resources when required, but for now just run until the system kills the Runner process/job. Bug: 952560 Change-Id: I5fdd220fbb1e0587d584f14a5447948e1d1d99bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1566766 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#652703}
-
Harry Cutts authored
This was messing up the formatting on the rest of the line. Change-Id: I37263e8c7d2917b8b3bf620209282bfb576eb022 Bug: none Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576014 Auto-Submit: Harry Cutts <hcutts@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Commit-Queue: Harry Cutts <hcutts@chromium.org> Cr-Commit-Position: refs/heads/master@{#652702}
-
Kevin Marshall authored
Allows the Agent to provide the Runner with the touch support policy for Cast applications. Bug: 953796 Change-Id: I3bd383e3df196c8217e5a0eb7da37fee6f6d83f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1572246Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Kevin Marshall <kmarshall@chromium.org> Cr-Commit-Position: refs/heads/master@{#652701}
-
Chris Lu authored
When calling CallJavaScriptFunction, the timeout being creating is expecting a value in seconds, but the constant is in milliseconds. Bug: 919685 Change-Id: Iae35a9fdbccaabf1c2c53ec650b75042bfb0cf87 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575998Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Reviewed-by:
Mike Dougherty <michaeldo@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Commit-Queue: Chris Lu <thegreenfrog@chromium.org> Cr-Commit-Position: refs/heads/master@{#652700}
-
- 19 Apr, 2019 16 commits
-
-
Bret Sepulveda authored
The test virtual/nobinary-for-devtools/http/tests/devtools/elements/highlight/highlight-display-locked.js does not exist, which is causing lint for the TestExpectations file to fail. This patch deletes that line. TBR=rakina@chromium.org Bug: 934458 Change-Id: I40ed3ddb425e462d89b0bf2647cc1c54dd928118 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576219Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#652699}
-
Regan Hsu authored
This CL removes code Autofill section from OS-settings entirely, since it is not used by the OS. Bug: 950007 Change-Id: Ia59926a8861b329c5ab678e296bd91a6fda8a0ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574295Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Regan Hsu <hsuregan@chromium.org> Cr-Commit-Position: refs/heads/master@{#652698}
-
Reilly Grant authored
This change finishes wiring up the CONTENT_SETTINGS_TYPES_SERIAL_GUARD permission which can be used to entirely block the chooser UI shown by navigator.serial.requestPort(). As with the USB guard permission this needs special casing in the page info and site settings UIs because it can't be set to "allow," only "ask" or "block." Bug: 908836 Change-Id: I085c9e60e8a3cd6fa85ca0b18a58f359ef35e7f3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538681Reviewed-by:
Avi Drissman <avi@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Commit-Queue: Reilly Grant <reillyg@chromium.org> Cr-Commit-Position: refs/heads/master@{#652697}
-
rajendrant authored
Bug: 936999 Change-Id: Ib67c739cfc862c942489fe9213abb7db410b100c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576119Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: rajendrant <rajendrant@chromium.org> Cr-Commit-Position: refs/heads/master@{#652696}
-
Chris Blume authored
GLSurface's ctor does not have special logic, but isn't marked '= default'. It could gain std::is_trivially_constructible if it was marked as such. This CL replaces the empty ctor function body with '= default'. TBR=ericrk@chromium.org Change-Id: I2003ecb142ca8d5d61173fb6fc2cd754db9cf8fa Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576023Reviewed-by:
Chris Blume <cblume@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Chris Blume <cblume@chromium.org> Commit-Queue: Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#652695}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/5609aa373c8e..0ab4b3dfabf6 git log 5609aa373c8e..0ab4b3dfabf6 --date=short --no-merges --format='%ad %ae %s' 2019-04-19 dhanyaganesh@chromium.org Revert "pre-cq-default-configs: disable betty-arcnext" 2019-04-19 jclinton@chromium.org Revert "ManifestCheckout: enable FetchAll for all scenarios" 2019-04-19 dhanyaganesh@chromium.org builder_status_lib: fix a confusing build message 2019-04-19 zamorzaev@chromium.org lib: Use a subdirectory of /tmp 2019-04-19 nednguyen@google.com [cbuildbot]: only run CPEExport stage in the release builder 2019-04-19 saklein@chromium.org controller/artifacts: Updated todos to reference bugs. 2019-04-19 jrosenth@chromium.org chromeos-config: make gale, whirlwind important 2019-04-19 haddowk@chromium.org chromeos_config: make moblab-generic-vm important again. Created with: gclient setdep -r src/third_party/chromite@0ab4b3dfabf6 The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@chromium.org Change-Id: Iee12f0b87fa0e06562d8a2df5ef8c25494fc0ee9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1576065Reviewed-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@{#652694}
-
David Bienvenu authored
When Chrome is launched with a URL on the command line, and Chrome is already running, it looks for a browser window to load the URL in. This CL makes it ignore browser windows that are not on the current desktop, on Windows 10 (or higher). Bug: 757853 Change-Id: Ib251619cbcef18809b886b96e21034507ea14654 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538834 Commit-Queue: David Bienvenu <davidbienvenu@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#652693}
-
Alex Newcomer authored
The check can only be hit if item_count() is 1, but the to/from indices are not equal. If item_count() is 1, we shouldn't be doing a move anyways, so return early. This crash is likely occurring because a sync is coming in to delete the second item in a 2-item folder during the drag reorder. Bug: 937431 Change-Id: I2521f2e89ace9ca8c9218bfd050c04cc6bb89d2d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575776 Auto-Submit: Alex Newcomer <newcomer@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#652692}
-
David Black authored
Previously, if a text query were initiated via deeplink and the UI was visible but not in main/embedded UI mode, the interaction would not be visible to the user. Bug: b:113278359 Change-Id: I1f65e0955df688cc08f1b511d09d6bf99a9a55a2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1566725 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Tao Wu <wutao@chromium.org> Cr-Commit-Position: refs/heads/master@{#652691}
-
Bret Sepulveda authored
external/wpt/payment-request/payment-request-canmakepayment-method.https.html is timeout-flaky on Windows. TBR=robertma@chromium.org Bug: 954628, 954586 Change-Id: I5061ee3afd90cfe4b82f7d84f4e48bbac23e5f2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574454Reviewed-by:
Bret Sepulveda <bsep@chromium.org> Commit-Queue: Bret Sepulveda <bsep@chromium.org> Cr-Commit-Position: refs/heads/master@{#652690}
-
Klaus Weidner authored
This change is needed to match the API change in https://chromium-review.googlesource.com/c/chromium/src/+/1570217 but was missing in that CL. TBR=bajones@chromium.org Change-Id: Ic49436558c3cde7e42b9b8a4bd26c416f0b80f2b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574457 Commit-Queue: Klaus Weidner <klausw@chromium.org> Reviewed-by:
Klaus Weidner <klausw@chromium.org> Auto-Submit: Klaus Weidner <klausw@chromium.org> Cr-Commit-Position: refs/heads/master@{#652689}
-
John Budorick authored
This should make resource_sizes output files in a format and structure amenable to process_perf_results. Bug: 807273,861983 Change-Id: I5d359b35ad4ba836cd1282f804daf996fbbec7e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574455 Commit-Queue: John Budorick <jbudorick@chromium.org> Commit-Queue: Ben Pastene <bpastene@chromium.org> Auto-Submit: John Budorick <jbudorick@chromium.org> Reviewed-by:
Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#652688}
-
Krishna Govind authored
TBR=meacer@chromium.org Change-Id: Id7fcc09482854f18a722bc0e8e951fb96846b4ec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1574447 Commit-Queue: Krishna Govind <govind@chromium.org> Reviewed-by:
Krishna Govind <govind@chromium.org> Cr-Commit-Position: refs/heads/master@{#652687}
-
Eric Roman authored
Change-Id: I7d9ca04bc1c82a0a7341bd76dd265653cb0fba55 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575162Reviewed-by:
Ryan Hamilton <rch@chromium.org> Commit-Queue: Eric Roman <eroman@chromium.org> Cr-Commit-Position: refs/heads/master@{#652686}
-
David Benjamin authored
We can just tell //base to act case-insensitively. Change-Id: I5d4a9ecfa09cd7fada0b0af792d0f414de888904 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575172 Auto-Submit: David Benjamin <davidben@chromium.org> Reviewed-by:
Eric Roman <eroman@chromium.org> Commit-Queue: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#652685}
-
Misha Efimov authored
Bug: 954372 Change-Id: Id148ead661409576491fec9b37226e975098ef76 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1575115Reviewed-by:
Wez <wez@chromium.org> Commit-Queue: Misha Efimov <mef@chromium.org> Auto-Submit: Misha Efimov <mef@chromium.org> Cr-Commit-Position: refs/heads/master@{#652684}
-