GitLab

I added parentheses to the frame calculation expression to work around
clang-format issues, but I added them in the wrong place and made the math
wrong.  This CL removes the parentheses and restores the original expression.

BUG=675595

Review-Url: https://codereview.chromium.org/2590223002
Cr-Commit-Position: refs/heads/master@{#439813}

Creating a wrapper might fire events synchronously which in turn can
modify the wrapper. There is, however, an invariant for interceptors
that if they modify their holder, they need to signal this. Therefore,
when we create a wrapper, always signal that the write happened.

BUG=670155
R=tkent@chromium.org,haraken@chromium.org

Review-Url: https://codereview.chromium.org/2566663002
Cr-Commit-Position: refs/heads/master@{#439812}

The HTMLMediaElement no longer adds the MediaControls to the shadow tree
in order to prepare for the class to be an interface implemented by
modules. This CL also removes a couple of reset() call.

BUG=662761
R=zqzhang@chromium.org

Review-Url: https://codereview.chromium.org/2588823002
Cr-Commit-Position: refs/heads/master@{#439811}

Revert of Maintain HostZoom connection per-frame on browser side (patchset #4 id:100001 of https://codereview.chromium.org/2581143002/ )

Reason for revert:
Seems to cause problems on Mac:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.webkit%2FWebKit_Mac10.10%2F28001%2F%2B%2Frecipes%2Fsteps%2Fwebkit_tests%2F0%2Fstdout

Original issue's description:
> Maintain HostZoom connection per-frame on browser side
>
> HostZoomMapObserver is a per-WebContents class, while the HostZoom Mojo
> connection is per-frame. Before this CL, HostZoomMapObserver was
> maintaining one HostZoom connection and rebinding it every time a new
> RenderFrame was created. This meant that HostZoomMapObserver was
> continually losing connections to existing frames. This CL changes
> HostZoomMapObserver to maintain one connection per-RenderFrame in a
> map indexed by the corresponding RenderFrameHost.
>
> BUG=673065
> TEST=Visit news.ycombinator.com and increase the zoom level to 175%.
> Click the top link. Hit back: news.ycombinator.com should still be
> zoomed to 175%.
>
> Committed: https://crrev.com/8e24b7d133424fb221891601d6254b7df58188d5
> Cr-Commit-Position: refs/heads/master@{#439800}

TBR=scottmg@chromium.org,wjmaclean@chromium.org,nick@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=673065

Review-Url: https://codereview.chromium.org/2595603002
Cr-Commit-Position: refs/heads/master@{#439810}

TBR=eae,kojii

Review-Url: https://codereview.chromium.org/2590213002
Cr-Commit-Position: refs/heads/master@{#439809}

BUG=648557

Review-Url: https://codereview.chromium.org/2586363002
Cr-Commit-Position: refs/heads/master@{#439808}

Remove a TODO that I left before because I confirmed that
current code is correct, and chaging it makes many unit tests
fail.

BUG=670232

Review-Url: https://codereview.chromium.org/2587323002
Cr-Commit-Position: refs/heads/master@{#439807}

Now, core/fetch has a dependency to bindings/core/, but this won't be
permitted once the directory is moved to platform/loader/fetch.

This patch moves this last unexpected dependency, and make it
finally possible to move under platform/.

BUG=655920

Review-Url: https://codereview.chromium.org/2578983002
Cr-Commit-Position: refs/heads/master@{#439806}

This were reviewed upstream in https://github.com/w3c/csswg-test/pull/1155

This should reduce the noise in https://codereview.chromium.org/2450093005, and
make it more manageable and straight-forward to review.

BUG=657748

Review-Url: https://codereview.chromium.org/2588093003
Cr-Commit-Position: refs/heads/master@{#439805}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/32db0e1a..54a5b164

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.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

Review-Url: https://codereview.chromium.org/2595563002
Cr-Commit-Position: refs/heads/master@{#439804}

https://skia.googlesource.com/skia.git/+log/ace343286d90..3711e1185d53

$ git log ace343286..3711e1185 --date=short --no-merges --format='%ad %ae %s'
2016-12-20 reed switch to _deprecated suffix for exotic SkClipOps

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
TBR=rmistry@google.com

Review-Url: https://codereview.chromium.org/2590203002
Cr-Commit-Position: refs/heads/master@{#439803}

BUG=670247
TEST=Manual test by checking Configuration.hardKeyboardHidden when
     entering maximize mode.

Review-Url: https://codereview.chromium.org/2590893002
Cr-Commit-Position: refs/heads/master@{#439802}

BUG=568901
TEST=ScrollingCoordinatorTest.BackgroundAttachmentFixedShouldTriggerMainThreadScroll; FrameThrottlingTest.ScrollingCoordinatorShouldSkipThrottledFrame; ScrollingCoordinatorTest.RecalculateMainThreadScrollingReasonsUponResize
third_party/WebKit/LayoutTests/compositing/layer-creation/iframe-background-attachment-fixed.html
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2531603003
Cr-Commit-Position: refs/heads/master@{#439801}

HostZoomMapObserver is a per-WebContents class, while the HostZoom Mojo
connection is per-frame. Before this CL, HostZoomMapObserver was
maintaining one HostZoom connection and rebinding it every time a new
RenderFrame was created. This meant that HostZoomMapObserver was
continually losing connections to existing frames. This CL changes
HostZoomMapObserver to maintain one connection per-RenderFrame in a
map indexed by the corresponding RenderFrameHost.

BUG=673065
TEST=Visit news.ycombinator.com and increase the zoom level to 175%.
Click the top link. Hit back: news.ycombinator.com should still be
zoomed to 175%.

Review-Url: https://codereview.chromium.org/2581143002
Cr-Commit-Position: refs/heads/master@{#439800}

PasswordBranding has launched in the form of 100% being SAVE_PROMPT_ONLY.

This CL removes the code to obtain the Finch group for the experiment, related
tests, and also the functionality only executed for the unused FULL variant of
the experiment (mainly the SmartLock warm welcome).

BUG=486739

Original Review-Url: https://codereview.chromium.org/2579823003
Original Cr-Commit-Position: refs/heads/master@{#439740}

Reverted because of breaking iOS build.

Patch set 1 here is the reverted one, subsequent patches are fixing the breakage.

TBR=mathp@chromium.org,vasilii@chromium.org,stanisc@chromium.org,stevenjb@chromium.org,dfalcantara@chromium.org,

Review-Url: https://codereview.chromium.org/2588263003
Cr-Commit-Position: refs/heads/master@{#439799}

This CL fixes a tsan data race, which is caused by calling StopFetchingData
from different threads. It must not be allowed. Use the same thread by using
a PostTask.

BUG=673760
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_tsan_rel_ng

Review-Url: https://codereview.chromium.org/2569763004
Cr-Commit-Position: refs/heads/master@{#439798}

Initialize MetadataBoxController and QuickViewController only when
it is needed.
This CL doesn't lazy load polymer element of quick view. It will be
done with subsequent CL(s).

BUG=675536
TEST=manually.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2589133002
Cr-Commit-Position: refs/heads/master@{#439797}

BUG=629512
TEST=Builds both 32 and 64 bit dlls on a 32bit build system.

Review-Url: https://codereview.chromium.org/2590433003
Cr-Commit-Position: refs/heads/master@{#439796}

MakeWeakCopies is a helper function inside the FormFetcherImpl implementation.
It transforms a vector of unique_ptrs into the result of calling get() on them.
It used to be used for pointers of different types, therefore it was a function
template. But after https://codereview.chromium.org/2552263002, MakeWeakCopies
is only used with PasswordForms.

This CL:
 * Converts MakeWeakCopies to a simple function.
 * Removes some unnecessary "autofill::" in the same file.

BUG=621355
R=vasilii@chromium.org

Review-Url: https://codereview.chromium.org/2587313002
Cr-Commit-Position: refs/heads/master@{#439795}

The rebaseline request was landed in https://codereview.chromium.org/2573803002/ (Bump CSSFilterEffects pixel test revision number to force rebaseline)

BUG=672527
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2588793002
Cr-Commit-Position: refs/heads/master@{#439794}

BUG=613465
TEST=manually.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2592593002
Cr-Commit-Position: refs/heads/master@{#439793}

When the iPad Tab Switcher creates an incognito tab for the first time, an incognito
BVC is created.
The Tab Switcher takes a screenshot of the new incognito tab, and animates its size
from a small cell to a fullscreen tab.
Before this CL, the first incognito tab was created with an incorrect size, resulting in
the screenshot used in the animation being incorrect.

BUG=671724

Review-Url: https://codereview.chromium.org/2591683002
Cr-Commit-Position: refs/heads/master@{#439792}

Review-Url: https://codereview.chromium.org/2593603002
Cr-Commit-Position: refs/heads/master@{#439791}

BUG=664838
R=mlamouri@chromium.org

Review-Url: https://codereview.chromium.org/2590183002
Cr-Commit-Position: refs/heads/master@{#439790}

Use new time conversion API in accordance with
https://codereview.chromium.org/1988663002/

BUG=601905,601903,601900

Review-Url: https://codereview.chromium.org/2090713003
Cr-Commit-Position: refs/heads/master@{#439789}

The dll will always be present but only registered when running
a system install because we need access to the LOCAL_MACHINE hive.

BUG=642115
TEST=setup_util_unittest,mini_installer,manual

Review-Url: https://codereview.chromium.org/2507753002
Cr-Commit-Position: refs/heads/master@{#439788}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/62479fdb..32db0e1a

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.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

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

Review-Url: https://codereview.chromium.org/2595513002
Cr-Commit-Position: refs/heads/master@{#439787}

Changes: https://chromium.googlesource.com/external/webrtc/trunk/webrtc.git/+log/37d7b3f..ba5e7e2

$ git log 37d7b3f..ba5e7e2 --date=short --no-merges --format=%ad %ae %s
2016-12-20 nisse@webrtc.org Delete unused method PayloadRouter::MaxPayloadLength.
2016-12-20 danilchap@webrtc.org Style cleanup in RTCPReceiver Rename variables and private functions to follow style, replace remaining asserts with DCHECKs. add 'ms' suffix to time variables derived from clock_ add 'ntp' suffix to time variables derived from ntp time. No functional changes expected.
2016-12-20 magjed@webrtc.org Revert of Add disabled certificate check support to IceServer PeerConnection API. (patchset #8 id:140001 of https://codereview.webrtc.org/2557803002/ )
2016-12-20 henrik.lundin@webrtc.org Add a unit test for Opus complexity adaptation
2016-12-19 zhihuang@webrtc.org Add an abstract class for IceTransport
2016-12-19 brandtr@webrtc.org Add GUARDED_BY's in FlexfecReceiver.
2016-12-19 brandtr@webrtc.org Clean up storage of FlexFEC payload type in webrtc::VideoCodecSettings.
2016-12-19 danilchap@webrtc.org Revert of Rename RTPVideoHeader.isFirstPacket to .is_first_packet_in_frame. (patchset #1 id:1 of https://codereview.webrtc.org/2574943003/ )
2016-12-19 hbos@webrtc.org Removed undefined method from webrtcsession.h.
2016-12-19 johan@webrtc.org Rename RTPVideoHeader.isFirstPacket to .is_first_packet_in_frame.
2016-12-19 asapersson@webrtc.org Move histogram for number of pause events to per stream:
2016-12-19 kthelgason@webrtc.org Reland of Properly report number of quality downscales in stats. (patchset #1 id:1 of https://codereview.webrtc.org/2586783003/ )
2016-12-19 henrik.lundin@webrtc.org Re-enable Opus complexity tests on Android
2016-12-19 kthelgason@webrtc.org Revert of Properly report number of quality downscales in stats. (patchset #11 id:220001 of https://codereview.webrtc.org/2564373002/ )
2016-12-19 kwiberg@webrtc.org CodecInst operator<<
2016-12-19 asapersson@webrtc.org Reland of Disabling NOTREACHED which we're hitting flakily in browser tests. (patchset #1 id:1 of https://codereview.webrtc.org/2585183002/ )
2016-12-19 brandtr@webrtc.org Add multithreaded fake encoder and corresponding FlexFEC VideoSendStreamTest.
2016-12-19 hbos@webrtc.org Fix segfault when PeerConnection is destroyed during stats collection.
2016-12-19 kthelgason@webrtc.org Properly report number of quality downscales in stats.
2016-12-19 hbos@webrtc.org RTCStatsCollector: Utilize network thread to minimize thread hops.
2016-12-19 philipel@webrtc.org Now expect the correct number of streams in EndToEndTest.GetStats.
2016-12-19 hnsl@webrtc.org Add disabled certificate check support to IceServer PeerConnection API.
2016-12-19 denicija@webrtc.org Add QP stats to the statsview in AppRTCMobile for ios.
2016-12-19 henrik.lundin@webrtc.org Revert of Re-enable Opus complexity tests on Android (patchset #1 id:1 of https://codereview.webrtc.org/2589673002/ )
2016-12-19 henrik.lundin@webrtc.org Re-enable Opus complexity tests on Android
2016-12-19 brandtr@webrtc.org Move FlexfecReceiveStream from api/call/ to call/.
2016-12-18 asapersson@webrtc.org Revert of Disabling NOTREACHED which we're hitting flakily in browser tests. (patchset #1 id:1 of https://codereview.webrtc.org/2477663002/ )
2016-12-18 kjellander@webrtc.org Move tools/mb -> tools-webrtc/mb
2016-12-17 magjed@webrtc.org Put iOS H264 High profile under a field trial
2016-12-17 kjellander@webrtc.org Move tools/valgrind-webrtc -> tools-webrtc/valgrind
2016-12-16 skvlad@webrtc.org Revert of Delete unused code from systeminfo. (patchset #3 id:40001 of https://codereview.webrtc.org/2578323005/ )

TBR=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng
BUG=

Review-Url: https://codereview.chromium.org/2588273003
Cr-Commit-Position: refs/heads/master@{#439786}

This is also adding MediaControlsMediaEventListener::mediaElement() and
always call `m_fullscreenButton->setIsFullscreen` in ::reset() to avoid
the dependency when the controls are first created.

BUG=662761
R=zqzhang@chromium.org

Review-Url: https://codereview.chromium.org/2592443002
Cr-Commit-Position: refs/heads/master@{#439785}

Do not allow BlinkGC managed objects to include unsafe iterators of
other heap objects; that is, do not allow them to keep iterator
part objects as fields.

These iterators contain untraced references, which is in general
unsafe practice and breaks the general rule that all heap references
must be known to the GC infrastructure, and be marked and traced
through.

This applies to all heap collection iterators but HeapListHashSet<>'s,
which can be safely traced. It is also the only collection iterator
which is kept as a field of an on-heap object (CSSSegmentedFontFace.)

R=haraken
BUG=672030

Review-Url: https://codereview.chromium.org/2588943002
Cr-Commit-Position: refs/heads/master@{#439784}

Author: Sam McNally <sammc@chromium.org>

Reland of https://codereview.chromium.org/2525483002/, with trivial fixes
applied to fix bustage that occurred due to that CL racing with
https://codereview.chromium.org/2589663003/.

BUG=577685
TBR=sammc, dcheng

Review-Url: https://codereview.chromium.org/2590173002
Cr-Commit-Position: refs/heads/master@{#439783}

BUG=None

Review-Url: https://codereview.chromium.org/2590863002
Cr-Commit-Position: refs/heads/master@{#439782}

CSSGlobalRuleSet should always be collected as part of the active style
update. RuleSets may have been cleared from StyleSheetContents as a
result of media query changes for instance.

For the given issue, we tried to limit to a global ruleset when lazy-
loading fullscreen UA style, but as part of going fullscreen we had
already cleared rule sets for stylesheets with media queries due to the
media feature change.

BUG=675439

Review-Url: https://codereview.chromium.org/2590793003
Cr-Commit-Position: refs/heads/master@{#439781}

This CL does not change any behavior.

Moving internal classes into an internal namespace.
This simplfies testing these classes (JsonRequest and its Builder) and
prevents the use of internal enums (e.g. FetchResults) that were
publicly exposed but not intended for public use.

BUG=672422

Review-Url: https://codereview.chromium.org/2578173002
Cr-Commit-Position: refs/heads/master@{#439780}

BUG=675561

Review-Url: https://codereview.chromium.org/2586293002
Cr-Commit-Position: refs/heads/master@{#439779}

When this extended attribute is specified on a dictionary member
called 'foo', impl class will have getFoo() method instead of
foo().

BUG=673039

Review-Url: https://codereview.chromium.org/2587383002
Cr-Commit-Position: refs/heads/master@{#439778}

According to the PRD the Physical Web section should be first on the
NTP.

BUG=674890

Review-Url: https://codereview.chromium.org/2588303003
Cr-Commit-Position: refs/heads/master@{#439777}

MDC has a method to use the old style, but it's in a private component
and the wind is blowing towards removing it at some point, so this CL
just opt-s out of MDC driving the back button in Settings.

I still filed a request at MDC to have the method public, but it might
never come, or they might remove the old style before that.

BUG=none
R=sdefresne@chromium.org

Review-Url: https://codereview.chromium.org/2589193002
Cr-Commit-Position: refs/heads/master@{#439776}

ARC Kiosk bridge is also moved from components/arc into chrome/browser/chromeos/arc
because it uses parts of chrome/browser/*

BUG=b/32370502

Review-Url: https://codereview.chromium.org/2524673003
Cr-Commit-Position: refs/heads/master@{#439775}

This is a follow-up CL of crrev.com/2583083002. Make sure that calling
these methods doesn't take effect when the renderer is foregrounded.

BUG=674784

Review-Url: https://codereview.chromium.org/2583193003
Cr-Commit-Position: refs/heads/master@{#439774}