GitLab

https://chromium.googlesource.com/chromium/blink/+log/950989fb2c87550e55d5a09f04071998b96b1c1f..48c74875dbbd1bfdcab18a17a3b3710487aaefb5
TBR=dpranke@chromium.org,apavlov@chromium.org

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

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

This way we'll get saner and less noisy histogram-based results.

Rationale for moving histogram_utils.py: HistogramValue is part of
telemetry.value, so it makes sense that all utils it needs are there too;
doesn't make sense to import the utils from metrics.

BUG=

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

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

This implements a Cast button in the default media controls, and a second,
overlayed Cast button on videos that don't use the default controls.

This replaces the previous implementation of the cast button in the
Android UI.

Pressing the Cast button asks the browser UI to bring up cast device
selection menu. If a device is selected the video or audio will be played
on that device, rather than locally.

The Cast buttons are only displayed when there are cast devices available.

Currently this is only implemented for Android; on other targets Chrome
will always tell Blink that there are no devices available.

Depends on Blink issue https://codereview.chromium.org/291163004/

BUG=390125

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

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

This CL changes the padding values in the experimental app list to
precisely match the padding spec.

BUG=411775

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

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

Also makes the scheduler state machine output use the scheduler Now() value so
the output is identical on every run during tests.

BUG=380889

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

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

This CL updates the layout of the experimental app list's start page to
suit its new size.

BUG=411775

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

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

BUG=

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

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

Empirically it can take chrome up to 40 seconds to come up for a cold start on the 10.6 bots. Increase the timeout in Telemetry to reduce test flakiness.

BUG=405386

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

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

R=darin@chromium.org

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

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

Errors reported by closure linter are fixed to keep accordance with style guide and prepare for closure compiler
Some errors are ignored.
- E:0007: We don't adopt the blank-line policy as it's not specified by style guide.
- E:0011,0002,0225: There are some false positives around bind().
- E:0121: Comma at end of object literal is valid in ES5, and not prohibited by style guide.

