GitLab

https://chromium.googlesource.com/chromium/blink/+log/559a6d4ab7a84c539a58fbc8d8dbc64c97994359..78807df2ce8a43f50941cee9ae05458df0cc05bf
TBR=

Review URL: https://codereview.chromium.org/529633002

Cr-Commit-Position: refs/heads/master@{#292834}

Note: This only runs pylint and tests for the auto_bisect directory (this was much simpler than picking and choosing particular files from the parent tools directory).

I think the next step is to move bisect-perf-regression.py and related files into the auto_bisect directory.

BUG=

Review URL: https://codereview.chromium.org/529593002

Cr-Commit-Position: refs/heads/master@{#292833}

Previously, the error message was 'An error occurred.' This patch changes it to "This file could not be displayed."

BUG=405860
TEST=manually tested
TBR=fukino@chromium.org

Review URL: https://codereview.chromium.org/523663002

Cr-Commit-Position: refs/heads/master@{#292832}

If a service is enabled after initialization failed,
 SyncEngine didn't try initializing SyncWorker again, and that makes SyncFS stuck.

BUG=402356

Review URL: https://codereview.chromium.org/470403004

Cr-Commit-Position: refs/heads/master@{#292831}

- rename WebRtcBrowserTest.TwoCallsWithAecDump to WebRtcAecDumpBrowserTest.*CallWithAecDump*

TBR=phoglund@chromium.org
BUG=389132
NOTRY=true

Review URL: https://codereview.chromium.org/523123003

Cr-Commit-Position: refs/heads/master@{#292830}

MetadataCache is soon going to be generalized from Drive to also FSP file
systems. So, in the Files app layer, we should not have Drive specific code.

TEST=Tested manually.
BUG=408017

Review URL: https://codereview.chromium.org/517213002

Cr-Commit-Position: refs/heads/master@{#292829}

https://chromium.googlesource.com/chromium/blink/+log/332a56a9d8e5d800e3a1fb8aa69c241d0ce59912..559a6d4ab7a84c539a58fbc8d8dbc64c97994359
TBR=

Review URL: https://codereview.chromium.org/525693003

Cr-Commit-Position: refs/heads/master@{#292828}

Now that the Files.app uses default system header, the own profile badge is redundant.

BUG=408816
TEST=confirmed manually, ran browser_tests

Review URL: https://codereview.chromium.org/520623002

Cr-Commit-Position: refs/heads/master@{#292827}

Scrolling remains horizontal in the normal (and centered) app lists.

BUG=406222
TEST=Run with --enable-experimental-app-list --show-app-list. Apps grid
view should scroll vertically. Normal app list should still scroll
horizontally.

Review URL: https://codereview.chromium.org/491973004

Cr-Commit-Position: refs/heads/master@{#292826}

TBR=phoglund@chromium.org
BUG=389132
NOTRY=true

Review URL: https://codereview.chromium.org/518173004

Cr-Commit-Position: refs/heads/master@{#292825}

TBR=henrika@chromium.org
NOTRY=true
BUG=409485

Review URL: https://codereview.chromium.org/528653002

Cr-Commit-Position: refs/heads/master@{#292824}

This replaces usage of NPObject in pepper with gin-backed V8 objects. It is unfortunate that this CL is so large, but there isn't a nice way to have the old implementation and the new one side-by-side.

There are 4 major parts to this CL:
1) Changing the HostVarTracker to track V8ObjectVars rather than NPObjectVars (host_var_tracker.cc).
2) Changing plugin elements (in plugin_object.cc) to be gin-backed objects.
3) Changing postMessage bindings (message_channel.cc) be gin-backed objects.
4) Changing the implementation of PPB_Var_Deprecated (ppb_var_deprecated_impl.cc) to call directly into V8.

BUG=351636

Committed: https://chromium.googlesource.com/chromium/src/+/21f446ae855d60cc896b40cb9a3249ed07f150b3

Committed: https://chromium.googlesource.com/chromium/src/+/ee49e63baf57e503bd71dfe61c8a80df63eac9aa

Review URL: https://codereview.chromium.org/459553003

Cr-Commit-Position: refs/heads/master@{#292823}

Use registry virtualization so that stale data on the test machines
don't interfere with the tests.

BUG=375739

Committed: https://chromium.googlesource.com/chromium/src/+/6d6ac988d6a7a311876f3b0d0257086ac58c8181

Review URL: https://codereview.chromium.org/521703002

Cr-Commit-Position: refs/heads/master@{#292822}

Suppresses

In file included from ..\..\remoting/host/verify_config_window_win.h:10:
..\..\third_party\wtl\include\atlapp.h(1683,17) :  warning(clang): using namespace directive in global context in header [-Wheader-hygiene]
using namespace WTL;
                ^

BUG=5027,82385
TBR=jamiewalch@chromium.org

Review URL: https://codereview.chromium.org/526863002

Cr-Commit-Position: refs/heads/master@{#292821}

Applying new styling, with some refactoring and bug fixes. Details:

- Sizing and positioning tweaks.
- Change of assets for "X" and for default favicon (if it's missing).
  - Using image as mask, and setting color by CSS (since design uses
    solid colors).
- The new PNG files have been minimized.
- Fixed bug: "Undo" and "Restore All" links continue to be selectable
  via tab after they fades out.
- Alternative Google logo (white): using image as mask and setting
  background color to #eee.  Will delete old image for M39.
- Theme title color in title.html <iframe>: now injecting it from local
  NTP in "c=RRGGBBAA" format. The old behavior of reading it from
  <iframe> and using it to override "c=RRGGBB" are kept, for
  compatibility with server-side NTP.

BUG=407943
NOTRY=1
R=mathp@chromium.org, oshima@chromium.org

Review URL: https://codereview.chromium.org/512583002

Cr-Commit-Position: refs/heads/master@{#292820}

https://skia.googlesource.com/skia/+log/21e613505562368e7719215f824fa8b3b4be652d..700a78d72fdf4f09b28446818028b39f31e26b38

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=jvanverth@google.com

Review URL: https://codereview.chromium.org/525973002

Cr-Commit-Position: refs/heads/master@{#292819}

Revert of Revert of clang/win: Fix a few warnings in targets not in chromium_builder_tests. (patchset #1 id:1 of https://codereview.chromium.org/519463003/)

Reason for revert:
Reland, wasn't the reason for the redness. (https://codereview.chromium.org/521703002/ was.)

Original issue's description:
> Revert of clang/win: Fix a few warnings in targets not in chromium_builder_tests. (patchset #3 id:40001 of https://codereview.chromium.org/526513002/)
>
> Reason for revert:
> Speculative, might have broken InstallerStateTest.InitializeTwice on XP: http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32481
>
> InstallerStateTest.InitializeTwice (run #1):
> [ RUN      ] InstallerStateTest.InitializeTwice
> c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str())
>   Actual: false
> Expected: true
> [  FAILED  ] InstallerStateTest.InitializeTwice (219 ms)
>
> InstallerStateTest.InitializeTwice (run #2):
> [ RUN      ] InstallerStateTest.InitializeTwice
> c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str())
>   Actual: false
> Expected: true
> [  FAILED  ] InstallerStateTest.InitializeTwice (109 ms)
>
>
> Original issue's description:
> > clang/win: Fix a few warnings in targets not in chromium_builder_tests.
> >
> > Also don't use "default" as a variable name, as it's a keyword.
> > Also fix a bug where a wstring was passed to %ls.
> >
> > No real behavior change.
> >
> > BUG=82385
> > R=hans@chromium.org
> > TBR=cpu, vitalybuka
> >
> > Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/d7efa09
>
> TBR=hans@chromium.org,robertshield@chromium.org,gab@chromium.org,vitalybuka@chromium.org,cpu@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=82385
>
> Committed: https://chromium.googlesource.com/chromium/src/+/dae20dafcfd3ccc439ccc5b3d5e21ad611d13c23

TBR=hans@chromium.org,robertshield@chromium.org,gab@chromium.org,vitalybuka@chromium.org,cpu@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=82385

Review URL: https://codereview.chromium.org/525013003

Cr-Commit-Position: refs/heads/master@{#292818}

Cr-Commit-Position: refs/heads/master@{#292817}

Revert of EasyUnlock: Work around the ghost overlay from CSS animation. (patchset #3 id:40001 of https://codereview.chromium.org/487373002/)

Reason for revert:
No longer needed since underlying blink issue http://crbug.com/405250 is fixed.

Original issue's description:
> EasyUnlock: Work around the ghost overlay from CSS animation.
>
> BUG=404779
>
> Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=290743

TBR=tbarzic@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=404779

Review URL: https://codereview.chromium.org/521343002

Cr-Commit-Position: refs/heads/master@{#292816}

Cr-Commit-Position: refs/heads/master@{#292815}

https://chromium.googlesource.com/chromium/blink/+log/a00905c53f2a7e9c79d324b3ac36dc8b05fb0e7f..332a56a9d8e5d800e3a1fb8aa69c241d0ce59912
TBR=

Review URL: https://codereview.chromium.org/525163002

Cr-Commit-Position: refs/heads/master@{#292814}

https://skia.googlesource.com/skia/+log/9e19ce1641422857b850cdedaee40927643b2f42..21e613505562368e7719215f824fa8b3b4be652d

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=jvanverth@google.com

Review URL: https://codereview.chromium.org/524263002

Cr-Commit-Position: refs/heads/master@{#292813}

Revert of Fix ProductTest.ProductInstallBasic and InstallerStateTest.InitializeTwice flakes. (patchset #4 id:80001 of https://codereview.chromium.org/521703002/)

Reason for revert:
Breaks InstallerStateTest.InitializeTwice on XP:
http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32481

Original issue's description:
> Fix ProductTest.ProductInstallBasic and InstallerStateTest.InitializeTwice flakes.
>
> Use registry virtualization so that stale data on the test machines
> don't interfere with the tests.
>
> BUG=375739
>
> Committed: https://chromium.googlesource.com/chromium/src/+/6d6ac988d6a7a311876f3b0d0257086ac58c8181

TBR=robertshield@chromium.org,grt@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=375739

Review URL: https://codereview.chromium.org/520863005

Cr-Commit-Position: refs/heads/master@{#292812}

https://chromium.googlesource.com/chromium/blink/+log/244a450e5891380a808296994588063b762f2a63..a00905c53f2a7e9c79d324b3ac36dc8b05fb0e7f
TBR=

Review URL: https://codereview.chromium.org/520383005

Cr-Commit-Position: refs/heads/master@{#292811}

OpenGL/gl.h is required to be included to make the build pass.

BUG=
TEST=

Review URL: https://codereview.chromium.org/522533002

Cr-Commit-Position: refs/heads/master@{#292810}

Revert of clang/win: Fix a few warnings in targets not in chromium_builder_tests. (patchset #3 id:40001 of https://codereview.chromium.org/526513002/)

Reason for revert:
Speculative, might have broken InstallerStateTest.InitializeTwice on XP: http://build.chromium.org/p/chromium.win/builders/XP%20Tests%20%281%29/builds/32481

InstallerStateTest.InitializeTwice (run #1):
[ RUN      ] InstallerStateTest.InitializeTwice
c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str())
  Actual: false
Expected: true
[  FAILED  ] InstallerStateTest.InitializeTwice (219 ms)

InstallerStateTest.InitializeTwice (run #2):
[ RUN      ] InstallerStateTest.InitializeTwice
c:\b\build\slave\win_builder\build\src\chrome\installer\util\installer_state_unittest.cc(639): error: Value of: wcsstr(installer_state.target_path().value().c_str(), BrowserDistribution::GetSpecificDistribution( BrowserDistribution::CHROME_BINARIES)-> GetInstallSubDir().c_str())
  Actual: false
Expected: true
[  FAILED  ] InstallerStateTest.InitializeTwice (109 ms)

Original issue's description:
> clang/win: Fix a few warnings in targets not in chromium_builder_tests.
>
> Also don't use "default" as a variable name, as it's a keyword.
> Also fix a bug where a wstring was passed to %ls.
>
> No real behavior change.
>
> BUG=82385
> R=hans@chromium.org
> TBR=cpu, vitalybuka
>
> Committed to pending queue: https://chromium.googlesource.com/chromium/src/+/d7efa09

TBR=hans@chromium.org,robertshield@chromium.org,gab@chromium.org,vitalybuka@chromium.org,cpu@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=82385

Review URL: https://codereview.chromium.org/519463003

Cr-Commit-Position: refs/heads/master@{#292809}

https://skia.googlesource.com/skia/+log/7843a24ab2c2dc4fe77e82b9689cde953d6abd7c..9e19ce1641422857b850cdedaee40927643b2f42

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=djsollen@google.com

Review URL: https://codereview.chromium.org/521283002

Cr-Commit-Position: refs/heads/master@{#292808}

There was a race condition in the added test check in the previous
attempt. That made the test Super Flaky. This is more noticeable in the
Release builds than the Debug build that I was working with.
(see https://codereview.chromium.org/471233003/)

With a Release build I was able to repro the flakiness (40% failure
over 200 runs). After the fix, I tested 500 runs and the success rate
is now 100%. I can't say it won't flake for other reasons, but
this fixes the main flake that was introduced. Historically,
the test does ocassionally need to be retried due to timeouts,
according to the bot log history. We may want to split the
CORS and the non-CORS test case to see if that will help w/ timeouts.

I didn't want to do that for this patch, since it would get
much more complicated and this is intended for merging to M38.

Retry with the race condition fixed. Also re-enable the test since
it got disabled by another CL for being flaky.

BUG=401755
BUG=315328

Review URL: https://codereview.chromium.org/524683002

Cr-Commit-Position: refs/heads/master@{#292807}

Hide registration promos for supervised and guest users. Also hide cloud print
connector section and irrelevant "cloud devices" section (since users cannot log
in).

BUG=288614,340893

Review URL: https://codereview.chromium.org/521953002

Cr-Commit-Position: refs/heads/master@{#292806}

https://chromium.googlesource.com/chromium/blink/+log/dafd1d0b92247b98a38dfa8bc29ceb80e77582f1..244a450e5891380a808296994588063b762f2a63
TBR=

Review URL: https://codereview.chromium.org/528453002

Cr-Commit-Position: refs/heads/master@{#292805}

Was necessary for https://codereview.chromium.org/517213005/.

R=lcwu@chromium.org,damienv@chromium.org
BUG=None

Review URL: https://codereview.chromium.org/526693002

Cr-Commit-Position: refs/heads/master@{#292804}

Depends on https://codereview.chromium.org/495763005/

BUG=406622

Review URL: https://codereview.chromium.org/497273002

Cr-Commit-Position: refs/heads/master@{#292803}

https://chromium.googlesource.com/chromium/blink/+log/3bbc0b2b6d62c60bd38ce576e1af5a68e1e7cb6d..dafd1d0b92247b98a38dfa8bc29ceb80e77582f1
TBR=

Review URL: https://codereview.chromium.org/526703002

Cr-Commit-Position: refs/heads/master@{#292802}

TBR=niklase
NOTRY=true

Review URL: https://codereview.chromium.org/525083002

Cr-Commit-Position: refs/heads/master@{#292801}

Also change remote video and pepper plugin MediaStreamVideoSource implementations to not set a guessed frame rate.

BUG= 394315

Review URL: https://codereview.chromium.org/517973002

Cr-Commit-Position: refs/heads/master@{#292800}

TBR=niklase@chromium.org
NOTRY=True

Review URL: https://codereview.chromium.org/526683002

Cr-Commit-Position: refs/heads/master@{#292799}

https://chromium.googlesource.com/chromium/blink/+log/b28553bee158127ed49ff22547f909f49ed9e00a..3bbc0b2b6d62c60bd38ce576e1af5a68e1e7cb6d
TBR=

Review URL: https://codereview.chromium.org/521163002

Cr-Commit-Position: refs/heads/master@{#292798}

_GetFileVersionFromCache/_GetFileListingVersionFromCache, because they're
protected not public methods. More importantly, make them return Futures.
Plus a little more cleanup.

These are small changes in preparation to the more interesting patch which
addresses bug 402903.

BUG=402903
TBR=yoz@chromium.org

Review URL: https://codereview.chromium.org/522183002

Cr-Commit-Position: refs/heads/master@{#292797}

https://skia.googlesource.com/skia/+log/6d134201a90692b0c7a1c0c4a97c894dae7b2158..7843a24ab2c2dc4fe77e82b9689cde953d6abd7c

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=djsollen@google.com

Review URL: https://codereview.chromium.org/521143003

Cr-Commit-Position: refs/heads/master@{#292796}

BUG=none
R=mukai@chromium.org

Review URL: https://codereview.chromium.org/527473002

Cr-Commit-Position: refs/heads/master@{#292795}