GitLab

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/3a586236..1b50b896

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

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

BUG=537412

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

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

BUG=535323,572777

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

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

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

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

https://chromium.googlesource.com/chromium/src/+/c2a7942a1

BUG=572822
TBR=xidachen@chromium.org

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

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

gles2_conform_test isn't present yet.

BUG=542370
TBR=dpranke@chromium.org,phajdan.jr@chromium.org

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

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

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

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

Picks up https://codereview.chromium.org/1557633002/

BUG=573250
TBR=jiayl@chromium.org

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

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

BUG=566085

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

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

Previous CL (crrev.com/1386873003) made MHTML serialization work in
presence of out-of-process iframes, but can potentially include the same
MHTML part twice.  MHTML with duplicated parts can be rendered by Chrome
and Internet Explorer without problems (at least according to the
limited ad-hoc testing I've done), but the unnecessary increase of the
file size is a performance regression.

After the current CL, the browser keeps track of already serialized
MHTML parts and communicates them to the renderer/serializer for
each frame (so they can be deduped).  This fixes the issue pointed
out above.

BUG=538766

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

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

1. Use int instead of int64.
2. Define ENABLE_BIDIRECTIONAL_STREAM on net_unittests target.

BUG=516342

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

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

When we transition to RunningOnCompositorButNeedsUpdate, we need to call
registerForAnimation again to ensure that updateCompositorAnimations is called.
We also cannot rely on m_lastTickTime since we do not get tickAnimation for
composited scrolls.

BUG=571680

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

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

This change adds the following metrics:
- Count of times the help page is opened.
- Count of times the user explicitly navigates from the route details to sink list.
- Count of times the user explicitly navigates from the sink list to route details.
- Count of times the user explicitly navigates from the sink list to cast mode picker.

BUG=547977

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

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

(╯^□^)╯︵ ❄☃❄

BUG=557422
R=avi@chromium.org
TBR=pasko@chromium.org

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

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

Similar to r367111, send the reply to software draw in swap as well, and
allow PrepareTiles to run in parallel.

BUG=545633

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

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

The existing division-by-0 guard was not correct, since MaximumDimension can
return negative values.  This led to occasional wild jumps when scrolling up and
down quickly (because new_velocity had the wrong sign).

This patch addresses the sign issue and improves handling of cases where we have
a large current velocity and a small delta to the new target.

BUG=575
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

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

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

https://chromium.googlesource.com/skia.git/+log/e3c2f80006e7..f880e45e92fe

$ git log e3c2f8000..f880e45e9 --date=short --no-merges --format='%ad %ae %s'
2015-12-30 reed change factory to method, to parallel pattern in imagefilters. no functionality change.

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
TBR=mtklein@google.com

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

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

ALIGN_CENTER is already default for STYLE_BUTTON.

BUG=none

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

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

BUG=570113

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

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

Should fix:
../../buildtools/third_party/libc++/trunk/include/map:843:5: error: static_assert failed
    "Allocator::value_type must be same type as value_type"
    static_assert((is_same<typename allocator_type::value_type, value_type>::value),
    ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
../../components/metrics/leak_detector/leak_analyzer.h:69:55: note: requested here
           Allocator<std::pair<ValueType, uint32_t>>> suspected_histogram_;

BUG=573250

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

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

Picks up https://codereview.chromium.org/1557653002/

BUG=573250
TBR=thestig

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

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

This occurred because a ChildBrokerHost noticed that its connection to the child process died and so it nulled out its channel. On POSIX, this leads to the destruction of the ChildBrokerHost immediately and so the singleton BrokerState is notified synchronously. On Windows, there's another channel for sandboxing related sync IPCs which uses IO completition ports. To ease lifetime issues because of that, the ChildBrokerHost is only deleted in response to errors from that second channel. So if the first channel dies first, it is possible that BrokerState tries to connect to a message pipe in that ChildBrokerState with a null channel.

The fix is to null check the channel and to inform the other side of the message pipe that its peer has died so that it can fire peer closed notifications.

BUG=573244

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

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

(╯^□^)╯︵ ❄☃❄

BUG=557422
R=avi@chromium.org
TBR=ddorwin@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/73c62df3..3a586236

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

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

FPDFAvail_IsFormAvail() must be called before the PDF form is
initialized if new data was downloaded since the document was created.

BUG=572943

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

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

It's no longer used.

BUG=none
TEST=none
R=ben@chromium.org
TBR=ben@chromium.org

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

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

BUG=573259,573250

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

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

In a previous CL: https://codereview.chromium.org/1502283002/, this equation
was changed and now when the blur radius is 2px the blur effect completely
disappears. This CL change the equation back to the original one but add
a special case handler so that when the blur radius is 0 there is no blur
effect at all.

Also, together with this change, a bunch of layout test will need to
rebaseline.

BUG=572822

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

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

BUG=570955
TBR=sullivan@chromium.org

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

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

Instead of casting Numbers to Integer or Long (which doesn't work), we
use longValue() and intValue().

BUG=529962

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

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

BUG=568706
R=eae@chromium.org

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

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

This change adds the following metrics:
- The type of route when the user closes it.
- The type of cast source when the user picks one.
- The index of the sink that the user selected in the sink list.

BUG=547977

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

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

BUG=chromium:573287

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

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

Suppport to override existing locale resources in dep graph for
android_apk targets.

- Used is_locale_resource in build_config file to indicate that
  android_resources is locale_resource

- Removed all locale_resource in deps graph if
  alternative_locale_resource_dep is specified in android_apk target.

This feature was implemented in gyp.

BUG=565532

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

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

Chromium does not support PDF content in <img> tags, which this test uses.
Hence we are removing the test.

TBR=chrishtr@chromium.org
BUG=365400

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

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

Picks up:
gn roll https://codereview.chromium.org/1547563004/
libcxx/libcxxabi roll: https://codereview.chromium.org/1559483002/
Revert gn roll: https://codereview.chromium.org/1559513002/
Fix libcxx build: https://codereview.chromium.org/1554723003/

BUG=573250
R=thestig@chromium.org
TBR=agrieve

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/c80d154e..73c62df3

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

TBR=hablich@chromium.org,machenbach@chromium.org,yangguo@chromium.org,vogelheim@chromium.org

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

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

This was used by the legacy chromium ARM builder and
is not longer needed (AFAICT).

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

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

Pass() is a legacy of the original C++03 move emulation implementation.
New code should use std::move from <utility>. ChromeOS is exempt until the
migration there is complete.

❆(੭ु ◜◡‾)੭ु⁾☃❆

BUG=557422
R=thestig@chromium.org

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

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

* Adds InfoBarDelegate::GetIdentifier(), which returns an
  enum identifying the InfoBar being created.  Generally,
  this is the name of the InfoBarDelegate, but in cases of
  classes using SimpleAlertInfoBarDelegates (or other wrapper
  classes), it uses the name of the class using the delegate.

* Adds a histogram that records which infobars were shown to
  the user via InfoBarContainer#AddInfoBar().

BUG=567483

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

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