GitLab

Add an overlay strategy that promotes a single quad to an
overlay iff the buffer that backs the quad can be directly used
as main framebuffer.

BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel

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

map should be included, rather than imported.

BUG=

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

This patch updates the code to prevent auto deducing to a raw pointer.

R=jochen, danakj, dcheng
BUG=554600

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

BUG=612331
TEST=covered by tests
R=jamescook@chromium.org

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

crrev.com/1304303002 changed the display size used in overlay_unittests.
The quad size used in CreateFullscreen* utility functions didn't change,
thus the fullscreen quads created for testing are not fullscreen.

This CL fixes that.

BUG=b/29430506
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel

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

tools/git/move_source_file.py ash/wm/lock_state_observer.h ash/common/wm/
tools/git/move_source_file.py ash/shelf/shelf_locking_manager* ash/common/shelf/

Move LockStateObservers and OnLockStateEvent helper to WmShell temporarily.
(to avoid converting LockStateController to ash/common for now)

Move ShelfLockingManager test access to Shelf/WmShelf.

BUG=629254
TEST=Automated tests; no behavior changes.
R=jamescook@chromium.org
TBR=piman@chromium.org

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

We're TBRing UseCounter changes to Ilya, et al. more often than I'd
like. This patch adds Jochen and I as EMEA-based reviewers for these
mechanical changes until such time as the mechanically-generated
UseCounter data is split out into a file with broader ownership.

R=jochen@chromium.org,isherman@chromium.org,asvitkine@chromium.org

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

BUG=627640

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

CacheStorage (and BackgroundSync) wrap callbacks in order to ensure that the scheduler
is bumped after running the callback. This resulted in lots of helper functions. This
CL cleans up the helper functions with the help of template magic.

BUG=629592

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

This CL removes the code that will scroll to the advanced content area
when toggling the advanced menu in the side nav.

BUG=613708
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/8efcb0b5..746bdef4

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_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,yangguo@chromium.org,vogelheim@chromium.org

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

BUG=

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

This addresses a TODO added by https://codereview.chromium.org/2141843002/, to transfer ownership rather than copy them.

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

AXLayoutObject has code to cache the computed bounding box of an element
because the computation can be expensive. However, it has a bug in that
the cache doesn't get properly invalidated when an object's position changes
due to only a CSS transformation.

This is resulting in a serious user issue on Android (b/28988142) so
this patch just temporarily removes the element rect caching. This will
unfortunately impact performance, some pages will load more slowly when
accessibility is enabled.

I've been working on a better longer-term fix for many months - storing
relative bounding boxes and transformation matrixes instead. It's much
faster to compute those and doesn't require any caching to be fast.
Unfortuantely it looks like we need to trade performance for correctness
in the short term because there isn't a trivial fix to make the cache
work.

BUG=629439

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

BUG=612846
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation;master.tryserver.chromium.linux:closure_compilation

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

This CL allows for scrolling to a section from within a sub section.
It also rearranges the behavior logic a bit to make it easier to follow.

BUG=617737
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:closure_compilation;master.tryserver.chromium.linux:closure_compilation

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

The image_bitmap_from_canvas conformance1 tests are failing with WebGL
2.0. It fails in this case:
Call texImage2D(ImageBitmap), which does GPU-GPU texture copy. Then
call texSubImage2D(ImageBitmap), which does GPU readback. In the call
to texSubImage2D, there is a error says glTexSubImage2D: invalid
internalformat/format/type combination, which doesn't make sense.

The solution here is to set m_image to be null after GPU-GPU texture
copy. So next time when m_image is needed, it has to use a new context
provider to consume the mailbox to get a new m_image.

To make sure this CL does the right thing, I have tested the following:
conformance1 tests with WebGL 1.0 and 2.0 (with GPU accelerated canvas)
conformance2 tests (with GPU accelerated canvas, WebGL 2.0 only)
layout tests:
fast/canvas/,
virtual/gpu/fast/canvas,
virtual/display_list_2d_canvas/fast/canvas.
All the above tests pass.

When committing this CL, we should add the optional GPU bots to make
sure they are green.

BUG=628954
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

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

https://chromium.googlesource.com/skia.git/+log/f2944815e5e4..e499adf328bd

$ git log f2944815e..e499adf32 --date=short --no-merges --format='%ad %ae %s'
2016-07-20 jvanverth Use dFdx in Vulkan to address distance field issues.
2016-07-20 mtklein Tune linear->sRGB constants to round-trip all bytes.
2016-07-20 msarett Refactor parsing and storage of SkGammas
2016-07-20 reed re-disable save/restore peephole optimization
2016-07-20 dvonbeck SkLS now accepting nullptr for diffuse shader and normal source, now accurately handling alpha

BUG=629408

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel
TBR=jcgregorio@google.com

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

Cache key state is currently stored in the renderer and is used
by the image processor to determine which images should be transcoded
and sent to the client, and which images do not need to be sent,
reducing latency and compute resources. A drawback of storing this
state in the renderer is that the cache state is dumped when the
renderer is torn down, as in the case when the user navigates
to another site.

This patch provides the renderer with a snapshot of the browser's
cache state, which means that the knowledge of cache state is
retained when the renderer of a given site is torn down
and brought back again.

* Add GetCacheState() to BlobCache
* Add GetCacheState() to BlobChannelSender()
* Add GetCacheState() to .mojom file.

R=nyquist@chromium.org,wez@chromium.org
BUG=600719

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

