GitLab

BUG=393291

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

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

So far, EarlyOut_DeferCommits is controlled by Proxy.
Because Scheduler doesn't know about this deferring, it triggers next
BeginMainFrame when main thread want to defer a commit.
This CL moves the EarlyOut_DeferCommits logic from Proxy to
Scheduler so Scheduler can stop next BeginMainFrame until defer commit is off.

R=brianderson@chromium.org, danakj@chromium.org, skyostil@chromium.org
BUG=382572
TEST=cc_unittests

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

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

R=tsepez@chromium.org

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

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

When sharing textures in mailboxes using MailboxManagerSync
there's an optimization that if textures are not updated then
there is no need to share it again. However the code that
checks whether textures are updated ignores the cleared
state, causing shared texture to remain uncleared
indefinitely.

Fix by check the cleared state. And add a unit test.

BUG=453199

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

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

Accelerated canvas and GPU raster both use the same Skia codepaths and
have generally displayed the same bugs.  It doesn't make sense to
blacklist one but not the other.  Since GPU raster sees more usage than
canvas nowadays, and I researched GPU raster compatibility as part of
https://codereview.chromium.org/760053003, it makes sense to fold canvas
into the same blacklist.

This immediately fixes an Adreno 225 canvas bug which slipped through a
driver-reported-version based blacklist, which is apparently unreliable
and which I deleted in this patch.  More generally it also should avoid
this kind of issue in the future.

BUG=453283
NOTRY=true

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

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

When "Choose what to sync" is clicked, we change the source to 3 in the
gaia continue url.

BUG=452440

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

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

BUG=449409

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

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

Those are not more used in chrome/browser and thus can be removed to
prevent others using them in chrome/browser land.

BUG=237267
TEST=checkdeps
R=thestig@chromium.org

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

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

This adds controls to the C++ test server but doesn't use them yet. Also
ensure that tlslite never selects a 1.2 cipher suite when negotiating an
older version.

BUG=450733

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

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

GetWindowObject() get called during plugin initialization, Ignore that.

BUG=451197

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

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

This CL handles the case when a token is revoked, e.g. via
https://security.google.com/settings/security/permissions.

We need to instruct chrome.identity to remove the invalid cached
token and fetch a new one.  The app would then be reloaded app as
some of the app components (wcs sandbox) cached the invalid token
and cannot be cleanly reinitialized.

Summary of changes:
1. Implement app reloading using chrome.app.AppWindow API's and
   base.Ipc so that it can be called from foreground pages
2. Implement handleAuthFailure in remoting.identity and remoting.oauth2.
   In remoting.identity, we would remove the cached token and request a
   new one by prompting the user for consent.
3. Modify the call sites to use the new method.

BUG=339677

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

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

This will allow hardware video overlays to be enabled in the compositor

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/5444393..19d8f9f

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

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

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

