- 12 Nov, 2018 40 commits
-
-
Owen Min authored
Fix ProfileWindowBrowserTest.OpenBrowserWindowForProfileWithSigninRequired on Linux The test is flaky due to race conditionin the test body. Bug: 904397, 901430 Change-Id: I80b73f507e3826fc7a707379b89b39a3e7aa68c5 Reviewed-on: https://chromium-review.googlesource.com/c/1330032Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#607260}
-
Michael Ludwig authored
Bug: 903289 Change-Id: I34dbf4c6b24c175941d95a3b8f3399d955af689d Reviewed-on: https://chromium-review.googlesource.com/c/1329901Reviewed-by:
Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#607259}
-
Camillo Bruni authored
This CL was created automatically with tools/perf/update_wpr script R=perezju@chromium.org Bug: 878390 Change-Id: Ibf6c38d09ce7bfa49e60e049f5ff240e17eade8b Reviewed-on: https://chromium-review.googlesource.com/c/1329625Reviewed-by:
Juan Antonio Navarro Pérez <perezju@chromium.org> Commit-Queue: Camillo Bruni <cbruni@chromium.org> Cr-Commit-Position: refs/heads/master@{#607258}
-
Jesse McKenna authored
This is a follow-up to https://chromium-review.googlesource.com/c/chromium/src/+/1320650 Bug: 889029 Change-Id: I4dd60a56fcab509578cdfe08c999c1e34fd72375 Reviewed-on: https://chromium-review.googlesource.com/c/1331219 Commit-Queue: Jesse McKenna <jessemckenna@google.com> Reviewed-by:
Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#607257}
-
Xida Chen authored
Right now this class takes a WTF::String, which would be hard to make that pass cross thread. This CL changes WTF::String to a std::string. Bug: 902429 Change-Id: Iaa7377a02e299e99ac8178eaf59cd228724609d6 Reviewed-on: https://chromium-review.googlesource.com/c/1331032Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#607256}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/7cdd680b3802..d608c792b594 git log 7cdd680b3802..d608c792b594 --date=short --no-merges --format='%ad %ae %s' 2018-11-12 lalitm@google.com Merge "trace_processor: track number of rss stat and mem counters without upids" Created with: gclient setdep -r src/third_party/perfetto@d608c792b594 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: Icc4c3d0dd155cf3defb9158fa884b544fbfaebe4 Reviewed-on: https://chromium-review.googlesource.com/c/1331220Reviewed-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@{#607255}
-
Alex Clarke authored
This is necessary for using SequenceManager inside ScopedTaskEnvironment. It also makes Virtual Time a bit cleaner from an interface PoV. Bug: 863341, 891670 Change-Id: Iaa8300b6c4b2f889b4cfe185959e3a280730636d Reviewed-on: https://chromium-review.googlesource.com/c/1329969 Commit-Queue: Alex Clarke <alexclarke@chromium.org> Reviewed-by:
Sami Kyöstilä <skyostil@chromium.org> Reviewed-by:
Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#607254}
-
Etienne Pierre-doray authored
base::ThreadRestrictions::ScopedAllowWait is deprecated in favor of its more explicit counterpart. It should have been replaced by : * base::ScopedAllowBaseSyncPrimitivesForTesting in test files. * base::ScopedAllowSyncPrimitives in non-test files * base::ScopedAllowSyncPrimitivesOutsideBlockingScope when it's used on threads that don't allow blocking The last one is strongly frowned upon but this CL aims to document existing behavior rather than address it. Owners are encouraged to follow-up by fixing unnecessary waits and more particularly unnecessary waits outside-blocking-scope. Note: The non-for-testing versions require friend'ing in thread_restrictions.h but care was taken to add these friends ahead of git cl split (since it wasn't possible to do a line-by-line associated CL split). Refer to the top-level CL if necessary : https://chromium-review.googlesource.com/c/chromium/src/+/1288533 Please CQ if LGTY! This CL was uploaded by git cl split. R=fdoray@chromium.org Bug: 766678 Change-Id: I49ce1451e0a57c832a2ef09d00364aba4b6a69e9 Reviewed-on: https://chromium-review.googlesource.com/c/1324397Reviewed-by:
François Doray <fdoray@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#607253}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/50bf0429394d..48f63f90f6f3 git log 50bf0429394d..48f63f90f6f3 --date=short --no-merges --format='%ad %ae %s' 2018-11-12 jiajia.qin@intel.com ES31: add row major matrix support (part 2) Created with: gclient setdep -r src/third_party/angle@48f63f90f6f3 The AutoRoll server is located here: https://autoroll.skia.org/r/angle-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: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 TBR=cwallez@chromium.org Change-Id: I9925fa75c8feb9418d8d453d4dd18a1a7825a761 Reviewed-on: https://chromium-review.googlesource.com/c/1331414Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/5ac7f8c6..85604c46 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: I6876c0b071b79ca6c6099456e9aadd263239e044 Reviewed-on: https://chromium-review.googlesource.com/c/1330027Reviewed-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@{#607251}
-
Majid Valipour authored
AnimationWorklet and ScrollTimeline heavily depend on WebAnimations API. For example to build the most basic example based on AnimationWorklet one needs to construct a KeyframeEffect. This CL will ensure that AnimationWorklet flag also enables WebAnimationsAPI runtime feature. Bug: 903953, 811922 Change-Id: Ie1f92ee2e579541675f88b762cbc30895bc66942 Reviewed-on: https://chromium-review.googlesource.com/c/1329891Reviewed-by:
Robert Flack <flackr@chromium.org> Commit-Queue: Majid Valipour <majidvp@chromium.org> Cr-Commit-Position: refs/heads/master@{#607250}
-
jonross authored
Currently LocalSurfaceId allocation time is not always accurate. This is because ParentLocalSurfaceIdAllocator generates on at creation time. This change stops that, moving the generation to each of the times where we need to first allocate an ID. A few sites will need further investigation, as they seem to be accessing the allocator ID, out of their normal allocation flow. Which would lead to sending invalid LocalSurfaceIds. These will be addressed in follow-ups, and for now just fallback to generating an ID at creation time. BUG: 655231 Change-Id: I1e6c6b49f9b593896318814e50eea21b3671fb6b Reviewed-on: https://chromium-review.googlesource.com/c/1312118Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Commit-Queue: Jonathan Ross <jonross@chromium.org> Cr-Commit-Position: refs/heads/master@{#607249}
-
edchin authored
A bug in WKWebView causes video playback to get gummed up for all future playbacks even on new WebStates until the app is killed. This fix runs the simple reduced motion animator for iOS 10 so users are able to playback video. Bug: 891851 Change-Id: I22e84f5a7e3b7aa453ba8fe4632ddc12619d86a7 Reviewed-on: https://chromium-review.googlesource.com/c/1330028Reviewed-by:
edchin <edchin@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: edchin <edchin@chromium.org> Cr-Commit-Position: refs/heads/master@{#607248}
-
Mathias Carlen authored
Before this patch, we stopped autofill assistant with an error message when the cvc step got cancelled. We are currently not able to distinguish between a bad cvc and cancel. This patch makes a first step towards a nicer flow where stop script becomes a graceful shutdown with a status message. Bug: 806868 Change-Id: I5d0e3f74d21f54df1276f994be2ae8c798fa2c0f Reviewed-on: https://chromium-review.googlesource.com/c/1331394 Commit-Queue: Mathias Carlen <mcarlen@chromium.org> Reviewed-by:
Stephane Zermatten <szermatt@chromium.org> Cr-Commit-Position: refs/heads/master@{#607247}
-
Daniel Vogelheim authored
Change the client header announcing OP capability to use the value "0". This is meant to mitigate an apparently reasonably common bug where the header value is blindly copied into the request (as observed in the referenced bug). Bug: 751996, 901477 Change-Id: I85c67cfdad3d15fc8e76e62bf1f84323faa1f790 Reviewed-on: https://chromium-review.googlesource.com/c/1328982Reviewed-by:
Mike West <mkwst@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org> Cr-Commit-Position: refs/heads/master@{#607246}
-
Xida Chen authored
Right now at PassthroughTouchEventQueue::ProcessTouchAck, we fetch an event from outstanding_touches_ queue, modify its ack info and latency info, and put the event back in the queue again. Note that the outstanding_touches_ is a std::set sorted according to each event's unique_touch_id. Setting ack info and latency info won't change the unique_touch_id on the event, in order words, it won't affect the ordering of the event in that set. So it is un-necessary to remove the event and put it back in the queue again. This CL does clean up. It sets ack and latency info directly on the event in the outstanding_touches_. The only catch here is that the iter of a std::set is always const, so we have to do a const_cast. Bug: None Change-Id: I45cf1001ce82b3fd876b856618c99df25d5b9b4a Reviewed-on: https://chromium-review.googlesource.com/c/1297714Reviewed-by:
Navid Zolghadr <nzolghadr@chromium.org> Commit-Queue: Xida Chen <xidachen@chromium.org> Cr-Commit-Position: refs/heads/master@{#607245}
-
Camillo Bruni authored
This CL was created automatically with tools/perf/update_wpr script R=perezju@chromium.org Bug: 878390 Change-Id: Ic159261bb0296e11d58bd7adb98b6a5a8fb5cece Reviewed-on: https://chromium-review.googlesource.com/c/1329747Reviewed-by:
-
Camillo Bruni authored
This CL was created automatically with tools/perf/update_wpr script R=perezju@chromium.org Bug: 878390 Change-Id: I753f512b660e613d8e6f622f81ef2aa3600d4310 Reviewed-on: https://chromium-review.googlesource.com/c/1329631Reviewed-by:
-
Nicolas Ouellet-Payeur authored
The test runs an 'echo' command to a temporary file when the user visits example.com, and then checks that the temporary file exists and contains the right string. Bug: 884211 Change-Id: Ifa8a129aa8e98583a6060274bb85a8bc4d235b8d Reviewed-on: https://chromium-review.googlesource.com/c/1318057Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org> Cr-Commit-Position: refs/heads/master@{#607242}
-
Owen Min authored
DeveloperToolsDisabled policy is marked as deprecated in policy_templates and DeveloperToolsPolicyHandler. Remove the one in handler so that the deprecation message only appears once. Bug: 902603 Change-Id: Ia9a78cd4667eb39b5ec3e82399fcd44ae1556de1 Reviewed-on: https://chromium-review.googlesource.com/c/1326843Reviewed-by:
Pavol Marko <pmarko@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#607241}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/c7a34cae21d2..0243949545eb git log c7a34cae21d2..0243949545eb --date=short --no-merges --format='%ad %ae %s' 2018-11-12 reed@google.com Revert "route getFontMetrics to SkFont" 2018-11-12 reed@google.com add getwidths to SkFont 2018-11-11 reed@google.com impl SkFont::measureText 2018-11-11 reed@google.com route getFontMetrics to SkFont 2018-11-11 skia-recreate-skps@skia-swarming-bots.iam.gserviceaccount.com Update SKP version 2018-11-11 mtklein@google.com work around Flutter/Fuchsia GN problems 2018-11-11 mtklein@google.com Reland "Improve degenerate 2pt conical gradient cases" 2018-11-10 fmalita@chromium.org [skottie] Make animated image support generally available 2018-11-10 reed@google.com move GlyphCacheProc into fontpriv 2018-11-10 reed@google.com add fontmetrics to SkFont 2018-11-10 kjlubick@google.com Convert skottie's cloud build to use CanvasKit Created with: gclient setdep -r src/third_party/skia@0243949545eb The AutoRoll server is located here: https://autoroll.skia.org/r/skia-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-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel TBR=rmistry@chromium.org Change-Id: I12e66c7ffa4160419dbf559753c4758ac4b62fb5 Reviewed-on: https://chromium-review.googlesource.com/c/1331410Reviewed-by:
-
Etienne Pierre-doray authored
base::ThreadRestrictions::ScopedAllowWait is deprecated in favor of its more explicit counterpart. It should have been replaced by : * base::ScopedAllowBaseSyncPrimitivesForTesting in test files. * base::ScopedAllowBaseSyncPrimitives in non-test files * base::ScopedAllowBaseSyncPrimitivesOutsideBlockingScope when it's used on threads that don't allow blocking The last one is strongly frowned upon but this CL aims to document existing behavior rather than address it. Owners are encouraged to follow-up by fixing unnecessary waits and more particularly unnecessary waits outside-blocking-scope. Note: The non-for-testing versions require friend'ing in thread_restrictions.h but care was taken to add these friends ahead of git cl split (since it wasn't possible to do a line-by-line associated CL split). Refer to the top-level CL if necessary : https://chromium-review.googlesource.com/c/chromium/src/+/1288533 Please CQ if LGTY! This CL was uploaded by git cl split. R=boliu@chromium.org Bug: 766678 Change-Id: Id041725a059110494a5805f98f3eba5b5bab6bff Reviewed-on: https://chromium-review.googlesource.com/c/1324453Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
-
Philip Rogers authored
When a canvas is accelerated, the background is sometimes painted into the content layer (see ContentLayerSupportsDirectBackgroundComposition). When this occurs, we can skip painting the background but still need to paint the hit test rects. Bug: 903480 Change-Id: Iec5d564ffed9611011d78adc381c9d7d7153e960 Reviewed-on: https://chromium-review.googlesource.com/c/1329854Reviewed-by:
Xianda Sun <sunxd@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#607238}
-
Shawn Gallea authored
Created FullscreenShellSurface class, similar to ShellSurface but without any ash dependencies, to be used with fullscreen wayland protocol to allow EXO use on other platforms. Added unit test for FullscreenShellSurface for cast. Bug: 896710 Test: Compiled on ChromeOS Change-Id: I542c9b0729b847f4cde2fa67d727143e18f4a2a5 Reviewed-on: https://chromium-review.googlesource.com/c/1298172 Commit-Queue: Shawn Gallea <sagallea@google.com> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#607237}
-
Philip Rogers authored
There was an early-out in ReplacedPainter::Paint that would skip painting touch-action hit test rects if the replaced element did not also paint a background. This patch skips the early-out if there is a hit test rect that should be painted. This does not fully fix https://crbug.com/903480 because accelerated canvases still do not generate the correct touch action rects. Bug: 903480 Change-Id: If6cda7339c5a20a04fe669c4d54fae30e83debf2 Reviewed-on: https://chromium-review.googlesource.com/c/1331033Reviewed-by:
-
Shawn Gallea authored
Moved shell and aura shell checks to when not using fullscreen shell. This is needed for cast to run fullscreen clients. Bug: 896710 Test: Compiled client on ChromeOS. Change-Id: Ie4c408d8266a0957ad9b57ea332a8e38171b2be9 Reviewed-on: https://chromium-review.googlesource.com/c/1331170Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/279259d2591e..4e5366120355 Created with: gclient setdep -r src-internal@4e5366120355 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: I1942c7d861d9619559010043f5a53141152bc9bf Reviewed-on: https://chromium-review.googlesource.com/c/1331412Reviewed-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@{#607234}
-
Mustaq Ahmed authored
|RenderFrameImpl::CommitNavigation*| functions got |WebScopedUserGesture|s few years ago to fix a PasswordManager test (crrev.com/2378393002). Those artificial user gesture tokens seem to have become redundant in the meantime through changes in navigation and/or PasswordManager. This CL removes those tokens because they are causing popup-blocker failure with UAv2. Bug: 865243 Change-Id: I6ed7d73a5784f82ce3ba695067a4cbff0a9c24b4 Reviewed-on: https://chromium-review.googlesource.com/c/1323626 Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#607233}
-
Daniel Vogelheim authored
HTMLScriptElement::text() and setText() functions modified to accept TrustedScript as an argument. Added HTMLScriptElement::innerText and HTMLScriptElement:textContent attributes to override existing implementations in HTMLElement and Node, and to accept TrustedScript. Changed HTMLElement::innerText and Node::textContent .idl definitions and adjusted setters and getters to accept TrustedScript. Bug: 739170 Change-Id: I63da5714ffac328c6ae2f76e5da58c05f44a1cbf Reviewed-on: https://chromium-review.googlesource.com/c/1178046 Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Cr-Commit-Position: refs/heads/master@{#607232}
-
Peter Wen authored
Original CL: https://crrev.com/c/1327157 Fixes: - Cronet target: cronet_sample_apk_java__lint - Removes suppression for autofill TBR=mheikal@chromium.org Bug: 900912 Change-Id: Ib3c7fd69e394ade0b2b80aed72a993e02b0a153c Reviewed-on: https://chromium-review.googlesource.com/c/1331011 Commit-Queue: Peter Wen <wnwen@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#607231}
-
Leonard Grey authored
Despite the expectation that capture has been turned off when we enter a move loop, traces show that when the mystery move loop crash happens, capture is indeed on. Let's see if: - Releasing capture actually works - Whether we're succesfully releasing capture on the widget entering the move loop, but still have capture on another widget. Bug: 876493 Change-Id: I22db7aca92d25df9e97fd57f9b005b45770191b5 Reviewed-on: https://chromium-review.googlesource.com/c/1329903Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#607230}
-
Jeremy Roman authored
In the intervening almost-nine years, we have managed to add this #include. Change-Id: I68673f7c0f6e83fd604e8d227062acd179a18a9e Reviewed-on: https://chromium-review.googlesource.com/c/1330210Reviewed-by:
-
Egor Pasko authored
The orderfile bot currently produces a warning like this: WARNING:root:2540 symbols repeated with multiple offsets: __ThumbV7PILongThunk__lala 26256476 38958560 __ThumbV7PILongThunk__tada 26232216 38911480 ... // 10 examples, to avoid expanding the log a lot It is expected that multi-jump thunks would be duplicated, and we suspect they would have the same symbol name. This change makes the warning to ignore such cases in order to leave space for more suspicious cases. Bug: 893981 Change-Id: I9e1ad0f1e0786c1d12d7490c7a597c7d9344b8a6 Reviewed-on: https://chromium-review.googlesource.com/c/1331395Reviewed-by:
Matthew Cary <mattcary@chromium.org> Commit-Queue: Egor Pasko <pasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#607228}
-
Takuto Ikuta authored
This is to reduce the number of files sent to swarming under third_party/perl directory. e.g. https://isolateserver.appspot.com/browse?namespace=default-gzip&hash=91c35f024b8949020a3a25f3421f27689f35dff5 from https://chromium-swarm.appspot.com/task?id=40da578fadf59310&refresh=10&show_raw=1 Bug: 899991 Change-Id: I8608283399e817c8d3d8558782d67587a735e96a Reviewed-on: https://chromium-review.googlesource.com/c/1331097Reviewed-by:
Marc-Antoine Ruel <maruel@chromium.org> Commit-Queue: Takuto Ikuta <tikuta@chromium.org> Cr-Commit-Position: refs/heads/master@{#607227}
-
Mike Reed authored
Strictly an alias/name change. No functional changes. TBR= Change-Id: I81e1482b91c23feaa7c24b6a0ca241988c01538b Reviewed-on: https://chromium-review.googlesource.com/c/1331134 Commit-Queue: Mike Reed <reed@google.com> Reviewed-by:
-
John Budorick authored
Bug: 817842 Change-Id: I1d360830b8e2a7c72dbca5083f9d803d4a229917 Reviewed-on: https://chromium-review.googlesource.com/c/1331036Reviewed-by:
-
Leonard Grey authored
This is a reland of 002dfaef Original change's description: > Mac: Disallow dragging tab into blocked tabstrip > > This is a really longwinded way for me to fix some NOTIMPLEMENTED spam, > though I'm pretty sure the thing that happens right now when you open > a certificate prompt and drag a tab into the parent window is bad. > > This introduces a new method because we're not sure if it matches the > Aura semantics, and this is better than the nothing we were doing previously. > > Bug: 825834 > Change-Id: I316ca4fbaa7a27dacb09bce0866203b79567016d > Reviewed-on: https://chromium-review.googlesource.com/c/1315956 > Commit-Queue: Leonard Grey <lgrey@chromium.org> > Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org> > Cr-Commit-Position: refs/heads/master@{#606619} Bug: 825834 Change-Id: Ice8fa3f991e3e672a2266c444e281bbad1b47819 Reviewed-on: https://chromium-review.googlesource.com/c/1329350Reviewed-by:
-
Fredrik Söderquist authored
This if cascade consumes both a fair bit of footprint while it makes the switch slightly less readable. Change-Id: Ia8cb0d14ec831ccfcf7f22154a12f16d44fbc9ad Reviewed-on: https://chromium-review.googlesource.com/c/1329147 Commit-Queue: Fredrik Söderquist <fs@opera.com> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#607223}
-
Boris Sazonov authored
This CL removes passwords checkbox enabled state dependency on ProfileSyncService::IsCryptographerReady(). This datatype can be configured without cryptographer being ready (that's what other platforms do) and removing this dependency fixes issues when this checkbox was spontaneously becoming disabled. Bug: 902880 Change-Id: I58f8b1b0375bc2bdfb77c1db1cf3f998020b935d Reviewed-on: https://chromium-review.googlesource.com/c/1323720Reviewed-by:
Marc Treib <treib@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: Boris Sazonov <bsazonov@chromium.org> Cr-Commit-Position: refs/heads/master@{#607222}
-
Boris Sazonov authored
This CL modifies SigninHelper.validateAccountSettings so that accounts are accessed only after AccountManagerFacade cache is populated. SigninHelper.validateAccountSettings is called early during startup, so postponing this call should reduce the amount of time spent waiting for accounts on UI thread. Bug: 837374 Change-Id: Ica027806d9a329e08b880abec12dc0954c540e63 Reviewed-on: https://chromium-review.googlesource.com/c/1329247Reviewed-by:
-
