GitLab

R=dpapad@chromium.org
BUG=585553
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Replace uses of the individual Top, Left, Width, and Height accessors in
NGPhysicalFragment with Offset and Size respectively to reduce API size,
improve compiler enforcement of logical & physical, and reduce code size.

R=ikilpatrick@chromium.org

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

This patch changes StyleRareInheritedData to make it easier to generate:
- Remove unused destructor
- Replace explicit copy constructor with RefCountedCopyable
- Replace forward declarations with #includes.

This is in preparation for generating StyleRareInheritedData. While this
patch may increase compile times, we would have to pay this cost anyway
when we generate StyleRareInheritedData in ComputedStyleBase.

BUG=628043

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

* Adds new Offliner::RequestStatus::BROWSER_KILLED.
* Reports UMA for requests updated in reconcile task with the status
  above.

BUG=705115
R=petewil@chromium.org

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

Test expectation is that there is no signed in account at the beginning, and by
the end of setUp test expects mAccount to appear signed in from
AndroidSyncSettings perspective. It is done by calling updateAccount(mAccount)
from setUp(). The issue happens when there is another signed in account
according to ChromeSigninController.getSignedInAccountName, potentially left
from some other test execution.

The fix is to reset signed in account with
ChromeSigninController.setSignedInAccountName before AndroidSyncSettings
initialization.

R=skym@chromium.org
BUG=717960

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

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

It turns out that:

1) We can get resource loads for frames that don't belong to the page
and
2) We can get notice of frame commits for frames that don't belong to the page.

We hope this is rare (and record histograms to verify) and relax the constraints.

BUG=708570

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

design doc:
https://docs.google.com/document/d/1O-fB83mrE0B_V8gzXNqHgmRLCvstTB4MMi3RnVLr8bE/edit#
https://docs.google.com/document/d/1obEb8K4mJpG9Y_x4_ZhvnbFHojZoLLzMCFsalkl8voc/edit#

This CL calls ImageResourceObserver::ImageNotifyFinished()
after SVG image loading is completed, not necessarily the same time as
ResourceLoader completion (e.g. DidFinishLoading()).
This makes SVG's <img> load event to be fired after SVG image loading
is completed even when the SVG has subresources loaded asynchronously.

1. Image::SetData() can return SizeAvailableAndLoadingAsynchronously
   when SVG image loading is not completed synchronously (due to
   subresources), and notifies the completion asynchronously via
   ImageObserver:LoadCompleted().
   LocalFrameClient::DispatchDidHandleOnloadEvents() is used as a
   notification of SVG image loading completion in SVGImage.

2. ImageResourceContent delays calling ImageNotifyFinished() and
   setting |content_status_| to kCached until SVG image loading is
   completed.

3. ImageLoader additionally delays the Document load event
   - from the first ImageChanged() during image loading
   - until (potentially delayed) ImageNotifyFinished().

In image loading not by ImageLoader, the Document load event is not
delayed from ResourceLoader completion until ImageNotifyFinished(),
but this is not a regression.

In image loading by ImageLoader, the Document load event is delayed
until ImageNotifyFinished() by the combination of
ResourceFetcher::loaders_ and ImageLoader.
ImageLoader starts delaying the Document load event
- Not from |image_| is set, because we shouldn't delay the Document
  load event when image loading is deferred.
- Not from ImageResourceContent::NotifyStartLoad(), to avoid additional
  ImageResourceObserver callbacks.

BUG=382170, 690480, 667641

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

We need to make the cell's height unusable for calculating perecent/calc height
against before computing its preferred width, otherwise replaced children may use it
to scale up their intrinsic width.

BUG=666730

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

This is speculative as I can't easily compile with use_cras=true
without updating a full chrome os checkout first

BUG=722827
TBR=justinlin
NOTRY=true

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

* add support for detecting URLs on the amp cache domain using the new
  host prefixing scheme

* add support for amp documents loaded in the google news viewer

* factor existing google url detection logic into a common helper, and use
  it in both amp observer and from search observer

* add metrics broken out by type of amp load (amp cache, google search viewer, etc)
  since performance may vary depending on view type (especially for bare amp cache
  urls)

BUG=722690

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

By AudioManager API, Shutdown() must be called before destruction. This
isn't caught by tests because media_service_unittests is only run on
release builds.

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

This is a mechanical change.

BUG=687231

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

BUG=722975
TBR=dpranke
NOTRY=true

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

are currently in c/b/android/offline_pages but are not Android-dependent.
This is to avoid mixing of Android-specific code with cross-platform and to
facilitate offline_pages on other platforms (like ChromeOS) in the future.

Also, move the first file there. Tighter DEPS will come in the next CL.

BUG=722467

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/649a1343..fefd00d2

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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

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

BUG=719535

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

BUG=681694

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

