Skip to content

GitLab

T
tangled
Switch branch/tag
  1. 28 Jul, 2018 13 commits
  3. 27 Jul, 2018 27 commits
    • Jordy Greenblatt's avatar
      [CrOS MultiDevice] Feature toggle element for controlling prefs · c8df356d
      Jordy Greenblatt authored 
      Toggle button that controls a given pref.

The element uses the SettingsBooleanControlBehavior, which causes its
behavior to parallel that of settings-toggle-button. However, the
settings-toggle-button has a few disadvantages for the MultiDevice Settings
UI use case that made it expedient to create this new element to use
instead:

(1) The settings-toggle-button is designed as an entire row (as opposed to
just a toggle) and it does not allow a button to replace the feature toggle
in the DOM. Because the type of user input varies depending on the state of
the user's GAIA account, this UI would have either have to completely detach
and reattach its items when they switched input types or else it would have
to change the settings-toggle-button to allow it to detach or at least hide
its toggle.
(2) In order to match the pattern of the network-summary-item used for the
Instant Tethering feature our items have SVG icons, which would require
further refactoring of the settings-toggle-button to accommodate.
(3) We need toggles to control the unified MultiDevice feature suite and
toggles for individual features. The latter case requires more explanation
and as a result has some identifying features that do not necessarily
apprear in the former case.


Bug: 824568
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I986382a8aac31781e3d27c1f44397ddb048761f4
Reviewed-on: https://chromium-review.googlesource.com/1149001
Commit-Queue: Jordy Greenblatt <jordynass@chromium.org>
Reviewed-by: default avatarHector Carmona <hcarmona@chromium.org>
Reviewed-by: default avatarKyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578869}
      c8df356d
    • Daniel Murphy's avatar
      Revert "[DOMStorage] Moving cached namespaces to std::map" · 198b861b
      Daniel Murphy authored 
      This reverts commit 080fe17e.

Reason for revert: The corruption is still there, but now failing on Windows.

Bug: 865393

Original change's description:
> [DOMStorage] Moving cached namespaces to std::map
> 
> There has been a bunch of memory-corruption-smelling crashes around this
> area. Changing the container to a std::map should give us more
> information about where this is happening, and maybe cause it to crash
> somewhere else.
> 
> R=​mek@chromium.org
> 
> Bug: 845400,821371,857464
> Change-Id: Ife8cb10336a4f0752db89e2bca836ed873f7c711
> Reviewed-on: https://chromium-review.googlesource.com/1129849
> Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
> Commit-Queue: Daniel Murphy <dmurph@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#573452}

TBR=dmurph@chromium.org,mek@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 845400, 821371, 857464
Change-Id: Ie120acf347713d9c6172f26082ed9b18c3cc38bc
Reviewed-on: https://chromium-review.googlesource.com/1153987
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
Reviewed-by: default avatarDaniel Murphy <dmurph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578868}
      198b861b
    • Lu Chen's avatar
      Record timing information of anchor element clicks. · bec13327
      Lu Chen authored 
      Record duration between page load and first click, and intervals
between consecutive clicks. This timing information can help us
analyze the available opportunities to perform preload (and other
pre* mechanisms).

Bug: 850624
Change-Id: I8924d6409ce5695e0d9eb56e9be87b7bf5bfc5c3
Reviewed-on: https://chromium-review.googlesource.com/1149415
Commit-Queue: Lu Chen <chelu@chromium.org>
Reviewed-by: default avatarSteven Holte <holte@chromium.org>
Reviewed-by: default avatarTarun Bansal <tbansal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578867}
      bec13327
    • angle-chromium-autoroll's avatar
      Roll src/third_party/angle 9bf86f07acd5..306b6c168dc7 (2 commits) · 3e236e93
      angle-chromium-autoroll authored 
      https://chromium.googlesource.com/angle/angle.git/+log/9bf86f07acd5..306b6c168dc7


git log 9bf86f07acd5..306b6c168dc7 --date=short --no-merges --format='%ad %ae %s'
2018-07-27 jmadill@chromium.org D3D11: Use angle::Result error pattern. 1/3
2018-07-27 geofflang@chromium.org Fix/supress issues with EGL images on Android.


