GitLab

Revert of Update V8 to version 4.3.10. (patchset #1 id:1 of https://codereview.chromium.org/965873003/)

Reason for revert:
Speculative revert to see whether this affects some failing tests on linux:

http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29%2832%29

The two other Chromium commits in the narrowed-down range were reverted but that didn't help.

Original issue's description:
> Update V8 to version 4.3.10.
>
> Summary of changes available at:
> https://chromium.googlesource.com/v8/v8/+log/27a38796..627ffe9a
>
> Please follow these instructions for assigning/CC'ing issues:
> https://code.google.com/p/v8-wiki/wiki/TriagingIssues
>
> TBR=machenbach@chromium.org
>
> Committed: https://crrev.com/a4c7ab8af2530c04fb32939585d3b021aa7d9363
> Cr-Commit-Position: refs/heads/master@{#318556}

TBR=machenbach@chromium.org,v8-autoroll@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

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

Revert of Revert of Add a new enable_nonblocking_io field trial param for QUIC (patchset #1 id:1 of https://codereview.chromium.org/970513004/)

Reason for revert:
Revert apparently didn't help, still failing:
http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/39616

Original issue's description:
> Revert of Add a new enable_nonblocking_io field trial param for QUIC (patchset #6 id:120001 of https://codereview.chromium.org/938003003/)
>
> Reason for revert:
> Speculative revert to see whether this affects some tests that involve loading fonts in gfx_unittests (see failures at http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29).
>
> Original issue's description:
> > Add a new enable_nonblocking_io field trial param for QUIC which enables the new non-blocking IO support on windows.
> >
> > BUG=442392
> >
> > Committed: https://crrev.com/83e8c1477cf5d41928b8c9035f0c4812828752f4
> > Cr-Commit-Position: refs/heads/master@{#318555}
>
> TBR=rtenneti@chromium.org,rch@chromium.org
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=442392
>
> Committed: https://crrev.com/369bca8b735e6275616052b9b9e3755f3305eb1f
> Cr-Commit-Position: refs/heads/master@{#318601}

TBR=rtenneti@chromium.org,rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=442392

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

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

Revert of Add a new enable_nonblocking_io field trial param for QUIC (patchset #6 id:120001 of https://codereview.chromium.org/938003003/)

Reason for revert:
Speculative revert to see whether this affects some tests that involve loading fonts in gfx_unittests (see failures at http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29).

Original issue's description:
> Add a new enable_nonblocking_io field trial param for QUIC which enables the new non-blocking IO support on windows.
>
> BUG=442392
>
> Committed: https://crrev.com/83e8c1477cf5d41928b8c9035f0c4812828752f4
> Cr-Commit-Position: refs/heads/master@{#318555}

TBR=rtenneti@chromium.org,rch@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=442392

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

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

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

Reason for revert:
Revert did not help, restoring original patch

Original issue's description:
> 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=
>
> Committed: https://crrev.com/6197d8798bb0282d42b69c535b17bd2cd92c3b38
> Cr-Commit-Position: refs/heads/master@{#318581}

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

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

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

This change splits out the targets being used to track the gn
migration from  all.gyp into a separate, new file
(gn_migration.gypi), and reworks their structure to

  1) better match the gn_all target in src/BUILD.gn,
  2) identify the targets built by 'all' in gyp that
     are not yet explicitly listed in the gn_all target
     (the new 'add_to_gn_all' target),
  3) identify the targets left to be converted.

The 'add_to_gn_all' targets will simply be moved to gn_all in
a follow-up CL.

Right now these targets will be linux-only, but follow-up CLs
will expand them to the other configurations.

This CL should produce no actual changes in the builds.

R=brettw@chromium.org
BUG=461019
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:android_chromium_gn_compile_dbg,android_chromium_gn_compile_rel;tryserver.chromium.win:win8_chromium_gn_rel,win8_chromium_gn_dbg;tryserver.chromium.mac:mac_chromium_gn_rel,mac_chromium_gn_dbg

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

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

This CL adds some VLOG to BluetoothMediaClient,
BluetoothMediaTransportClient, BluetoothMediaEndpointServiceProvider
and BluetoothAudioSinkChromeOS.

BUG=429016
TEST=None

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

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

remaining instrumentation of non-janky functions.

BUG=418183,462774,462778,462780,462784,462788,462789
TEST=none
TBR=rvargas,jochen,rpaquay,zea

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

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

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}