BUG=412662
TEST=find ui/file_manager -name *.js ! -path */*_scripts.js | xargs gjslint --strict --disable 0007,0011,0002,0121,0225

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

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

Revert of Second attempt to land change to remove NativeViewportService and (patchset #9 id:160001 of https://codereview.chromium.org/568883003/)

Reason for revert:
ViewManagerTest failure.

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#master=chromium.win&testType=mojo_view_manager_unittests&tests=ViewManagerTest.AddAncestorFails,ViewManagerTest.AddViewWithNoChange,ViewManagerTest.CantAccessChildrenOfEmbeddedView,ViewManagerTest.CantGetViewTreeOfOtherRoots,ViewManagerTest.CantModifyChildrenOfEmbeddedView,ViewManagerTest.CantRemoveViewsInOtherRoots,ViewManagerTest.CreateView,ViewManagerTest.DeleteView,ViewManagerTest.DeleteViewFromAnotherConnectionDisallowed,ViewManagerTest.EmbedWithSameViewId,ViewManagerTest.EmbedWithSameViewId2,ViewManagerTest.MultipleEmbedRootsBeforeWTHReady,ViewManagerTest.OnViewInput,ViewManagerTest.ReorderView,ViewManagerTest.ReuseDeletedViewId,ViewManagerTest.SetViewVisibility,ViewManagerTest.SetViewVisibilityNotifications,ViewManagerTest.ViewHierarchyChangedAddingKnownToUnknown,ViewManagerTest.ViewHierarchyChangedViews,ViewManagerTest.ViewsRemovedWhenEmbedding

Original issue's description:
> Second attempt to land change to remove NativeViewportService and
> ViewManager from mojo_shell. First attempt here:
> https://codereview.chromium.org/477923004/
>
> Failed the linux gn dbg build after cq landed:
> http://build.chromium.org/p/chromium.linux/builders/Linux%20GN%20%28dbg%29/builds/12251
>
> BUG=
> TBR=ben@chromium.org
>
> Committed: https://crrev.com/d3e622b0bb31f20c274bc9ad634b8ee7bae99ac1
> Cr-Commit-Position: refs/heads/master@{#295230}

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

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

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

Move non-trival constructor and destructor out of line.

R=phajdan.jr@chromium.org
BUG=none

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

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

https://chromium.googlesource.com/chromium/blink/+log/829f4154fb8a3afc1bee0c3487e2f5a4060e66ea..950989fb2c87550e55d5a09f04071998b96b1c1f
TBR=loislo@chromium.org,dpranke@chromium.org

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

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

BUG=414488

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

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

BUG=401492

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

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

Changing in the intialization order of WeakPtrFactory 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/570713007

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

TBR=machenbach@chromium.org

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

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

Changing in the intialization order of WeakPtrFactory 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/567943002

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

This wasn't caught by the trybots or CQ since this is only compiled in
official builds.

BUG=110610
TBR=davemoore@chromium.org

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

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

https://chromium.googlesource.com/chromium/blink/+log/cf5fdf9447d2c84224c74bf676957d6aac0f7f42..829f4154fb8a3afc1bee0c3487e2f5a4060e66ea
TBR=loislo@chromium.org,dpranke@chromium.org

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

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

* getMultiProfileShareEntries_ is very slow.

BUG=409711
TEST=manually
1.Copy the large directory in the drive of user A.
2.Paste the A's large directory into the drive of user B.
3.Confirm notification is displayed.

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

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

The implicit conversion has already been disabled for Linux builds for
some time now. https://codereview.chromium.org/510323002 explains the
rationale for proceeding incrementally.

BUG=110610

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

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

Changing in the intialization order of WeakPtrFactory 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/572853002

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

BUG=413944
TEST=Instlal ChromeVox and confirms that it speaks both on the icon and on menu item.

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

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

This gets the //chrome subtree down to a fairly small number of build
errors on Android. Most of the changes are making features not enabled
on android like extensions and plugins properly guarded. This also adds
a few jni targets and fixes up some bitrotted bits.

R=brettw@chromium.org, cjhopman@chromium.org

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

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

ViewManager from mojo_shell. First attempt here:
https://codereview.chromium.org/477923004/

Failed the linux gn dbg build after cq landed:
http://build.chromium.org/p/chromium.linux/builders/Linux%20GN%20%28dbg%29/builds/12251

BUG=
TBR=ben@chromium.org

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

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

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

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

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

This CL changes the experimental app list's AppListItem size when
displayed in the AppsGridView and adds padding between each AppListItem.

BUG=411775

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

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

The re-land fixes the unittests to test with/without the field trial set.

Sending is controlled by a finch experiment.

The verdict from the server is ignored for now.

BUG=413967

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

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

R=lcwu@chromium.org,byungchul@chromium.org
BUG=400914

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

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

Original CL broke the Win official bots because they report their default
version channel as "canary" instead of "trunk".

https://codereview.chromium.org/572813002

Original description:

This refactor will allow some manifest tests running in Chrome's unit_tests
suite to move to the extensions_unittests suite.

* Rename ExtensionManifestTest to extensions::ManifestTest and move it into
  src/extensions.
* Introduce ChromeManifestTest to load manifests from Chrome's test data dir.
* Eliminate some unnecessary use of chrome::VersionInfo::Channel.
* Move SharedModuleManifestTest to extensions_unittests as an example.

FileHandlerManifestTest and ExternallyConnectableManifestTest will be next,
but require more refactoring (e.g. of permissions).

BUG=397165
TEST=unit_tests, extensions_unittests
TBR=rockot@chromium.org

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

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

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

BUG=110610

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

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

Revert of Clean up GestureEventDetails constructors and fix unit tests (patchset #1 id:20001 of https://codereview.chromium.org/573963005/)

Reason for revert:
This broke TouchEditableImplAuraTest.TouchSelectionOnLongPressTest.
[24874:24874:0916/151803:584385367:FATAL:gesture_event_details.cc(54)] Check failed: false. Invalid event type for constructor: 32
http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=undefined&tests=TouchEditableImplAuraTest.TouchSelectionOnLongPressTest&testType=content_browsertests

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#group=undefined&tests=TouchEditableImplAuraTest.TouchSelectionOnLongPressTest&testType=content_browsertests

Original issue's description:
> Clean up GestureEventDetails constructors and fix unit tests.
>
> A new version of Issue 565583005: Clean up GestureEventDetails
> constructors
> https://codereview.chromium.org/565583005/
>
> BUG=350942
>
> TBR=tdresser@chromium.org, jdduke@chromium.org, sadrul@chromium.org
>
> Committed: https://crrev.com/874611d5ab5b879648682c1ea41062bebc9c95e5
> Cr-Commit-Position: refs/heads/master@{#295144}

TBR=sky@chromium.org,lanwei@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=350942

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

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

Routing policy override is needed at the point where ping test is executed in AndroidRndisConfigurator or the ping may fail.

It's also needed after DNS is changed in AndroidRndisForwarder constructor as executing setifdns may reset the policy table.

Thus we need to make multiple calls to OverrideRoutingPolicy() during RNDIS forwarder creation lifecycle.  But this is ok since the function is idempotent.

BUG=

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

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

BUG=411173
TEST=layouttest http/tests/serviceworker/

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

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

The memory allocated by AllocAndCopyName was not being freed
if the in-process policy engine did not allow the request to
query the broker.

This was nicely reported by typo.pl@gmail.com

TEST=see bug
BUG=414039

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

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

Add plumbing for a ResetStreamTextureProxy which should be
called on the event of a context loss. Only hooked up the
call for the in-process implementation in this CL when the
contex is restored.

ResetStreamTextureProxy currently handles "onContextLoss" as
well as "onContextRestored" concepts. It deletes the old
stream texture id, and recreates and binds a new
StreamTextureProxy.

BUG=412578

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

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

https://chromium.googlesource.com/chromium/blink/+log/435acf076b2d5f0243b4a87c5d5ef904ae59bed3..cf5fdf9447d2c84224c74bf676957d6aac0f7f42
TBR=loislo@chromium.org,dpranke@chromium.org

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

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

source - fixes to includes.

R=rch@chromium.org

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

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

TEST=Tested that easy unlock still works
BUG=None

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

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