Created with:
  gclient setdep -r src/third_party/angle@306b6c168dc7

The AutoRoll server is located here: https://angle-chromium-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.

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

Change-Id: I01ee932f0a5415853216c178e2b860ba85be4ddf
Reviewed-on: https://chromium-review.googlesource.com/1153767Reviewed-by: default avatarangle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#578866}
      3e236e93
    • Penny MacNeil's avatar
      [ui/display] Change component name on Windows. · 9b65c69b
      Penny MacNeil authored 
      "display.dll" is a Windows system dll.  This causes a clash in debug component builds.
Change the GN output_name for the display component to "ui_display".

Ref: crbug/818388

Test: chromium.gpu waterfall bot (debug component), or chromium.gpu.fyi.
Change-Id: Ibdb41cb6841708c61893a0e05d00f5af17b583cd
Reviewed-on: https://chromium-review.googlesource.com/1152099Reviewed-by: default avatarRobert Liao <robliao@chromium.org>
Reviewed-by: default avatarMitsuru Oshima <oshima@chromium.org>
Commit-Queue: Penny MacNeil <pennymac@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578865}
      9b65c69b
    • Kenneth Russell's avatar
      Update shaderoperator_common_functions suppression. · 84424c67
      Kenneth Russell authored 
      It should be applied on all macOS versions, not just Sierra. This
affects the following WebGL 2.0 conformance test:
  deqp/functional/gles3/shaderoperator/common_functions.html

Tbr: kainino@chromium.org
Bug: 820225
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
Change-Id: Ib0ff46c871d8ab74e56026aff96cd150e61a33d9
Reviewed-on: https://chromium-review.googlesource.com/1152452Reviewed-by: default avatarKenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578864}
      84424c67
    • Weiyong Yao's avatar
      Add braveyao as OWNERS in desktopCapture related logic in Chromium · 631d8192
      Weiyong Yao authored 
      Bug: 867003
Change-Id: I2ef56eeafcee60f23fbd2d67fb3193eb9cd360d4
Reviewed-on: https://chromium-review.googlesource.com/1148694
Commit-Queue: Weiyong Yao <braveyao@chromium.org>
Reviewed-by: default avatarSergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578863}
      631d8192
    • Scott Violet's avatar
      Revert "Misc. cleanup to browser_theme_pack_unittest.cc; no functional change." · 2411cb26
      Scott Violet authored 
      This reverts commit ec2cd434.

Reason for revert: Reverting in hopes of fixing unit_tests on chromeos bot. This one will hopefully take. Sorry for so many reverts here. The chain of patches made it difficult.

Original change's description:
> Misc. cleanup to browser_theme_pack_unittest.cc; no functional change.
> 
> * Add GetDefaultColor() mapping function to make it easy to fix DCHECKs due to
>   this code using raw _INCOGNITO color constants but
>   ThemeProperties::GetDefaultColor() not expecting them.  This is the change
>   that led me to get annoyed and do all the other stuff here.
> * Convert typedefs to type aliases.
> * Add TP = ThemeProperties type alias for brevity.
> * Use =default.
> * Mark functions that don't touch member variables "static".
> * Fix wrapping issues.
> * Because the class here is so long, split it into declaration and definition
>   pieces so it's easier to see what's the API and what's helper functions.
> * Make the API protected; make helpers private.
> * Make data members private.  Add a const accessor for the one member tests use.
> 
> Bug: none
> Change-Id: I4cc058bb29747cfa84835edbc5ce765401e14527
> Reviewed-on: https://chromium-review.googlesource.com/1152108
> Reviewed-by: Evan Stade <estade@chromium.org>
> Commit-Queue: Peter Kasting <pkasting@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578775}

TBR=pkasting@chromium.org,estade@chromium.org

Change-Id: I2407b13ff9c69c240bb7479e21a515c17c1388b7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: none
Reviewed-on: https://chromium-review.googlesource.com/1153991Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578862}
      2411cb26
    • Scott Violet's avatar
      Revert "Add background tab colors and inactive images to the theme system." · 62a8750b
      Scott Violet authored 
      This reverts commit 8c35e22c.

