GitLab

This further shifts Data Reduction Proxy code from using Params to using
Config.

BUG=452773

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/dd9c584..e51aa6f

TBR=hclam@chromium.org,alancutter@chromium.org

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

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

Removes a compile-time switch and associated logic that was necessary in
the pre-Aura days where content change signals were not reliable.

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

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

BUG=455423,455469,455942,456327,456331,456373,457517,457525
TEST=none
TBR=jochen,mmenke

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

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

Revert of Test PostMessage to popup window (patchset #7 id:110001 of https://codereview.chromium.org/964753002/)

Reason for revert:
This landed soon after https://crrev.com/65b9392697befefd009dbe73d75dfe66894d814c which changed the signature of setWebEventHandler

As a result, it broke the linux build bot.
http://build.chromium.org/p/chromium.linux/builders/Android%20Arm64%20Builder%20%28dbg%29/builds/10739

Original issue's description:
> Test PostMessage to popup window
>
> BUG=460998
>
> Committed: https://crrev.com/51c1620185396384cfe6e2f4e3359e57c6431600
> Cr-Commit-Position: refs/heads/master@{#318591}

TBR=sgurun@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=460998

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

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

BUG=460998

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

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

It's sometimes hard to tell which test is being retried because
tests are usually run in parallel among many devices. So just print the
name of test of device to make debugging easier.

BUG=

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

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

Provide the mechanism to allow applications to provide their own handler
for receiving postmessages on a thread separate than UI thread.

BUG=393291

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

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

QUIC - use a range-based for loops. Sync'ing with the internal tree.

Merge internal change: 86845495
https://codereview.chromium.org/963333002/

Deprecate FLAGS_quic_record_send_time_before_write.

Merge internal change: 86804950
https://codereview.chromium.org/965873004/

QuicConnection no longer owns the debug visitor. No behavior change.

The visitor_ isn't owned by the connection, this makes the
debug_visitor_ ownership model the same.

Merge internal change: 86801250
https://codereview.chromium.org/961173003/

Cleanup of QUIC's TransmissionInfo constructor to make more fields const
and supply them in the constructor.  No functional change.

Merge internal change: 86734009
https://codereview.chromium.org/968493002/

Fixing a bug in the QUIC prober where if we received a GOAWAY before
stream creation it would result in a crash.

I verified that given our improper client, the prober would crash
as-was, and would fail cleanly with the nullptr check but the client not
ignoring goaways.

Test changes and quic server changes.

Merge internal change: 86703503
https://codereview.chromium.org/965943002/

Remove dead method QuicUnackedPacketMap::RestoreInFlight.

Merge internal change: 86690324
https://codereview.chromium.org/960913004/

Deprecating FLAGS_quic_attach_ack_notifiers_to_packets.

Merge internal change: 86688595
https://codereview.chromium.org/969443002/

Remove RevertRetransmissionTimeout from QUIC's SendAlgorithmInterface
now that it's no longer used by QuicSentPacketManager.

Merge internal change: 86651704
https://codereview.chromium.org/963293002/

Remove stats for recording reno vs cubic mode in internal server's varz.

Removed now that we know how much cubic mode is in play.

Merge internal change: 86629919
https://codereview.chromium.org/964483005/

R=rch@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/748e82f..dd9c584

TBR=hclam@chromium.org,alancutter@chromium.org

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

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

host_gcc.mk was already honoring this variable, as was
the SO_RULE in nacl_gcc.mk but the LINK_RULE for nacl_gcc.mk
and nacl_llvm.mk was ignoring LDFLAGS.

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

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

The bots still use 'cpu_arch' to set the architecture, and the build
file was keying off of build_cpu_arch to determine whether or not
to enable NaCl, so things were failing. Updating the the code to
'current_cpu' fixes the problem correctly.

TBR=brettw@chromium.org

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

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

This was inspired by jank in the RDH method that calls the function,
but will almost certainly have no effect on the jank.

Also switch to using a range loop in GetLoadState as well.

BUG=455952

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

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

remaining instrumentation of non-janky functions.

BUG=455884,462811,462812,462813,462815
TEST=none
TBR=mmenke

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

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

TODO: apply the same logic (as well as the suggestion hiding logic) to addresses

BUG=459741

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

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

Revert of SK_DEBUG when DCHECK_ALWAYS_ON (patchset #1 id:1 of https://codereview.chromium.org/966593002/)

Reason for revert:
May have caused FontRenderParamsTest failures on Linux debug bots.

Original issue's description:
> SK_DEBUG when DCHECK_ALWAYS_ON
>
> BUG=
>
> Committed: https://crrev.com/879405ae813ae4a9bcbd7684d3551420d4980383
> Cr-Commit-Position: refs/heads/master@{#318557}

TBR=jam@chromium.org,mtklein@google.com,mtklein@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=

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

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

Sorts sources blocks as part of gn format. This ended up being more
complicated than I expected because of the subtlety of header comments,
and because no other formatting reorders expressions at the moment.
But, there seems to be quite a lot of unsorted lists per below, so I
guess it's a good thing to have.

Application to tree at https://codereview.chromium.org/960413003.

R=brettw@chromium.org
BUG=348474, 456014

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

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

Summary of changes:
1. Fix a bug in identity.js that breaks first run scenario of the v2 app,
   Previously, we are resetting the deferred object (the list of listeners)
   when auth is failed even when interactive is set to false, which always
   happened on first runs.  At a result, when auth is completed, the listeners
   are not notified.
   This CL only clears the deferred object after it has notified its listeners.
2. Fix a missing parameter in session_connector_impl.js that causes It2Me to fail
   to connect.
3. Changes it2me_browser_test.js to match the changes in the identity API.

BUG=462708
TEST=Browser test passes on swarming bots.
https://chromium-swarm.appspot.com/user/task/25e6711d0deacc10
NOTRY=true

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

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

This CL changes every place a MessagePort is transferred between process to
properly propagate the send_messages_as_values flag, as well as makes it
possible for NavigatorConnectContext to set this flag when setting up a
new connection.

This is part of a series of changes:
[1/5] Blink side changes in https://codereview.chromium.org/924983002/ that expose a needed v8 context
[2/5] https://codereview.chromium.org/921013002/ allow MessagePort to send messages as values
[3/5] This CL
[4/5] https://codereview.chromium.org/938403005/ which uses this for navigator.connect services and adds infrastructure for layout tests
[5/5] https://codereview.chromium.org/940423004/ adds layout tests to blink to test these new features

BUG=426458

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

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

WebRTC 8509:8540
Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/976aecb..a22bcca

Libjingle 8510:8539
Changes: https://chromium.googlesource.com/external/webrtc/trunk/talk.git/+log/e574ba0..b87b283

TBR=magjed@chromium.org

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

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

It is possible to have one DocumentActivity create a Tab and add its ID to the
DocumentTabModel, but get interrupted (somehow) before the Tab Entry is added
to the DocumentTabModel.  This results in getCount() returning the correct number
of Tabs that exist but fail to have any information about the new Tab, resulting
in getTabAt() returning |null|.  We should instead keep the ID list and the Entry
list be as in sync as possible, adding the Entry directly to the DocumentTabModel
as soon as we let the DocumentTabModel know about the Tab's existence.

Add new methods to tie the addTabId and updateEntry functions together via addTab,
deprecate the old ones, and assert that we should never get into a bad state.
Deprecated methods will be culled when the downstream patch lands.

BUG=457427

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

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

BUG=447208
NOPRESUBMIT=true

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

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

Application of https://codereview.chromium.org/962003002/.

TBR=keybuk@chromium.org
R=brettw@chromium.org
BUG=456014

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

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

```-------
  Show Autofill/Wallet options only when signed in.

  Before this patch, you could still see the options if you signed into a whitelisted account (setting the sync experiment preference) and then signed out of the account.

  BUG=none
  R=brettw@chromium.org
```

-------

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

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

On the first 401 Unauthorized, invalidate the oauth token and try again. If there is more than one 401 Unauthorized in a row, we're probably borked, so don't keep retrying.

BUG=none

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

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

BUG=462407

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

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

Perf try jobs running on Win x64 bisect bots should build and run tests on x64 binaries.

BUG=462639

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

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

When running ppapi_simple-based application in sel_ldr
we almost never want to use nacl_io so don't initialize
it by default.  This makes it easier for application build
on ppapi_simple to run both under and Chrome and sel_ldr.

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

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

BUG=462239

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/6c51e22..748e82f

TBR=hclam@chromium.org,alancutter@chromium.org

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

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

Wallet was not receiving invalidation previously because the Android side
did not know to register the type. This fixes that.

BUG=none

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

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

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

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

We interchangeably used probe and canary throughout the Data Reduction
Proxy component; neither is very descriptive about what is taking place.

SecureProxyCheck is sufficiently descriptive but succinct.

BUG=460299

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

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

BUG=461137

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

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

This class is a common base class for all Telemetry exceptions.

BUG=460625

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

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

My recent refactor (http://crrev.com/879533004) modified
AndroidSyncSettings to only read the account when it changed, but did
not correctly initialize the account on startup.

BUG=462658,462618

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

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

    This is only available on Windows 8+ because the media foundation API which exposes the device
    manager to be pased to the decoder MFCreateDXGIDeviceManager only exists on Windows 8+.

    Changes in this patch are mostly around using DX11 or D3D wherever needed. These are as below:-

    1. In the initialization code path where we use D3D or DX11 based on whether we are on Windows 8+
       and ANGLE and the decoder say they support DX11.

    2. The output frame processing code where we extract the DX11 texture and copy it out to ANGLE. One change
       here is that DX11 does not provide an automatic way for format conversion for textures. The decoder outputs
       YUV12 textures and ANGLE expects RGB. We can achieve this by setting a shader for conversion. That seemed
       like too much work. Thankfully there is a video processor media foundation transform on Windows which
       does the conversion for us on the GPU. We use this object to convert the output frame and copy it out to
       ANGLE.

    3. We pass the GL context to the decoder to enable us to query ANGLE to see if it is using D3D or DX11.

    BUG=456418

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

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

Don't send frames with show_frame syntax element set to false to display.

BUG=chrome-os-partner:35953
TEST=video playback, vdatest

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

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

BUG=461494
TEST=Crash report should show that --site-per-process flag is present.

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

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

BUG=

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

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