GitLab

https://pdfium.googlesource.com/pdfium.git/+log/40c223e4ed41..d1ffda2acaee

$ git log 40c223e4e..d1ffda2ac --date=short --no-merges --format='%ad %ae %s'
2018-03-15 thestig Cleanup CStretchEngine.

Created with:
  roll-dep src/third_party/pdfium


The AutoRoll server is located here: https://pdfium-roll.skia.org

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=dsinclair@chromium.org

Change-Id: I7d16e2a8ed63b5fcb7295d8bcb35b68abe672218
Reviewed-on: https://chromium-review.googlesource.com/964388
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#543442}

This CL adds support for upload in the download driver, model, client
and entries. The upload logic and state machine changes in the
controller will be added in another CL.

Bug: 812327
Change-Id: Ib3b60e2ad71740a28b0aa53a08cce9b046ab6e9e
Reviewed-on: https://chromium-review.googlesource.com/919867
Commit-Queue: Shakti Sahu <shaktisahu@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Reviewed-by: Dmitry Titov <dimich@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543441}

This CL moves ReplacePlaceholders() to //content in preparation for using
ReplacePlaceholders() in manifest_parser.cc

Change-Id: Ia4fb6533bcc6f914e3e5499577edb2212583af8a
Reviewed-on: https://chromium-review.googlesource.com/924550
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543440}

- On channel error, erase the sink from |current_sinks_map|.
  crrev.com/541271 added a check in OpenChannel() to short circuit if
  a sink already exists on the map. In order for retry on error to kick
  in properly, we will need to erase the entry in OnError().
- In OnChannelOpenSucceeded, erase stale sink with the same IP. This is
  done to maintain the invariant of having at most 1 sink of a given ID
  in the map. Stale sink is defined as having the same sink ID but
  different IP endpoint than the current one.
- In OnChannelOpenFailed, when erasing the sink from
  |current_sinks_map_|, also verify the sink ID. It is possible that
  a different sink now occcupies the IP endpoint that it is trying to
  erase.

Change-Id: I12bfd5af5f664c4773e7587dc30c6709d2352021
Bug: 698940
Reviewed-on: https://chromium-review.googlesource.com/961555Reviewed-by: Bin Zhao <zhaobin@chromium.org>
Commit-Queue: Derek Cheng <imcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543439}

I'm splitting most of StackLayout into a base class in
https://chromium-review.googlesource.com/c/chromium/src/+/957875 to support
building UI prototypes on top of the tab switcher UI.

This CL cleans up the interface for child classes so that they don't have to
manually manage the list of Stacks. Instead, they can call a setTabList() method
with a list of TabLists, and StackLayoutBase will set up the Stacks accordingly.

Bug: 648314
Change-Id: I627da998baaf9d74e3fa14da3cf392097a6e0944
Reviewed-on: https://chromium-review.googlesource.com/961442Reviewed-by: Matthew Jones <mdjones@chromium.org>
Commit-Queue: Ryan Landay <rlanday@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543438}

DisplayManager's constructor is called from WindowServer's
constructor. This means DisplayManager shouldn't call to WindowServer,
else we get use of unitialized values like this.

BUG=822246
TEST=covered by tests

Change-Id: I296911502021a135276a9db49b98927efea95618
Reviewed-on: https://chromium-review.googlesource.com/963831Reviewed-by: Michael Wasserman <msw@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543437}

Selection bounds are sent from Blink to CC as part of a commit cycle so
that we can draw touch handles for it. Currently, the selection bounds
are relative to the main graphics layer of a CompositedLayerMapping. In
the case of a scroller, this will be its clip rect - rather than its
scrolling contents layer. Unfortunately, this means that scrolling on
the compositor isn't applied as part of the ToScreen transformation on
the selection bounds so scrolling wont update the selection bounds
location until another Blink commit. This went unnoticed until now
because the root layer was not considered a scroller. The page would
paint into a document-sized layer and the compositor would provide extra
scrolling layers to handle frame scrolling. In this configuration, the
ToScreen transformation from the root layer does include the scroll
offset.

Now that root layer scrolling has been turned on, frame scrolling works
much the same as other scrollers. Thus, this shortcoming is seen on
frame scrolling also.

The solution in this CL is to move the selection bounds rect to be
relative to the scrolling contents layer - if one exists. The ToScreen
transformation done in CC will correctly compensate for any scroll
offset applied in the compositor and touch selection handles stick to
the selection as its scrolled. For the CC side, see
ComputeViewportSelectionBound in layer_tree_impl.cc

