GitLab

A few method definitions in Skia need to be moved into a new cpp file,
while this define is active the old code will continue to compile until
the Chromium build of Skia can be updated.

NOTRY=true
Adds a define. Built locally, but analyze says "No compile necessary".

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

Add a NativeDisplayDelegate implementation that handles fake displays.
Fake displays mimic physical displays when running off device. Allow
user the specify initial display state with command line flag and
provide API to programatically change display state at runtime.

Except for changing the attached displays other NativeDisplayDelegate
functionality is a noop.

Add FakeDisplaySnapshot to store the fake display state. For now it's
very simple but I intend to add a builder to make it easy to construct
more complex displays.

Add FakeDisplayController interface and method on NativeDisplayDelegate
to get one. This interface will allow changing display state at runtime
if implemented. Only FakeDisplayDelegate will implement it for now.

Fake isn't the best term to describe "not real" displays but there are
already virtual displays implemented for casting.

BUG=611475

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

BUG=none

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

This adds support for displaying overriding enabled base::Feature
features in the Launch.FlagsAtStartup histogram.

BUG=642858

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

PlayStore is special app that actually represents 2 apps in Chrome.
One is OptIn flow as native Platform App, second is Android Play
Store app itself. For user we represent them as one app and do
special mapping for Chrome shelf. This CL adds missing functionality
for correct mapping.

TEST=unit_tests
TEST=Manually on device with all combination, app was pinned or
     not before, App (PlayStore and OptIn) was running or not.
BUG=b/31556052
BUG=648285

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

https://pdfium.googlesource.com/pdfium.git/+log/c48089977dc4..a5b37f6b916f

$ git log c48089977..a5b37f6b9 --date=short --no-merges --format='%ad %ae %s'
2016-09-19 kcwu Remove dead code in CCodec_IccModule

TBR=dsinclair@chromium.org

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

BUG=504347

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

Remove "delay_tcp_race": "true" from QUIC testing variations since it is now on by default and not enabled in the finch configs.

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

In the getUnownedAutofillableFormFieldElements_() function for iOS
autofill, inside a conditional that is triggered if a <fieldset> is not
a descendant of a <form> element, there is a line that appends an element
to a non-existant variable `fieldset` when it is clear that `fieldsets`
is what was intended.

BUG=570095
R=jif@chromium.org

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

Arc has a limitation to be visible for primary profile only. However,
there are situations when Arc task can be started while the secondary
profile is shown. Before this led to case when Arc window was shown,
which is unexpected. This CL fixes this issue and prevents showing Arc
window for non-primary profiles.

Test=Manually on the device. Start Arc app in deferred mode and switch
     to the secondary profile fast. Arc app is started but window is not
     shown. Switch back to primary profile and window is automatically
     restored there.
BUG=b/31500778
BUG=647379

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

BUG=646443

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

R=dpranke@chromium.org
BUG=

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

https://chromium.googlesource.com/skia.git/+log/122fac306189..927ac9c53208

