GitLab

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/179d620..356af1f

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

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

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

BUG=none
TEST=none
R=jamesr@chromium.org

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

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

Although passing SkBitmap by value doesn't copy the underlying pixels, it indeed involves
memcpy (~64 bytes to be precise) to replicate the settings from the other bitmap in the copy constructor.
Along with this, it has to do some book keeping with respect to reference counts, updating pixel lock count
etc.

This overhead, albeit minor, can be avoided by passing SkBitmap as reference.

R=abarth@chromium.org, jamesr@chromium.org

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

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

BUG=285481
R=courage@chromium.org

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

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

BUG=none
TEST=none
R=jamesr@chromium.org
TBR=perkj@chromium.org
NOPRESUBMIT=true
NOTRY=true

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

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

This CL moves the core web_request code, except:
-web_request_api_unittest.cc
-web_request_apitest.cc
-web_request_permissions_unittest.cc

Also moves json files:
-web_request.json
-web_request_interal.json

BUG=352293

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

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

Fix some style issues regarding order of includes and forward declararions.

BUG=416623

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/1b3738e..179d620

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

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

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

BUG=399852,416729
NOTRY=true
TBR=groby@chromium.org

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

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

Remove/ifdef (as appropriate) a few Linux-specific assumptions.

BUG=399363
TEST=app_shell_browsertests

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

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

Adding build support and JNI hooks for undoing
bookmark actions.

BUG=415411

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

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

This is to allow unpacked extensions to load without checking
a shared module's whitelist.  Webstore installs and side-loading
via CRX are still restricted.

BUG=414950

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

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

BUG=408049
TEST=Manual, see bug

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/fdec548..1b3738e

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

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

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

BUG=122753

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

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

The big change in this version is that an installation attempt initiated
by the application can show up as ondemand on the server. This will
allow us to provide throttled Mac downloads (such as a "5% push") with
correct behavior.

A previous registration framework, 1.2.0.6782, intended to fix this bug
was backed out (via DEPS) because it was not compatible with older
installed Keystones. This version contains a fix for that problem.

Another previous registration framework, 1.2.0.7407, was backed out
because it contained bugs in its installer.

Previous commit: r245361, https://codereview.chromium.org/141193002
Backed out: r245603, https://codereview.chromium.org/141583005
Tried again: r261532, https://codereview.chromium.org/212703006
Backed out: r262037, https://codereview.chromium.org/227173002

BUG=35495
R=oemilyo@chromium.org

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

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

BUG=None
NOTRY=true

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

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

BUG=416673

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

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

BUG=
TBR=vmpstr@chromium.org
NOTRY=true

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

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

For now the vsync signal can be sent to the browser compositor, which will handle distributing it like before.

BUG=

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

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

This CL fixes a bug where drop targets were not being calculated correctly
which led to duplication on reparents.

BUG=415530
TEST=Drag an app from inside a folder onto the page switcher. The app
should disappear from the folder and appear in the root grid.

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

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

There seems to be a problem with this test, as it seems to change
often after being rebaselined recently.

BUG=416719

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

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

Do not change state of More/less settings button until printer capabilities are retrieved to avoid unnecessary fade in/out cycles.

BUG=415898

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

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

* Remove all #if defined guards from data reduction proxy functionality.
* Hard code proxy urls in params rather than providing them in gyp files.
* Add DataCompressionProxyRollout field trial to all platforms.
* Pass in bypass duration into UpdateRetryInfoOnFallback rather than computing
it based on SPDY_PROXY_AUTH_ORIGIN. Remove unused UMAs.

BUG=384394,384397

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

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

The test shouldn't have worked since the reload() would have prevented
the implicit domAutomationContoller.send() from executing. However,
runtime.reload() is async, so sometimes it continue executing, and
sometimes it doesn't.

BUG=407409

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

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

The method will return the user email and whether the user's logged in,
i.e. whether the app is running on signin screen.

Adds an event that's triggered when the user focused on login
screen changes.

While here, add logic to unload the app and reset signin profile
EasyUnlockService when user logs in.

Create the signin service only if enable-easy-signin switch is set.

BUG=401634

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

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

Revert of Class for allocating a chunk of memory for RenderPass (patchset #31 id:660001 of https://codereview.chromium.org/400463002/)

Reason for revert:
Broke cc_unittest for Linux ASan bot

Original issue's description:
> Class for allocating a chunk of memory for RenderPass
>
> For DrawQuads and SharedQuadState, RenderPass used allocate them one by
> one whenever needed. This new class helps RenderPass manages allocation
> and iteration of those two types. This container allocates a chunk of
> memory at one time and hands out raw pointers. It also provides iterator
> and reverse iterators for going through its contents.
>
> Unittest for ListContainer makes sure the raw pointers it hands out are
> valid and iterator has same behavior as vector iterators.
>
> Follows 398533002, and 404563005.
>
> BUG=344962
>
> Committed: https://crrev.com/6ae8c50c839a8c0d8fb399578256c754ed3ce39a
> Cr-Commit-Position: refs/heads/master@{#296100}

TBR=danakj@chromium.org,jamesr@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=344962

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

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

A visibile to invisible state change will also release memory and should therefore wait as an unload does

A visibility change will also release memory and should therefore also stall an immediate unload.

Updated comments and cleaned up the ResourceManager function.

BUG=414918
TEST=unittest

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

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

- add -o pipefail
- do not hide stderr

BUG=none
R=thestig@chromium.org

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

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

BUG=176648

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

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

The test NSMenuItemAdditionsTest.TestMOnDifferentLayouts was failing because a
new French keyboard layout was added that the test logic failed to account
for.

BUG=390276

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

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

Once the Address Book prompt has been shown on 5 times (repeatedly being shown
on the same form doesn't count), don't show it ever again.

This CL also re-enables the access Address Book prompt, which was disabled due
to bugs in the implementation.

BUG=414359

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

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

BUG=393873
R=dbeam@chromium.org

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

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

Due to GetResolveParameterType(), the template was specialized for all
the cases used. Adding CdmPromiseTraits so that the specializations
are not needed, and the type set in the constructor appropriately.

BUG=358271
TEST=existing EME tests still pass

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

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

Avoid queueing up too many swaps on the GPU.

BUG=

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

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

Multiple tests cases in these tests fail. Mass disabling rather than
playing whack-a-mole.

BUG=416620,416621
TBR=davemoore@chromium.org
NOTRY=true

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

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

BUG=414501

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

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

On the windows side (and likely chromeos for some devices) we can end
up in a situation where the pixel bounds is not an even multiple of the
scale factor. This means when TiledLayer calculates the scale it ends
up off.

BUG=416655
TEST=added test coverage
R=danakj@chromium.org

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

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

BUG=412126

Committed: https://crrev.com/01c68e8765254d9971296b291ff718b4f6af3004
Cr-Commit-Position: refs/heads/master@{#295591}

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

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

I don't know how to fix this issue yet. UMA is not worth of crashes.

BUG=406227

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

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