Bug: 812048
Change-Id: I219ac3209d334ac91e9c13ffaaa51cbc9db57886
Reviewed-on: https://chromium-review.googlesource.com/963006Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: David Bokan <bokan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543436}

The concept is only well-defined on Linux, but the measurement was only
implemented and used on Windows.

Bug: 819289
Change-Id: Ie69e550019e29b473c8efcae8f3c8bbc390e737b
Reviewed-on: https://chromium-review.googlesource.com/956575Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Commit-Queue: Erik Chen <erikchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543435}

This was originally disabled because we still ran tests against the old
code path. That's no longer supported, so we can safely reenable the
test.

This also turns a comment into a proper TODO, since it shouldn't be left
too long.

Bug: 822124
Bug: 769126
Test: run_webview_instrumentation_test_apk -f AwContentsClientShouldInterceptRequestTest#testLoadDataWithBaseUrlTriggersShouldInterceptRequest
Change-Id: I0947560b44c565edb3be2966e121d916b003ed01
Reviewed-on: https://chromium-review.googlesource.com/963647Reviewed-by: Bo <boliu@chromium.org>
Commit-Queue: Nate Fischer <ntfschr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543434}

Also, add tests for other client hints.

Finally. use the built-in sub pipe to enable running the
cross-origin test.

Bug: 817049
Change-Id: Ib4155f50e0ffd3a0447cf250cd4018b650f3b419
Reviewed-on: https://chromium-review.googlesource.com/963403
Commit-Queue: Tarun Bansal <tbansal@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543433}

Ozone BUILD.gn files for each platform have this line in them except headless.
Now it has it too. Still no head.

It was causing double definition of some symbols when building for windows.

Bug:

Change-Id: I3455ab67848f05c0b3596a53cf6f6279bf078f0c
Reviewed-on: https://chromium-review.googlesource.com/963227Reviewed-by: Robert Kroeger <rjkroege@chromium.org>
Commit-Queue: Mustafa Çamurcu <camurcu@google.com>
Cr-Commit-Position: refs/heads/master@{#543432}

This exception was created to better understand a crash stacktrace in
crbug.com/662877, and led to fixing the original cause of the crash.
However new codepaths now trigger this exception (crbug.com/822203) thus
renaming the exception message to be more generic.

Bug: 662877,822203
Change-Id: I0c1ae488b47bab739a839073f9775c7f45d0901d
Reviewed-on: https://chromium-review.googlesource.com/964387Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Commit-Queue: Mohamed Heikal <mheikal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543431}

This reverts commit 242fe9d5.

Reason for revert: The bot is working again before the revert when in: https://uberchromegw.corp.google.com/i/chromium.mac/builders/Mac10.10%20Tests/builds/30249

Original change's description:
> Revert "Fixed first_run test failure because of the global variables"
> 
> This reverts commit 8d01fd2f.
> 
> Reason for revert: crbug.com/822316 looks like it broke net_unittests?
> 
> bug: 822316
> 
> Original change's description:
> > Fixed first_run test failure because of the global variables
> > 
> > Fixed typical problem with unit tests and global variables.
> > If unit test modifies global variables then it can affect other tests
> > which are runned in the same process. Added reseting of the global
> > variables to avoid this.
> > 
> > Change-Id: I2045700473ac074209f4d48a6cedbd3dad24ccd5
> > Reviewed-on: https://chromium-review.googlesource.com/952905
> > Commit-Queue: Alexander Yashkin <a-v-y@yandex-team.ru>
> > Reviewed-by: Carlos Pizano <cpu@chromium.org>
> > Reviewed-by: Gabriel Charette <gab@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#543345}
> 
> TBR=gab@chromium.org,cpu@chromium.org,a-v-y@yandex-team.ru,vitreb@yandex-team.ru
> 
> Change-Id: I616e5415b0e786dc5bd3075f1360b31a6452e256
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Reviewed-on: https://chromium-review.googlesource.com/964621
> Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
> Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#543423}

TBR=gab@chromium.org,cpu@chromium.org,crouleau@chromium.org,a-v-y@yandex-team.ru,vitreb@yandex-team.ru

Change-Id: I3e37c027b65d1df944d3246dd6c8314d9dc9a7ee
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/963853Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543430}

Spec is here:
https://w3c.github.io/webdriver/webdriver-spec.html#get-element-rect