$ git log 122fac306..927ac9c53 --date=short --no-merges --format='%ad %ae %s'
2016-09-19 egdaniel Refactor vulkan buffer mapping and unmapping
2016-09-19 bsalomon Revert of Stop flattening GrCoordTransforms in parent GrFragmentProcessors. (patchset #3 id:40001 of https://codereview.chromium.org/2339203002/ )
2016-09-19 mtklein To prep for removing CMake bots, take them off the CQ and presubmit.
2016-09-19 bsalomon Stop flattening GrCoordTransforms in parent GrFragmentProcessors.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=borenet@google.com

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

A recent GN change (https://codereview.chromium.org/2308813002) broke the
target used for building our signed package as it removed one of the binaries.

This change adds the binary back so we can resume getting signed MSIs.

BUG=

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

upload data stream and it can directly ask the stream for the progress instead of
going to URLRequestJob, HttpTransaction and HttpStream layers.

BUG=472740

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

There is logic in the ClientSession/ClientContext which attempts
to drop a connection on the IO thread. Unfortunately, in doing so,
it called a thread-unsafe member getter within the call to PostTask.

This CL moves the getter into a helper method which runs entirely
on the IO thread.

R=nyquist@chromium.org,wez@chromium.org
CC=steimel@chromium.org
BUG=647814

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

Now that Attr never has childen, a Text node can't have an Attr parent,
and so the condition would always return true.

BUG=305105
R=tkent@chromium.org

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

Per https://codereview.chromium.org/2352693002/#msg11

BUG=None

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

controls.

In the future it might be desirable to remove some disabled color IDs
(like LabelDisabledColor) in favor of just applying this alpha at
callsites.

BUG=none

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

The GAIA ID is only necessary on Drive and should only be set there.
As the Mirror cookie doesn't have the granularity to only set it on Drive,
it is set on https://google.com instead.

BUG=647260

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

Use TYPE_DEFAULT message-loop for browser-tests, since that's what
mojo_runner uses by default. We would normally use TYPE_DEFAULT
message-loops for chrome --mash too, but keep using TYPE_UI for now,
since mojo:ui needs that, and it is not known whether mojo:ui will be
run at the type of the message-loop creation.

Also, remove references to MessagePumpMojo since that has been removed.

BUG=none
TBR=jam@ for base/threading/

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

https://chromium.googlesource.com/chromium/src/+/f7e6a1c1a

BUG=641338
TBR=rob.buis@samsung.com

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

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

This CL implements ImageCapture's takePhoto() method by enqueueing a
callback every time the method is called. Check the queue when a new
frame is captured in DoCapture(), and execute callbacks with a blob
argument.

Add a new method GetPhotoBlob() to construct blob. If the source image
is originally in JPEG format, it's copied to blob directly. Otherwise
convert the source image to RGB format and encode it to PNG.

R=mcasas@chromium.org
BUG=646430
TEST=All tests in VideoCaptureDeviceTests passed.

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

This includes:
-switching NetErrorTabHelper to use the new WebContentsObserver navigation callbacks that work with PlzNavigate
-fixing NavigationHandle::IsErrorPage to return true for reloads of error pages

This fixes
DnsProbeBrowserTest.CorrectionsDisabled
DnsProbeBrowserTest.CorrectionsLoadStopped
DnsProbeBrowserTest.CorrectionsLoadStoppedSlowProbe
DnsProbeBrowserTest.Incognito
DnsProbeBrowserTest.NoInternetProbeResultWithBrokenCorrections
DnsProbeBrowserTest.NoInternetProbeResultWithSlowBrokenCorrections
DnsProbeBrowserTest.NxdomainProbeResultWithWorkingCorrections
DnsProbeBrowserTest.NxdomainProbeResultWithWorkingSlowCorrections
DnsProbeBrowserTest.ProbesDisabled
DnsProbeBrowserTest.SyncFailureWithBrokenCorrections
ErrorPageNavigationCorrectionsFailTest.FetchCorrectionsFails

BUG=504347,647859
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
R=clamy@chromium.org, juliatuttle@chromium.org

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

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

These are flaky tests detected by chromium-try-flakes.appspot.com
Most of these have not flaked enough to trigger a bug yet, but they did
end up triggering an additional bog run, which is still bad.

BUG=619055
TBR=tedchoc@chromium.org

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

In mustash we need to have some D-Bus clients in the ash window manager
process and some in the browser process. In traditional ash we need everything
in the browser process. Allow per-process initialization of subsets of clients.

* Split ownership of clients into DBusClientsBrowser, DBusClientsAsh and
DBusClientsCommon
* Make DBusThreadManager::Initialize() take a process enum. This isn't great,
but see code review comments
* Remove unnecessary setters from DBusThreadManagerSetter
* Use ash/DEPS to restrict which clients can be used in //ash
* Only initialize ash clients in ash_unittests
* When running in mustash, limit which clients are initialized in the ash
and browser processes

This is a transitional step toward cleaner multi-process initialization of
clients as discussed in go/chromeos-dbus-clients option (G)

BUG=644414,647367
TEST=chromeos_unittests, ash_unittests

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

https://pdfium.googlesource.com/pdfium.git/+log/ea3c3be83dae..c48089977dc4

$ git log ea3c3be83..c48089977 --date=short --no-merges --format='%ad %ae %s'
2016-09-19 dsinclair Revert of Pdfium: Fix fonts leaking on ClosePage. (patchset #10 id:180001 of https://codereview.chromium.org/2158023002/ )

BUG=79367,48791

TBR=dsinclair@chromium.org

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

Revert of Force U.S. English keyboard layout for TextfieldTest.KeysWithModifiersTest (patchset #11 id:200001 of https://codereview.chromium.org/2197113002/ )

Reason for revert:
Tests are failing on Mac ASAN builder. See details here:
https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20(1)

First failed build:
https://uberchromegw.corp.google.com/i/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/21876

Original issue's description:
> Force U.S. English keyboard layout for TextfieldTest.KeysWithModifiersTest
>
> BUG=633136
>
> Committed: https://crrev.com/19cb1e03c46a4df0c0fd54171369152bcb19c237
> Cr-Commit-Position: refs/heads/master@{#419454}

TBR=pkasting@chromium.org,chongz@chromium.org,sadrul@chromium.org,frankf@chromium.org,wez@chromium.org,samuong@chromium.org,tmoniuszko@opera.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=633136

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

This updates the Flag expectations for PlzNavigate, adding a test that
started failing, and removing several tests that no longer fail/exist.

BUG=576261

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

This fixes the following browser test for PlzNavigate:
  InProcessBrowserTest.ExternalConnectionFail

BUG=638596

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

BUG=639401

While fetching the original request, ScriptLoader sets a member flag.
On receiving notifyFinished for the original request, HTMLScriptRunner
creates a new ScriptLoader instance using the original Element with
another flag set to true. It then calls ScriptLoader::prepareScript which
further calls fetchScript. prepareScript sets the defer flag as LazyLoad and
fetchScript sets the intervention info and header in the resource request.
The intervention info is used to set the priority in ResourceFetcher::requestResource.

This CL also adds a link in the dev tools console warning.

Also added internals.getResourcePriority() for
the layout test to test that the new request goes out with priority very low.

The code was tested using netlog to verify the second request is sent and has
the correct priority (net::IDLE).
One of the example sites created by jkarlin@ is:
http://www.karlin.me/test/docwrite/

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

Using update-w3c-deps in Chromium 6c4ad338.

BUG=646837
R=rbyers@chromium.org

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

Using hard-coded coordinates for mouse input events resolves the problem
of GetViewBounds returning CSS pixels, which are inconsistent across
different device scale factors.

BUG=647202

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

If a user selects text in a readonly textfield, all editing operations
will silently fail. This change adds a new callback, Textfield::OnKeypressUnhandled,
which serves to provide feedback in this case. Currently feedback is only
implemented for Mac in the form of a system error beep.

BUG=647523

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

This test has been flaky for a couple of weeks now. Disabling on
Windows platform until that can be corrected.

TBR=qiangchen@chromium.org
BUG=644614

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

Many developers have added utility functions to check if the many Mojo
instances are ready and with the correct version, which seems redundant.
This change adds a function to do that (and log failures).

BUG=647853
TEST=cheets_SettingsBridge

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

https://chromium.googlesource.com/skia.git/+log/950fc6c6360d..122fac306189

$ git log 950fc6c63..122fac306 --date=short --no-merges --format='%ad %ae %s'
2016-09-19 mtklein GN: remove old Android recipe code.
2016-09-19 reed add 'g' to fatbits
2016-09-19 mtklein Delete unused coverage recipe code.

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
TBR=borenet@google.com

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

TBR=falken@chromium.org
BUG=648247

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

When setting tFoot or using createTFoot the new
element should be inserted at the end.

This patch also changes behavior so we throw a
HierarchyRequestError [1] when setting tFoot with
a new element not of type tfoot.

Behavior matches Safari, Firefox and Edge.

[1] https://html.spec.whatwg.org/#dom-table-tfoot

BUG=641338

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

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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