Reason for revert: Reverting to fix unit_tests on chromeos bot. This is next in the chain. Ugh!

Original change's description:
> Add background tab colors and inactive images to the theme system.
> 
> These will eventually be used for three purposes:
> (1) To allow generating different background tab images from a common tint for
>     the inactive and incognito cases, thus allowing background tabs that blend
>     into the frame for custom themes.
> (2) To allow autogenerating the background tab colors from the background tab
>     images, thus letting us autocalculate better tab text/separator/etc. colors.
> (3) To allow theme authors to customize/override this behavior more precisely.
> 
> Reasons (1) and (2) dictated the design here; reason (3) is basically a free
> bonus.  I thought about not making these resources overrideable by theme
> authors, but there was some technical reason that turned out to not be easily
> feasible, and there seems little reason to prevent it.
> 
> This also allows us to read THEME_TAB_BACKGROUND_INCOGNITO on Mac (Views).
> 
> This shouldn't result in a visual change on its own.
> 
> Bug: 862664
> Change-Id: Icc57f78fd526d68b83c692fbac669a7371bb38ff
> Reviewed-on: https://chromium-review.googlesource.com/1147869
> Commit-Queue: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
> Reviewed-by: Evan Stade <estade@chromium.org>
> Reviewed-by: Allen Bauer <kylixrd@chromium.org>
> Reviewed-by: Bret Sepulveda <bsep@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578790}

TBR=pkasting@chromium.org,oshima@chromium.org,estade@chromium.org,bsep@chromium.org,kylixrd@chromium.org

Change-Id: I6abe212768916890c11aec28aac5fc96891ac695
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 862664
Reviewed-on: https://chromium-review.googlesource.com/1153990Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578861}
      62a8750b
    • Scott Violet's avatar
      Revert "Add background tab text colors for inactive and incognito windows." · 67f5bec3
      Scott Violet authored 
      This reverts commit 3b7859cb.

Reason for revert: Reverting in hopes of get unit_tests passing on chromeos bot. This appears to be next in the chain to revert.

Original change's description:
> Add background tab text colors for inactive and incognito windows.
> 
> With the default behavior of Chrome being to not show a tab background, we want
> to change themes to default to that as well, unless the theme author actually
> supplies an image/color for this explicitly.  To do this, we need to support
> different tab text colors for the cases where the frame colors can vary.  This
> adds the framework for that support.
> 
> Right now, the default is to use the same text colors in inactive windows as
> active ones.
> 
> This also gives themes that want normal and incognito windows to differ greatly
> (as the built-in theme's does) the ability to implement that and still have
> readable tab titles.
> 
> Bug: 866672
> Change-Id: I032a72bc799d0675c517db10283513a30db23328
> Reviewed-on: https://chromium-review.googlesource.com/1152418
> Commit-Queue: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Evan Stade <estade@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578807}

TBR=pkasting@chromium.org,estade@chromium.org,bsep@chromium.org

Change-Id: I5e3a5772cea3f7a0d3f790d27dcccc66c51cb161
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 866672
Reviewed-on: https://chromium-review.googlesource.com/1154067Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578860}
      67f5bec3
    • Robert Kroeger's avatar
      [ozone/drm/mojo] Remove a mash only DCHECK · a954d47c
      Robert Kroeger authored 
      Ozone/drm/mojo contained a DCHECK only appropriate for legacy mash mode.
This mode has been removed so remove the DCHECK as well.

BUG=620927
TEST=by hand

Change-Id: Ieb8c6b21ed79de2202ee06eedf47029d32183421
Reviewed-on: https://chromium-review.googlesource.com/1153968Reviewed-by: default avatarweiliangc <weiliangc@chromium.org>
Commit-Queue: Robert Kroeger <rjkroege@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578859}
      a954d47c
    • Sammie Quon's avatar
      overivew: Fix close button ripple black. · e035b8ce
      Sammie Quon authored 
      Test: manual