Bug: chromedriver:1937
Change-Id: I6b035a9f9526fdb5b3ba29c7c0a7269086216d50
Reviewed-on: https://chromium-review.googlesource.com/963044
Commit-Queue: Jonathon Kereliuk <kereliuk@chromium.org>
Reviewed-by: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543429}

Chrome launch bug: https://crbug.com/805851
Chrome bug: https://crbug.com/705744
Design doc: https://goo.gl/Rrrc7n

Bug: 705744
Change-Id: I6d7a0d748eb35dda988bd007eb8db10068db6bc7
Reviewed-on: https://chromium-review.googlesource.com/959016
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543428}

Windows silently updated and, because we target the exact version, the
benchmarks running on these machines started to fail with "out of
capacity" errors.

The long term solution here probably has to do with removing the OS
version from the targeting criteria altogether given the fact that we
already target the exact machines, but in the short term, this should
get our bots greener.

TBR=nednguyen@google.com

Bug: 818877
Change-Id: I95ecfc8b3381eaa45c0d69d2b0caa747d1b1f9eb
Reviewed-on: https://chromium-review.googlesource.com/963146
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543427}

This CL adds SigninView to SigninFragmentBase and introduces the first
chunk of SigninView layout: SigninScrollView and a bottom button bar.
SigninScrollView implements observable ScrollView similarly to
AccountSigninConfirmationView, but without special onMeasure. Buttons
doesn't have android:text specified as SigninFragmentBase will set texts
for UI elements programmatically (similarly to
AccountSigninView.updateConsentText).
The rest of this layout will be added by subsequent CLs.

Bug: 814728
Change-Id: I3ac2c5fbdb41101a768abc70b3d1c3daa85fccb1
Reviewed-on: https://chromium-review.googlesource.com/962792Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Commit-Queue: Boris Sazonov <bsazonov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543426}

This cl is the equivalent of https://codereview.chromium.org/2854683002 fix for smooth scrolling
path,if the compositor handles the GSB and during the scrolling we have to switch to main thread
since the compositor cannot handle a GSU a GSB must be sent to the main thread since the original
GSB was handled on the compositor rather than being sent to the main thread.

touchpad-scroll-impl-to-main.html is the test that forces switch from compositor thread to main
thread in the middle of scrolling. It used to get executed with --disable-smooth-scrolling
in this cl I've moved the test to a new virtual test suite that has default scroll animation
value.

I have removed the test expectations since I have changed the test and this might fix the flake.

subpixel-accumulation.html needs scroll animation to be disabled and that's why I had to move it to
a new test suite.

Bug: 820979, 708499, 757165, 800840
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I21b1f4285897d74559f6827e6a232f007dc897b0
Reviewed-on: https://chromium-review.googlesource.com/961264
Commit-Queue: Sahel Sharifymoghaddam <sahel@chromium.org>
Reviewed-by: Timothy Dresser <tdresser@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543425}

NOTRY=true

Bug: 822276
Change-Id: I7a1781912c47be7bd34cf39e2f1b6f0610cf281a
Reviewed-on: https://chromium-review.googlesource.com/964561
Commit-Queue: Mugdha Lakhani <nator@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Reviewed-by: Anita Woodruff <awdf@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543424}

This reverts commit 8d01fd2f.

Reason for revert: crbug.com/822316 looks like it broke net_unittests?

bug: 822316

Original change's description:
> Fixed first_run test failure because of the global variables
> 
> Fixed typical problem with unit tests and global variables.
> If unit test modifies global variables then it can affect other tests
> which are runned in the same process. Added reseting of the global
> variables to avoid this.
> 
> Change-Id: I2045700473ac074209f4d48a6cedbd3dad24ccd5
> Reviewed-on: https://chromium-review.googlesource.com/952905
> Commit-Queue: Alexander Yashkin <a-v-y@yandex-team.ru>
> Reviewed-by: Carlos Pizano <cpu@chromium.org>
> Reviewed-by: Gabriel Charette <gab@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#543345}

TBR=gab@chromium.org,cpu@chromium.org,a-v-y@yandex-team.ru,vitreb@yandex-team.ru

Change-Id: I616e5415b0e786dc5bd3075f1360b31a6452e256
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/964621Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543423}

This cl adds UMA to measure input latency to show/hide launcher UI.

Bug: 821184
Test: tested on eve with added UMA.
Change-Id: I8c87684371d92e8da0a7e7e8731932f5fd61d4d1
Reviewed-on: https://chromium-review.googlesource.com/961253
Commit-Queue: Tao Wu <wutao@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543422}