BUG=

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

This looks like it hasn't been used since CL923463003 landed
over a year ago.

BUG=331654

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

Merge InitializeAudio/InitializeVideo to support multiple
audio/video streams in CastTransport.

BUG=618469

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

R=thestig@chromium.org
BUG=629145,627474

TBR=dpranke@chromium.org  # for tools/OWNERS

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

Remove the browser test added in codereview.chromium.org/2166503003
That CL also adds a unit test with the same exact checks.

BUG=626995
TEST=NONE
TBR=sky@chromium.org

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

When borders collapse, a cell's border affects its neighbors' widths but
we weren't marking widths and needsLayout properly.

BUG=613728

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

This ensures that the non-client frame hit testing doesn't extend down into the caption button (minimize, maximize, close) area, thus giving the user the impression that the frame is resizable at that point.

BUG=548619

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

Now that we have build delegation for CQ jobs,
this should work.
BUG=chromium:629910

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

This CL fixes two bugs related to PointerEvents and gesture long
presses:

A. We fire pointer events from touch events. Gesture long-press
handler was calling handleMousePressEvent, which caused firing
of additional pointer events for the same touch sequence. This
CL skips firing PEs from PlatformMouseEvents synthesized from
touch.

B. Canceling a pointerdown should suppress all following
MouseEvents, including those fires through touch-gestures. We
have fixed the GestureTap case before. This CL fixes the only
remaining ME case: MEs from gesture long-press.

BUG=627207,606938

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

This macro provides a mechanism to tell trace consumers
that two different IDs are actually identical and they
refer to the same thing.

For more context:
https://docs.google.com/a/google.com/document/d/1s0DKjNJk85hDuRp5IqujwZvPML-MPsyAtDeksMwBw7s/edit?usp=sharing

BUG=catapult:#2465

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

We are removing all cases of a direct output surface in
LayerTreeHostImpl and the only remaining cases are in unit tests. This
coveres the LayerTreeHostImpl test suite.

TBR=piman
R=enne, vmpstr
BUG=606056
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel

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

This is needed to port the shelf to mash.

* Move app list show/hide/visible functions to WmShell
* Remove window parameter from ShowAppList/ToggleAppList because non-test
code always shows and hides on the default display
* Migrate ShelfModel ownership to WmShell

BUG=629250
TEST=ash_unittests, chrome unit_tests
TBR=dmazzoni@chromium.org

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

If the merchant does not accept cards as forms of payment, then do not
show [+ ADD CARD] button the PaymentRequest UI. Screenshot:
https://bugs.chromium.org/p/chromium/issues/attachment?aid=243022&inline=1

BUG=628781

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

Loosen the match requirements for ports so that URLs are matched
regardless of whether the port was implicitly (i.e. the default) or
explicitly specified.  This provides an optimistic filter that ensures
the disambiguation code doesn't get skipped in the case of potential
matches.

BUG=628788

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

During EGL initialization GetPlatformDefaultEGLNativeDisplay() gets
called. This function is conditionally compiled for each platform. Under
Ozone the function calls into OzonePlatform to get the EGL
native_display to use. This ordering requires that //ui/gl depends on
//ui/ozone which we want to break.

This CL removes GetPlatformDefaultEGLNativeDisplay() and instead passes
in the EGL native_display to GLSurfaceEGL::InitializeOneOff() or
GLSurfaceEGL::InitializeDisplay(). For tests EGL_DEFAULT_DISPLAY is
always passed in.

The GL initialization and autogen binding code needs to be rearranged to
accomplish this. For EGL generated code split the client extension
bindings and extension bindings into two separate methods. The client
extension bindings need to be initialized before
GLSurfaceEGL::InitializeDisplay() is called while the extension bindings
need to be initialized after. Also fix a problem preventing
generate_bindings.py from running with missing extension information due
to a recent change.

BUG=611142

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

This re-enables GPU partial raster for non-MSAA content. MSAA resolve
does not work well with partial raster right now - we may try to address
this in the future.

BUG=629683
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_blink_rel

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

This CL brings DateDividedAdapter to Chrome, and lets Download Manager
UI to utilize it. This adapter will take a list of timed items, groups
these items by date, and displays the groups in sections.

BUG=616324

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

Those bits should have been removed by https://codereview.chromium.org/1895113002.

An assertion error was thrown by either of the following
1) Navigating to chrome://md-settings/siteSettings/fullscreen, or
2) Triggering a force-rendering of the subpage in question (via search).

BUG=629665
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Previously WebrtcTransport clients had to use incoming_channel_factory() to
accept incoming data channels. That worked only if the receiver knows in
advance names of all channels it can receive. Now the transport calls
EventHandler for incoming data channels, which allows the receiver to decide
dynamically if it wants to accept that channel. Also channels now can be
closed dynamically and the transport doesn't terminate connection when one
of the channels is closed.

BUG=621691

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

If the HRTF panner isn't loaded yet, instead of zeroing the output,
copy the input to the output so that some audio is heard.  In either
case, there is a potential for a glitch, but it seems better to have
some audio output instead of silence.

The previous worst case was if loading the database is slow and the
sound is short, you won't hear anything.  Now you will.  If the sound
is long enough, you previously would hear silence and suddenly a
sound. Now you'll hear the sound which would suddenly change to due
spatialization.  We deem this to be less troublesome that not hearing the
initial sound at all.

BUG=629858
TEST=Run test from 602668 and audio from the first instance should be
heard

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

BUG=625258

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