Previously, given a device_id "plug:foo" alsa_util.cc would try to open
the control named ":foo". It will now open "foo".

This is safe because at this point we know we have already found a ':'
at pos1.

BUG=internal 35197382
TEST=manual
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_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

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

Relocated from https://chromium-review.googlesource.com/c/506720/
due to gerrit CQ issues.

BUG=722781
TBR=boliu@chromium.org,torne@chromium.org

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

Complement RendererScheduler.NumberOfTasksPerQueueType metric by
reporting total duration of tasks split per queue type.

R=skyostil@chromium.org,isherman@chromium.org
CC=haraken@chromium.org
BUG=702314

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

We're evaluating a different approach to hiding the `nonce` content
attribute, moving the behavior change up to `HTMLElement` and `SVGElement`
rather than placing it on `{HTML,SVG}{Script,Style}Element`. This
patch adds `nonce` to `ElementRareData` in order to support that
approach, and wires up a new `NoncedElement` interface to the new
properties.

Still behind a flag while we're working out details.

Intent to Implement and Ship: https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/wu_fMIYkyaQ/85j16Cg6BAAJ

BUG=680419

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

Also updates some constants to match spec.

BUG=715554

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

https://skia.googlesource.com/skia.git/+log/738aa4dc7047..eb99bfd85882

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;master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=egdaniel@chromium.org

$ git log 738aa4dc7..eb99bfd85 --date=short --no-merges --format='%ad %ae %s'
2017-05-16 herb Move lambdas to statics.
2017-05-15 halcanary SkCanvas: Helpers for draw{Point,Line,Circle}
2017-05-16 fmalita Add SK_LEGACY_SWEEP_GRADIENT on Android

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

Remove GpuChannelMessageQueue and GpuChannelMessageFilter from header.
Also remove the dependency between MediaGpuChannel and the filter.

R=piman
BUG=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_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

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

Before I landed crrev.com/2859193004 GridLayout didn't take into account
the view's border. The main profile button was adding its own padding to
account for that problem. After that patch landed it meant it was
doubly-padded. This patch removes the extra padding.

BUG=721660

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

This updates the designated requirement used in Chrome's
binary integrity incident checks to be the same designated
requirement that Chrome is signed with. Specifically, it uses
the certificate hash instead of the team identifier.

BUG=703793
DONOTSUBMIT=true

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

BUG=
NOTRY=true

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

BUG=718960

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

Map file generation is disabled on Linux Builders (see
http://crbug.com/716209), so the script fails if Linux is used with
--cloud.

BUG=717550

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

We need a method on DocumentMarkerList that returns the marker type the list
supports so we know when we can safely cast a DocumentMarkerList* to an impl
type.

This CL adds two subclasses of SpellCheckMarkerListImpl, SpellingMarkerListImpl,
and GrammarMarkerListImpl, so we can implement this method.

Note that adding this method also requires modifying
GenericDocumentMarkerListImpl to take a MarkerType param (we're going to
eliminate GenericDocumentMarkerListImpl once we add TextMatchMarkerListImpl but
we need this for now).

BUG=707867

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

The ReadingListMediator will now handle the ReadingListModel instead of
passing it directly in the ReadingListCollectionViewController.

BUG=721758, 688392

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

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

BUG=none
R=rdevlin.cronin@chromium.org

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

This CL refactors audioparam-testing.js so we can hide global variables
and functions from the window scope.

BUG=722884
TEST=(All layout tests passes after this change.)

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

Make DnsSdRegistry a leaky singleton so that it can be used by both CastMediaSinkService and MDnsAPI.

BUG=687377

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

In ChromeOS the graphics cards are opened in the browser process and
IPC-ed to the GPU.

BUG=482610
TEST=Ran on link, and daisy and verified Chrome starts up

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

With our recent change to merge paint timings across all frames in
a page, some of the timing ordering assumptions we make, such as
parse and layout always starting before paint, can be violated.

More specifically, we can observe sequences like the following:
* parse starts in main frame
* child frame created
* child frame paints
* child frame reports paint event
* parent frame reports parse start event

Even though the events happened in the expected order, due to cross
frame buffering, we are notified of the child paint before the
main frame parse start.

This change addresses this issue, by buffering observer callbacks until
we've received events with the expected ordering.

zhenw's recent change https://codereview.chromium.org/2872793002 encountered
this issue and had to be rolled back as a result.

I confirmed that, with zhen's patch applied, the flaky test failed on 8 of
10 runs. With zhen's patch and this patch applied, that same test passed 10
of 10 times.

BUG=722860

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

BUG=591099
TBR=eae@chromium.org
NOTRY=true

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

https://chromium.googlesource.com/angle/angle.git/+log/82830ed..995c2ed

BUG=,chromium:721783, angleproject:1593

TBR=jmadill@chromium.org

TEST=bots

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

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