Cros splitscreen: do not show black backdrop on the non-snapped side of the screen if there is only one snapped window.

Bug: 818310
Change-Id: I08ced2fcf88da0d65c01253880e7cd49c0d0913c
Reviewed-on: https://chromium-review.googlesource.com/953337Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Commit-Queue: Xiaoqian Dai <xdai@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543421}

Some of the items in the downloads menu were wrapping because the menu
wasn't wide enough. Increase the width as a short-term fix.

BUG=821949

Change-Id: If4c7445d028354b159d40182d28a263bc59438ec
Reviewed-on: https://chromium-review.googlesource.com/963678Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543420}

If DevTools are disabled for primary profile,
they shouldn't be opened for all other profiles as well.

Bug: 817943
Change-Id: I3b83abf3a43f74f0003980bfcfcf4cd256b73d8f
Reviewed-on: https://chromium-review.googlesource.com/957047
Commit-Queue: Sergey Poromov <poromov@chromium.org>
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543419}

U2fDevice objects only exist for a single request so there's no point
holding these values in the class.

Change-Id: Ia42f40e4564a4128014f397a9669f0c22675bd4b
Reviewed-on: https://chromium-review.googlesource.com/963095Reviewed-by: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Adam Langley <agl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543418}

These bots are not doing the testing themselves so they don't need to be
on a specific version of Windows.

BUG=811828
BUG=811794
BUG=811791
BUG=811823
BUG=811871
BUG=811830
BUG=812108
BUG=811867
BUG=811817
BUG=811800
BUG=811826
BUG=811833
BUG=811805
BUG=811824
BUG=811793
BUG=811868
BUG=811783
BUG=811818
BUG=811788
BUG=811784
TBR=nodir@chromium.org
NOTRY=true

Change-Id: I76d933833e7944d36052e6dca64e7c3df2425190
Reviewed-on: https://chromium-review.googlesource.com/964603
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543417}

Bug: 818893
Change-Id: I58d04134464770368736cfb513a515d0debfead5
Reviewed-on: https://chromium-review.googlesource.com/956929Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Commit-Queue: Yixin Wang <wangyix@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543416}

TBR=sky@chromium.org

Bug: 822311
Change-Id: I26c95431a3497795cf7c44ec9663bc152f0d394c
Reviewed-on: https://chromium-review.googlesource.com/963837
Commit-Queue: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543415}

This CL sets up a common font environment for all tests to run under.  In
particular:
* All gtests and layout tests start with the same font config.
* The font config is isolated from any system font settings.

These changes were necessary:
* Moved fontconfig_util_linux.* into //base so that SetUpFontconfig() could be
  called during common test setup.
* Made DejaVu Sans the fallback sans font instead of Arial (ie the system font
  matched when the font family is set to 'sans').  The gtests used to use DejaVu
  Sans for this font (usually), but the layout tests used Arial.  I sided with
  the gtests in this case because it's easier to rebaseline the layout tests
  than to fix a slew of gtests.
* Added DejaVu Sans Bold to test_fonts.  Some gtests expected a bold default
  font.
* Moved third_party/content_shell_test_fonts to third_party/test_fonts.
* Fixed gtests broken by the new config.
* Rebaselined layout tests.
* Use bundled fontconfig for ChromeOS, matching the behavior on Linux.  (Only
  for desktop ChromeOS builds.  Real ChromeOS builds will still use the system
  fontconfig.)

Additionally, building the fontconfig cache is a nontrivial task.  It can take
~600ms when done from scratch.  To fix this, fontconfig cache files are saved to
the out directory.  Fontconfig initialization takes < 1ms with this
optimization, and the initialization only needs to happen once per test suite,
not at the beginning of each test.  Finally, to prevent the 600ms from being
added to the first test suite to run, the cache files are generated as part of
the build.

The tests still pull in system fonts from msttcorefonts, but those should soon
be replaced by alternatives in third_party/test_fonts.  Once this is done, this
should fix font related test failures caused by differences between Linux
systems or system font configurations.

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_dbg_32_ng;master.tryserver.chromium.linux:linux_arm
BUG=787020,818136

Change-Id: If6bcbd35bddf86dec3f02d105d8be551b898e9b2
Reviewed-on: https://chromium-review.googlesource.com/938578Reviewed-by: Lei Zhang <thestig@chromium.org>
Reviewed-by: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: Nico Weber <thakis@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Thomas Anderson <thomasanderson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543414}