Bug: 866376
Change-Id: I5527ea59fadfd4ecdd1eb3a9473a72b618a1ab53
Reviewed-on: https://chromium-review.googlesource.com/1153970Reviewed-by: default avatarXiaoqian Dai <xdai@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578858}
      e035b8ce
    • Daniel Murphy's avatar
      [SessionStorage] Fixed re-use of data maps that were purged · 1bca97b2
      Daniel Murphy authored 
      When PurgeUnusedAreas is called, the unbound areas are destroyed. If one
of these areas was holding onto a data map that was cloned to another
area in a different namespace, a re-bind to that area created a new map
instead of using the already-existing one.

This change allows the SessionStorageNamespaceImpl to look up maps that
are already in use to possibly re-use a map that already exists.

R=mek@chromium.org

Bug: 866887
Change-Id: Ifdb14278bb223bbdcb877e4fddccd158968fb575
Reviewed-on: https://chromium-review.googlesource.com/1153770
Commit-Queue: Daniel Murphy <dmurph@chromium.org>
Reviewed-by: default avatarMarijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578857}
      1bca97b2
    • Scott Violet's avatar
      Revert "Compute colors for the frame and tabs from provided images." · a37ba3b9
      Scott Violet authored 
      This reverts commit df84748a.

Reason for revert: To fix unit_tests (I have to revert this before the earlier patch). Likely suspect for BrowserThemePackTest.HiDpiThemeTest failing on bots. For example, https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/linux-chromeos-rel/11230 :

[ RUN      ] BrowserThemePackTest.HiDpiThemeTest
../../chrome/browser/themes/browser_theme_pack_unittest.cc:204: Failure
Expected equality of these values:
  ""
  error
    Which is: "File doesn't exist."
Stack trace:
#0 0x0000026103ac testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop()
#1 0x00000260fd89 testing::internal::AssertHelper::operator=()
#2 0x000000fc9b98 BrowserThemePackTest::BuildFromUnpackedExtension()
#3 0x000000fd22cc BrowserThemePackTest_HiDpiThemeTest_Test::TestBody()
../../chrome/browser/themes/browser_theme_pack_unittest.cc:205: Failure
Value of: valid_value.get()
  Actual: false
Expected: true
Stack trace:
#0 0x0000026103ac testing::internal::UnitTestImpl::CurrentOsStackTraceExceptTop()
#1 0x00000260fd89 testing::internal::AssertHelper::operator=()
#2 0x000000fc9e0b BrowserThemePackTest::BuildFromUnpackedExtension()
#3 0x000000fd22cc BrowserThemePackTest_HiDpiThemeTest_Test::TestBody()
Received signal 11 SEGV_MAPERR 000000000028
#0 0x000004d00e1c base::debug::StackTrace::StackTrace()
#1 0x000004d00981 base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x7f26bba79330 <unknown>
#3 0x0000050f3e24 BrowserThemePack::WriteToDisk()
#4 0x000000fd22db BrowserThemePackTest_HiDpiThemeTest_Test::TestBody()
#5 0x0000026161b2 testing::Test::Run()
#6 0x000002616d30 testing::TestInfo::Run()
#7 0x000002617247 testing::TestCase::Run()
#8 0x000002622747 testing::internal::UnitTestImpl::RunAllTests()
#9 0x0000026222bd testing::UnitTest::Run()
#10 0x0000045565d1 base::TestSuite::Run()
#11 0x000004557fca base::(anonymous namespace)::LaunchUnitTestsInternal()
#12 0x000004557e7a base::LaunchUnitTests()
#13 0x00000454e155 main
#14 0x7f26b86c9f45 __libc_start_main
#15 0x0000006fa82a _start
  r8: 0000000000000000  r9: 54656d6568546970 r10: 747365545f747365 r11: 0000000000000000
 r12: 000024a8af0c0d20 r13: 00007fff447924e8 r14: 00007fff44792560 r15: 0000000000000000
  di: 0000000000000000  si: 00007fff44792560  bp: 00007fff44792520  bx: 00007fff44792578
  dx: 00000000000005e7  ax: 0000000000001fdd  cx: 0000000000000023  sp: 00007fff44792490
  ip: 00000000050f3e24 efl: 0000000000010206 cgf: 0000000000000033 erf: 0000000000000004
 trp: 000000000000000e msk: 0000000000000000 cr2: 0000000000000028