Revert of Optimize all Chrome for Android PNG assets. (patchset #1 id:1 of https://chromiumcodereview.appspot.com/889683002/)

Reason for revert:
Images got corrupted

Original issue's description:
> Optimize all Chrome for Android PNG assets.
>
> This change decreases our APK size by 67,065 bytes.
>
> R=newt@chromium.org
> TBR=tedchoc@chromium.org
> BUG=407781
>
> Committed: https://chromium.googlesource.com/chromium/src/+/3952cc4232da78710734333340ae3181d5c20031

TBR=newt@chromium.org,tedchoc@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=407781

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

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

Also add a TODO requested by Nico.

BUG=453195
R=thakis@chromium.org
CC=fdegans@chromium.org

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

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

BUG=440443

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

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

Currently we fetch the average character width from the GetTextExtentPoint32 API for DirectWrite as skia
does not return this value. The width returned by the GetTextExtentPoint32 API is 1 px larger than the one
reported by the TEXTMETRIC structure causing some dialogs like the bookmark editor to display much larger than that
with GDI.

We now use the average character width from the TEXTMETRIC structure while reporting back the metrics for DirectWrite
fonts. Longer term fix is to find out a way to calculate this from DirectWrite. Does not appear to be straigthforward.
Added a TODO to that effect in the code.

BUG=434588
TEST=Updated the PlatformFontWinTest.Metrics_SkiaVersusGDI test to validate the average character widths.
For manual test, launch Chrome and open the bookmark
editor on Windows 7+. Observe that the window size is larger than the window size when using GDI for fonts.

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

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

To match LTHI, change the max frames pending calculation to output DEFAULT_MAX_FRAMES_PENDING if the output surface reports 0.

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

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

This is the first step in making more reliable cross-machine history synching. A subsequent step will support reading data from the history files from other machines.

Also, updated the debug log to employ machine id.

BUG=420680
TEST=browser_test: FileManagerJsTest.*

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

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

Now chrome_version.cc  and version .rc files are generated.

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

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

This test was expecting the main thread copy request to happen before a
new draw happens on the compositor thread which is racey. Instead do
this comparison on the main thread by posting and trying to end the test
after the draw happens but checking for the copy request result there
on the main thread instead of on the impl thread before posting.

R=vmpstr, weiliangc
BUG=439649

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

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

It was failing like this:

In file included from ../../media/mojo/services/mojo_cdm_service.cc:5:
../../media/mojo/services/mojo_cdm_service.h:71:40: error: [chromium-style] WeakPtrFactory members which refer to their outer class must be the last member in the outer class definition.
  base::WeakPtrFactory<MojoCdmService> weak_factory_;
                                       ^
BUG=444089

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

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

BUG=444586
TEST=
1. Change language to RTL language
2. Set chrome://flags/#enable-permissions-bubbles to 'Enabled'
3. Go to https://adrifelt.github.io/demos/bubble-demo.html
4. Verify that URL is displayed correctly

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

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

This was added for use in mojo and is still being used in the mojo
repo but is not in chromium.

R=thakis@chromium.org

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

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

This mirrors an earlier change to non-swarming runs:
https://codereview.chromium.org/810643006/

BUG=372458
R=vadimsh@chromium.org

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

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

IAccessible objects are reference-counted, so the COM object
may persist if the view is deleted. We should always check the
view (which will be nulled when the view is deleted).

BUG=436074

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/05dd7de..a8902e8

TBR=rjwright@chromium.org,junov@chromium.org

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

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

R=tsepez@chromium.org,mbarbella@chromium.org
BUG=450268

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

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

TBR=machenbach@chromium.org

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

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

This fixes a regression vs X11 where all touch events are missing
modifier flags.

BUG=450650
TEST=Open http://www.rbyers.net/eventTest.html on link, hold shift, and
     tap the touchscreen. Page shows "shift" on the click event line.

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

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

Revert of Fix shelf dimming on secondary display (patchset #1 id:40001 of https://codereview.chromium.org/850833004/)

Reason for revert:
Causing crbug.com/451804 - probably using an invalid window

Original issue's description:
> Fix shelf dimming on secondary display
>
> BUG=449863
> TEST=ShelfLayoutManagerTest.DimmingBehaviorDualDisplay
>
> Committed: https://crrev.com/665d66d93fe22a50ba3623397b596a394098faef
> Cr-Commit-Position: refs/heads/master@{#312244}

TBR=skuhne@chromium.org,pkotwicz@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=449863

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

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

Instead of using an AtomicSequenceNumber to find an ID for a worker runloop,
use the ID of the current thread instead.

BUG=none
R=jochen@chromium.org

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

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

Revert of Allow new zoom bubble to override current one. (patchset #6 id:100001 of https://codereview.chromium.org/849693006/)

Reason for revert:
The changes in this CL are causing a crash: crbug.com/453564

Original issue's description:
> Allow new zoom bubble to override current one.
>
> At present, if a user changes zoom on OSX while the zoom bubble is
> already being shown, the new zoom bubble (with the most up to date
> information) is not shown. This CL changes this behaviour by allowing
> a new zoom bubble to be created, and replace the existing one.
>
> Example: load a PDF and repeatedly click the zoom +/- controls in the
> lower-right corner of the viewer.
>
> BUG=444995
>
> Committed: https://crrev.com/a95ea7031e472fd5d0968955989c963381dfaf5f
> Cr-Commit-Position: refs/heads/master@{#313564}

TBR=rohitrao@chromium.org,erikchen@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=444995

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

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

Changes:

a87de9b Define WIN32_LEAN_AND_MEAN in the build system, not in the code.
4a0f0c4 Change CMakeLists.txt to two-space indent.
2e38008 Define _HAS_EXCEPTIONS=0 to avoid MSVC C4530 warning.
bcc4fa8 Add unit-tests for PBKDF2.
507c1ee Honor the standard BUILD_SHARED_LIBS cmake variable.
f4600ad Add missing OPENSSL_EXPORTs.
f058dae Revert "Add a test for CONF_parse_list."
dc94b54 Clean up use of Windows Platform SDK headers.
33970e6 Enable bssl client/s_client and server/s_server on Windows.
9a9a193 Build with -fvisibility=hidden.
afdaeee Enable bssl (md5sum, sha256sum, etc.) on Windows.
2fe7f2d Initialize HMAC keys to zero.
1d75c8b Improve compatibility with CMake 3.0 and later
ed30c0d Improve the documentation in BUILDING
b7f3144 Add a style guide.
69ad27f Run x86 perlasm on UNIX with -fPIC.
36eb7d5 Fix buffer size in aead_test.c.
53cbd6c Fix unused variable warning in OPENSSL_NO_ASM build.
bed8f73 Fix build after mistake addressing comments in cca4d599.
cca4d59 Add digest sum handling to the tool.
cd5c892 Add a test for CONF_parse_list.
fd4eb60 Remove duplication of common DH parameters.
b4a494c Reformat SSL/TLS headers.

For more details see: https://boringssl.googlesource.com/boringssl/+log/347f025..a87de9b

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

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

Main thread can scroll with non-integral scroll offset, but Blink currently
only handles integral part of the scroll offset. For example, if main thread
scrolls by 10.5, Blink tells CC that the scroll offset is 10 (currently truncated
inside Blink) and Blink also sets fixed-position layer's position at 10.

This CL makes the non-integral scroll offset works correctly in this case:
1. Blink can tell CC the non-integral scroll offset 10.5 (truncation inside Blink
will be removed after this CL).
2. Blink sets fixed-position layer at 10.
3. CC tracks the fractional part of the scroll offset 0.5 and applied it when
computing scroll compensation for fixed-position layer during drawing.

BUG=414283

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

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

Windows defines GetMessage as GetMessageW.
This makes linking fail if p224_spake included after windows headers.

NOPRESUBMIT=true

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

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

This...
- Wires up ExtensionView. Can apply src for navigation via inline and js call.
- Adds permission for manifests.
- Add browser tests for navigation and adding ExtensionViews to DOM.

Some code is duplicated from existing GuestViews (e.g. *ViewAttribute and some GuestViewBase implementations). Based on how GuestView code will be refactored, some of this can be consolidated.

This...
- Does not run in a detached state. (Refers to CR 845363004)
- Will have more functionality and restrictions implemented in the future.

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

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

Revert of Revert of IPC::ChannelMojo: Make IPC handling robust against bad messages. (patchset #1 id:1 of https://codereview.chromium.org/883373002/)

Reason for revert:
Wrong revert, ipc fuzzer broke because of something else.

Original issue's description:
> Revert of IPC::ChannelMojo: Make IPC handling robust against bad messages. (patchset #3 id:40001 of https://codereview.chromium.org/725733002/)
>
> Reason for revert:
> Broke ipc fuzzer
>
> Original issue's description:
> > IPC::ChannelMojo: Make IPC handling robust against bad messages.
> >
> > This change replaces some DCHECK()s with real error handling.
> > They happen when it receives broken messages.
> >
> > BUG=428800
> > R=viettrungluu@chromium.org
> >
> > Committed: https://crrev.com/185ffdf58f85e47db9666d2ad9b80ff9bd7fa54b
> > Cr-Commit-Position: refs/heads/master@{#304102}
>
> TBR=viettrungluu@chromium.org,morrita@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=428800
>
> Committed: https://crrev.com/726621f93d2802dfc2c90732c7133145c9854e38
> Cr-Commit-Position: refs/heads/master@{#313778}

TBR=viettrungluu@chromium.org,morrita@chromium.org,tsepez@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428800

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

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

This is a prerequisite change for supporting variable video frame sizes
in Cast.  This allows a follow-up change to remove the width/height
properties from VideoSenderConfig.  Instead, the width/height is passed
to the ctor in a separate gfx::Size argument.

BUG=451277

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

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

Revert of IPC::ChannelMojo: Make IPC handling robust against bad messages. (patchset #3 id:40001 of https://codereview.chromium.org/725733002/)

Reason for revert:
Broke ipc fuzzer

Original issue's description:
> IPC::ChannelMojo: Make IPC handling robust against bad messages.
>
> This change replaces some DCHECK()s with real error handling.
> They happen when it receives broken messages.
>
> BUG=428800
> R=viettrungluu@chromium.org
>
> Committed: https://crrev.com/185ffdf58f85e47db9666d2ad9b80ff9bd7fa54b
> Cr-Commit-Position: refs/heads/master@{#304102}

TBR=viettrungluu@chromium.org,morrita@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=428800

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

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