Actually somehow serializing these is going to be more difficult, but
this will at least help to not crash for tests that hit this codepath.

On of several changes towards making uploading blobs with network service work.

Tested in a separate CL because multiple unrelated bugfixes are needed to make this
all work and testable (https://chromium-review.googlesource.com/c/chromium/src/+/963008).

(see also some of the discussion in https://groups.google.com/a/chromium.org/d/msg/network-service-dev/Jby2gkdDhi8/3GURbFqRAAAJ
where the problem of serializing uploaded blobs in history was mentioned)

Bug: 821878, 777879, 761117
Change-Id: Id60791d26bc2f53c79290305cd25f7a95f247a85
Reviewed-on: https://chromium-review.googlesource.com/963015Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543413}

Since Chrome side never knows the identifier of requesting Android
package, pass the package name to Chrome side so that Chrome side can
respond correctly.

Bug: 776476
Bug: b:24572867
Change-Id: I14d29245d43d91bb43d83df0cfae23809d1bfd02
Reviewed-on: https://chromium-review.googlesource.com/954402Reviewed-by: Mattias Nissler <mnissler@chromium.org>
Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
Commit-Queue: Long Cheng <lgcheng@google.com>
Cr-Commit-Position: refs/heads/master@{#543412}

It is common for CLs to have no other changes to platform/ aside from creating
or updating a feature. platform/OWNERS review is more or less a rubber-stamp
in such cases. This expedites landing such CLs.

Change-Id: If5ae29e7969f1c9647b0310b28a689be1bf28d18
Reviewed-on: https://chromium-review.googlesource.com/909412
Commit-Queue: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Rick Byers <rbyers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543411}

Instead, use the size as calculated at compile time.

TBR=afakhry@chromium.org,stevenjb@chromium.org

Bug: none
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I33db6470385597e83c1459a7a9ef803d687568a7
Reviewed-on: https://chromium-review.googlesource.com/961125Reviewed-by: Evan Stade <estade@chromium.org>
Reviewed-by: Ahmed Fakhry <afakhry@chromium.org>
Reviewed-by: Patti <patricialor@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543410}

When sign-in or sync is not allowed by GPO policy,
the sign-in/sync promo should be hidden and
clicking the profile card should direct to
managePeople instead of syncSetup.

Bug: 801383
Change-Id: Id58ae3ffd7dbcca79a9ee42baa0dac6264726031
Reviewed-on: https://chromium-review.googlesource.com/962427
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543409}

Tests either progressed further or actually passed with
ScopedMacViewsBrowserMode.

BUG=817419
TBR=dvadym@chromium.org
Mechanical change to chrome/browser/password_manager/*

Change-Id: I9743ead8bd481e44f279f24b40d7e2f528c2948a
Reviewed-on: https://chromium-review.googlesource.com/963784Reviewed-by: Robert Liao <robliao@chromium.org>
Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543408}

BUG=817419

Change-Id: I27a6cbcd664f9de0a3f1c05cdb6b682c4ad9e64f
Reviewed-on: https://chromium-review.googlesource.com/963299Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Commit-Queue: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543407}

Remove Download.DatabaseInvalidState, it's not used and not in
histograms.xml.

Bug: 642288
Change-Id: I586701a9e20f10e483bb4b3b524eefafed5ea0c2
Reviewed-on: https://chromium-review.googlesource.com/963692Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Xing Liu <xingliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543406}

Bug: 820161
Change-Id: I5a458a374890a285daa594b3c2ca72a821d2f93a
Reviewed-on: https://chromium-review.googlesource.com/955891Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: anthonyvd <anthonyvd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543405}

TBR=mfoltz@chromium.org

Bug: 822300
Change-Id: Ie6f6ab2d24a28c7fab95764185f6e548aae23cd2
Reviewed-on: https://chromium-review.googlesource.com/964543Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543404}

Also add some helpers to allow running the gl_tests with EGL / GLES2.

BUG=584248
TEST=gl_tests --gtest_filter=*GpuOESEGLImageTest.EGLImageToTexture*

Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I5177062eb7c28b59fc2684ef7fc665271b53c12d
Reviewed-on: https://chromium-review.googlesource.com/958908Reviewed-by: Antoine Labour <piman@chromium.org>
Reviewed-by: Miguel Casas <mcasas@chromium.org>
Commit-Queue: Miguel Casas <mcasas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#543403}