[end of stack trace]

Original change's description:
> Compute colors for the frame and tabs from provided images.
> 
> If theme authors explicitly provide values here, we'll use them; but if they
> don't, but do provide images, set the colors to be the dominant colors of the
> images, using an existing K-means algorithm (similar to what we use for
> computing representative favicon colors etc.).
> 
> This allows us to rely on those colors later when deciding what color to make
> tab text, the new tab button, and similar tabstrip/window frame items.
> 
> This also does a bit of cleanup to the theme pack code, e.g. moving a map used
> in only one function into that function to make its provenance clearer.  I can
> try to split this apart into more CLs if desired.
> 
> Bug: 862664
> Change-Id: I8c3f15893ad491a6fee8aa5f76ae162cecb2c6fd
> Reviewed-on: https://chromium-review.googlesource.com/1152517
> Commit-Queue: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
> Reviewed-by: Evan Stade <estade@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578823}

TBR=pkasting@chromium.org,asvitkine@chromium.org,estade@chromium.org

Change-Id: I6a5a7f2916136ccf93101f36c34a17a4e5ef5d4f
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 862664
Reviewed-on: https://chromium-review.googlesource.com/1153988Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578856}
      a37ba3b9
    • Johannes Henkel's avatar
      Migrate InspectorCSSAgent to the new InspectorSessionState. · b6b32cb9
      Johannes Henkel authored 
      I've renamed the enable* terminology a bit, since it's a
two-phase enable here, so I think it's good to have
enable_requested_ (persistent) and enable_completed_ (transient).

For reference the PR that introduced the new
InspectorSessionState / InspectorAgentState / Fields
was https://chromium-review.googlesource.com/c/chromium/src/+/1149201.

Change-Id: I0ec3e9a9b8c20e5200bc5932f514bfe873f74a2c
Reviewed-on: https://chromium-review.googlesource.com/1153571
Commit-Queue: Johannes Henkel <johannes@chromium.org>
Reviewed-by: default avatarAndrey Kosyakov <caseq@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578855}
      b6b32cb9
    • Peter Collingbourne's avatar
      Reland "build: Enable ThinLTO in official Android builds." · 748ac91e
      Peter Collingbourne authored 
      This relands commit 1ba5d611.

Fixes chromeos build issue by checking whether we have a secondary
toolchain before comparing it with the current toolchain.

Original change's description:
> build: Enable ThinLTO in official Android builds.
>
> With this change we start building Chromium with ThinLTO when targeting
> Android. In 64-bit Monochrome builds, the 32-bit binary is also built
> with ThinLTO. This change is expected to reduce code size significantly
> (in local measurements it reduced the size of MonochromePublic.apk by 2.3MB)
> and is a prerequisite for enabling control flow integrity in official
> builds.
>
> Note that although this change enables ThinLTO, it does not enable
> the cross-TU optimizations normally associated with LTO. Enabling
> cross-TU optimizations is a separate project.
>
> On my local machine (a Lenovo P920) libmonochrome.so link times increase
> as follows:
>
>          before  after (first link)  after (incremental links)
> 32-bit    18s          140s                     50s
> 64-bit    13s          107s                     41s
>
> All known blockers have now been fixed. This CL is likely to uncover
> unknown blockers.
>
> Bug: 469376
> Change-Id: I8981e17abd50bc5ca00440e0d74dda878c911749
> Reviewed-on: https://chromium-review.googlesource.com/1150870
> Commit-Queue: Peter Collingbourne <pcc@chromium.org>
> Reviewed-by: Nico Weber <thakis@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#578195}

