GitLab

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/582258c..665ffc7

TBR=

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

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

In base.gyp, USE_SYMBOLIZE is added for Desktop Linux and
Chrome OS builds for both base and base_unittests. In the GN
build, it's added for 'is_linux' builds (which does not include
Android), but only for base_unittests. This adds the define for
//base as well.

BUG=335824
TEST=build base and base_unittests, make a debug build and force a DCHECK() -- you should get a useful stack trace
R=scottmg@chromium.org

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

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

Fix up some issues along the way.

BUG=318404
TBR=nyquist@chromium.org,boliu@chromium.org

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

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

Change the PRESUBMIT to call the script from Blink (run-webkit-tests)
directly.

BUG=338338
TEST=None
R=dpranke@chromium.org
TBR=sky@chromium.org

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

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

A previous refactoring had left some duplicate code which fetched the
host list twice!

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

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

This is so it matches with GYP version. This was reported by
http://build.chromium.org/p/chromium.linux/builders/Linux%20GN/builds/12122/steps/gyp_flag_compare/logs/stdio

BUG=None
TEST=gn gen out_gn/Debug && ninja -C out_gn/Debug events_base
events_unittests
R=brettw@chromium.org
TBR=sky@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/0721cb8..582258c

TBR=

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

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

The CL moves the final attribute on BrowserPlugin 'allowtransparency' out to the extensions module! No new content APIs had to be introduced! Transparency can be enabled from RenderWidgetHostView::SetBackgroundOpaque in the content embedder.

This means that BrowserPlugin no longer depends on NPAPI and so this patch removes a lot of unnecessary code.

TBR=asvitkine@chromium.org for histograms.xml, extension_function_histogram_value.h

TBR=asvitkine@chromium.org for histograms.xml, extension
BUG=330264

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

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

Remove the Stable whitelist.

BUG=324071,374440

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/d79096c..0721cb8

TBR=

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

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

ViewManager from mojo_shell.

Original cl was https://codereview.chromium.org/477923004/

Failure was
http://build.chromium.org/p/chromium.win/builders/Win7%20Tests%20%28dbg%29%284%29/builds/28279
TBR=ben@chromium.org
BUG=

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

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

Since set_message_loop() takes ownership of the MessageLoop
parameter anyway, just pass it a const scoped_refptr<T>&, rather
than converting from scoped_refptr to T* back to scoped_refptr.

BUG=110610
TBR=rvargas@chromioum.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/13645ea..3c196de

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

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

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

- Add a .get() to a boolean test until boolean tests on scoped_refptr
  can be re-enabled.
- Change the test fixture store() accessor to just return a raw pointer,
  since no one depends on it returning a scoped_refptr anyway.

BUG=110610
TBR=stuartmorgan@chromium.org

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

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

This is a silly remnant of the extensions refactor. We go to chrome to go
back to extensions. This removes this API and calls IsWebViewRequest directly.

BUG=352293
TBR=sky@chromium.org for code deletion in chrome_extensions_browser_client

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

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

BUG=
TBR=stgao@chromium.org

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

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

Enumeration and iteration were passing around void**. With this CL, we
instead use an Iterator object.

TBR=clamy@chromium.org,jkarlin@chromium.org,jsbell@chromium.org
BUG=413644

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

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

BUG=416127

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

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

Changing in the intialization order of WeakPtrFactory in "src/remoting" module
such that all member variables should appear before the WeakPtrFactory
to ensure that any WeakPtrs to Controller are invalidated before its members
variable's destructors are executed, rendering them invalid.

BUG=303818

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

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

Also, fix passing tpm ket to challenge creator-> the key array contains
0x00, so standard conversion const char* -> std::string won't work.

BUG=401634

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

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

Pipeline onDrawHardware, kModeSync, and kModeDraw. Each step will block
the previous step in the pipeline. This is the avoid the parent
compositor getting ahead of child.

BUG=

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

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

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

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