Bug: 469376, 868426
Change-Id: Ie46ba0db0ac68130bc600415777de5c813b28ebd
Reviewed-on: https://chromium-review.googlesource.com/1153612Reviewed-by: default avatarDirk Pranke <dpranke@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578854}
      748ac91e
    • Danyao Wang's avatar
      [Nav Experiment] Fix error retry state corruption for WebErrorPages. · 268a40ff
      Danyao Wang authored 
      When an error page is displayed in web view, the error retry state of
the navigation item is never transitioned to
kDisplayingWebErrorForFailedNavigation. This does not prevent the
displaying of error page on the first failure, but subsequent back-
forward navigation to this item starts with the incorrect state and
can result in DCHECKs of unexpected error state, error view not
being loaded and poor interaction with other types of errors, such as
SSL interstitial.

This CL refactors the state transitions for WebErrorPages so it is
handled identically to native error. This also required web error
load to not skip DidFinishNavigation callback to correctly update
the ErrorRetryStateMachine.

Also parameterized error_page_inttest.mm so both navigation manager
implementations are tested on trybots.

Bug: 837210
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I4f63c3c9bae21158deb705a3da5ed3eeab59a8bb
Reviewed-on: https://chromium-review.googlesource.com/1151542
Commit-Queue: Danyao Wang <danyao@chromium.org>
Reviewed-by: default avatarKurt Horimoto <kkhorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578853}
      268a40ff
    • Florin Malita's avatar
      Remove SK_SUPPORT_LEGACY_TEXT_BLOB · 1f3c2913
      Florin Malita authored 
      Removing this flag will allow skia to process text blobs
using the new glyph run code.

TBR=

Change-Id: Ib834240778121f628ac2b966211703e7a69588ef
Reviewed-on: https://chromium-review.googlesource.com/1153449Reviewed-by: default avatarFlorin Malita <fmalita@chromium.org>
Commit-Queue: Florin Malita <fmalita@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578852}
      1f3c2913
    • Klaus Weidner's avatar
      Avoid DCHECK failure on WebXR immersive session transition · b1471eca
      Klaus Weidner authored 
      The first frame of an immersive session may not yet have frame data
including the expected frame_id and output buffer mailbox holder,
don't treat that as an error. See also https://crbug.com/855722

BUG=868028

Cq-Include-Trybots: luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Icecd3ea8035ea783a625b11082e736d9609a15f8
Reviewed-on: https://chromium-review.googlesource.com/1153829Reviewed-by: default avatarBrian Sheedy <bsheedy@chromium.org>
Commit-Queue: Klaus Weidner <klausw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578851}
      b1471eca
    • CJ DiMeglio's avatar
      Pass the whether the video is opaque to the WebVideoFrameSubmitter. · 2721137b
      CJ DiMeglio authored 
      Before, we passed in dummy information that always assumed that the video
was opaque. Now we correctly communcate this information such that the video
can now be transparent.

Bug: 863203
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ic4617ef334f01d72ec4c5494d8d9c33accc581db
Reviewed-on: https://chromium-review.googlesource.com/1141342
Commit-Queue: CJ DiMeglio <lethalantidote@chromium.org>
Reviewed-by: default avatarFrank Liberato <liberato@chromium.org>
Reviewed-by: default avatarDaniel Cheng <dcheng@chromium.org>
Reviewed-by: default avatarJustin Novosad <junov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578850}
      2721137b
    • Xiyuan Xia's avatar
      Update DropTargetEvent ctor to use PointF · 5fef4c48
      Xiyuan Xia authored 
      Update DropTargetEvent ctor to take gfx::PointF instead of
gfx::Point. No behavior change.

Bug: None
Change-Id: I0d38527bd73edc49a0dfb974a33ce95eec94af61
Reviewed-on: https://chromium-review.googlesource.com/1153433Reviewed-by: default avatarScott Violet <sky@chromium.org>
Reviewed-by: default avatarMichael Wasserman <msw@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578849}
      5fef4c48
    • Takuto Ikuta's avatar
      Fix to remove /wd4018 (-Wno-sign-compare) for /services/service_manager/sandbox/win · beea8ba5
      Takuto Ikuta authored 
      This is a part of effort to remove /wd4018 warning suppression.

Master CL is
https://chromium-review.googlesource.com/c/chromium/src/+/1152755

This CL was uploaded by git cl split.

R=forshaw@chromium.org

Bug: 868248, 588506
Change-Id: I6d0a6763d4afbe3fd51f9841b19adbe70209c68d
Reviewed-on: https://chromium-review.googlesource.com/1152842Reviewed-by: default avatarPenny MacNeil <pennymac@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578848}
      beea8ba5
    • pdfium-chromium-autoroll's avatar
      Roll src/third_party/pdfium d72fd5888686..6146214d30ab (1 commits) · 0786d403
      pdfium-chromium-autoroll authored 
      https://pdfium.googlesource.com/pdfium.git/+log/d72fd5888686..6146214d30ab


git log d72fd5888686..6146214d30ab --date=short --no-merges --format='%ad %ae %s'
2018-07-27 npm@chromium.org Merge several methods into JBig2GrdProc::DecodeArithTemplateUnopt


Created with:
  gclient setdep -r src/third_party/pdfium@6146214d30ab

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: Ib69b3d66c1350be2f06380cb03bf852efa958a43
Reviewed-on: https://chromium-review.googlesource.com/1153291Reviewed-by: default avatarpdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#578847}
      0786d403
    • Takuto Ikuta's avatar
      Fix to remove /wd4018 (-Wno-sign-compare) for /chrome_elf · b6ae83fd
      Takuto Ikuta authored 
      This is a part of effort to remove /wd4018 warning suppression.

Master CL is
https://chromium-review.googlesource.com/c/chromium/src/+/1152755

This CL was uploaded by git cl split.

R=pennymac@chromium.org

Bug: 868248, 588506
Change-Id: Iba8b4516550b89b5acf534bc1fa11d959154550e
Reviewed-on: https://chromium-review.googlesource.com/1152841Reviewed-by: default avatarPenny MacNeil <pennymac@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578846}
      b6ae83fd
    • siyua's avatar
      change tooltip message for card migration bubble · 1f083a9b
      siyua authored 
      Bug: b/111843286
Change-Id: I9d198b12f7e3fc4c51146f91610eb1f00a19390e
Reviewed-on: https://chromium-review.googlesource.com/1153303Reviewed-by: default avatarJared Saul <jsaul@google.com>
Commit-Queue: Siyu An <siyua@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578845}
      1f083a9b
    • Aga Wronska's avatar
      Show offline demo mode option on network screen in demo mode setup flow. · a7120df5
      Aga Wronska authored 
      Real usage of the offline setup on network list will be implemented
separately.

Bug: 862445
Test: Run DemoSetupTest and wizard_controller_browsertests.
Change-Id: I67c0ca2862ca30fe2a2c2115d413fe0f7dae0191
Reviewed-on: https://chromium-review.googlesource.com/1151951Reviewed-by: default avatarSteven Bennetts <stevenjb@chromium.org>
Reviewed-by: default avatarMichael Giuffrida <michaelpg@chromium.org>
Reviewed-by: default avatarAlexander Alekseev <alemate@chromium.org>
Commit-Queue: Aga Wronska <agawronska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578844}
      a7120df5
    • Joshua Peraza's avatar
      Implement CrashReporterClient methods for Crashpad · 0c0fc42d
      Joshua Peraza authored 
      This new implementation of GetProductNameAndVersion() will replace the
existing one in the transition to Crashpad.

GetSanitizationInformation() (for webview) will replace
GetCrashKeysWhitelist() and the SanitizationInfo struct passed during
crash reporter initialization.

These new methods are currently unused.

Bug: crashpad:30
Change-Id: Ic60d17cb3af8b4e7997009c23a44fcc8b94915a7
Reviewed-on: https://chromium-review.googlesource.com/1142511Reviewed-by: default avatarBo <boliu@chromium.org>
Reviewed-by: default avatarMike West <mkwst@chromium.org>
Reviewed-by: default avatarRobert Sesek <rsesek@chromium.org>
Reviewed-by: default avatarLuke Halliwell <halliwell@chromium.org>
Commit-Queue: Joshua Peraza <jperaza@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578843}
      0c0fc42d