Includes:

2d28224 Fix a bug when assign the generation number of indirect objects
26019d4 Adjust the order of clearing resource in CPDF_DocPageData::Clear

TBR=tsepez@chromium.org
BUG=416155

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/5f95dbe..d79096c

TBR=caseq@chromium.org,pdr@chromium.org,wjmaclean@chromium.org

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

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

Previously LibjingleTransportFactory was creating and returning
transport before the transport is connected. This means that that the
PseudoTCP was trying to start handshake on a broken transport. That was
delaying PseudoTCP/SSL when the transport takes some time to setup
(e.g. over STUN/Relay). Now LibjingleTransportFactory returns the
transport only after it becomes writable. Empirical testing shows that
this change makes connection initiation much faster in cases when NAT
traversal is required (4 vs 10 seconds).

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/d49226c..13645ea

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

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

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

prevent PictureLayerImpl from creating tiles if the entire
layer is solid.

If the entire layer is solid, PictureLayerImpl will now draw
solid quads using a method provided in SolidColorLayerImpl.

The is_solid_color_ and solid_color members have been moved
from PicturePile To BasePicturePile so that the properties
are copied between PicturePile and PicturePileImpl.

BUG=396908

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

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

This is for http://src.chromium.org/viewvc/blink?view=revision&revision=182213

BUG=123896

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

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

CursorShapeStub is being called in perf tests and the test was
returning NULL for the cursor stub which causes it to crash.

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

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

FrameSender now decides whether to drop frames based on the media
duration in-flight (instead of the number of frames in-flight).  The
maximum allowed in-flight duration is the target playout delay plus the
one-way network trip time, the latter of which accounts for the time it
takes for an ACK to travel back from the receiver.

ShouldDropNextFrame() still limits by the number of frames in-flight,
but only so that a client cannot flood the FrameSender with frames at a
rate greater than the configured maximum frame rate.  Some burstiness is
allowed to mitigate false-positive detections.  In a typical session,
frames should never be dropped based upon the in-flight count.

This change also alters the design of PacketStorage where, instead of
being a fixed "large enough for anything" size, it only holds the frames
that are in-flight.  This is needed because we no longer limit by the
number of frames in-flight, and there's no way to determine what the
"large enough for anything" size has to be.  It's now the duty of
FrameSender to "cancel out" acknowledged frames, and of RtpSender to
release them from storage.

BUG=404813

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

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a Mac build.

BUG=110610

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a Mac build.

BUG=110610

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

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

BUG=None
TEST=ninja -C out/Debug gn_unittests
TBR=brettw@chromium.org

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

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

BUG=397754
R=mattm@chromium.org

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

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

This CL lays the ground work for implementing in process native messaging.

It extracts a common interface |NativeMessagingChannel| outs of
native_messaging_channel.h and renames the existing implementation
to pipe_messaging_channel.

This CL is best reviewed by diffing against patch set 1.

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

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

BUG=388885

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

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

When the InfoBarContainer is first attached to the window,
View.initialAwakenScrollBars() is called on it. Even though the scrollbars
aren't visible, this schedules a delayed runnable to fade out the scrollbars
1.2 seconds later. This delayed runnable (View$ScrollabilityCache) then calls
View.invalidate() repeatedly as it fades out the (already invisible) scrollbar,
which pauses scroll updates and touch event handling for 10 to 15 frames.

This CL disables the scrollbars on the InfoBarContainer, except in the rare
case that the container is actually scrollable (i.e. when infobars take up the
entire screen). In that case, page jank doesn't matter since the user can't
even see the webpage!

BUG=407149
NOTRY=true

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

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

All DataReductionProxy.BypassedBytes.* UMA that don't have an explicit triggering
request UMA need to count the triggering request bytes.

BUG=413771

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

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

This patch was generated by running the rewrite_scoped_refptr clang tool
on a Mac build.

BUG=110610

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

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