GitLab

BUG=458416

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

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

After this only the startup benchmarks will have a dependency. The goal is to remove CanRunForPage entirely.

session_restore has used CanRunForPage to select one page from the typical_25. The URL is not used because the navigation action is skipped. The only reason it is there is because there is a test when setting up Web Page Replay that the archive has the URL.

BUG=440101

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

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

Make the appropriate conversions between pixel-space and dip-space
coordinates/sizes depending on the display's device scale factor.

BUG=400837
R=pkotwicz@chromium.org, sky@chromium.org

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

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

R=rmsousa@chromium.org
TBR=beaudoin, binjin, kaliamoorthi, noms, rmsousa
BUG=461160

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

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

He seems to not work on Chromium anymore.

BUG=None
R=nsylvain@chromium.org

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

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

DevToolsClientBackend.StopChromeTracing() tries to call console.time & console.timeEnd.
But in ServiceWorker telemetry tests, ServiceWorker may have already stopped by itself.
This is causing the test flakiness.

So this cl add checking if the context type is 'iframe', 'page'  or 'webview'.

BUG=433943
TEST=./tools/perf/run_benchmark --browser-executable=~/chromium/src/out/Debug/chrome --also-run-disabled-tests service_worker.service_worker

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/abc2d6b..7af3c4a

TBR=zmo@chromium.org,hiroshige@chromium.org

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

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

The definition of nacl_irt_icache was moved to irt.h in the NaCl repository,
so irt_nonsfi.h is no longer necessary.

BUG= https://code.google.com/p/nativeclient/issues/detail?id=4033

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

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

This CL fixes an issue where closing dialog boxes in the app list would
not restore focus to the search box. This is fixed by refocusing the
search box when the app list overlay is hidden.

BUG=450922

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

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

That should prevent it from be used in production code.

BUG=280157
R=brettw@chromium.org

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

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

Currently, when an ObserverList iterator is rewound or reaches the end
of the list, it will trigger list compacting. However, the majority of
the time, no compacting is required as no elements were removed.

Change the policy to only compact if an element was removed during
list iteration, avoiding unnecessary O(N) list walks.

BUG=440134

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

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

BUG=None
TEST=git cl presubmit -uv
TBR=maruel@chromium.org,sky@chromium.org

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

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

This CL prevents http: or https: urls from referring to file: urls as search descriptor xmls.
Allowing urls to refer to other urls with the same scheme has been considered (e.g. a file: url
referring to an OSDD xml from a file: url), but this currently doesn't work for file: urls
because of http://b/863583, so is not implemented here.

BUG=429838

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

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

Log events and debugging information related to HID devices through the
device event log component so that these errors are visible at
chrome://device-log. A DEVICE_PLOG macro has been added which includes
the last reported system error in the log message.

BUG=448901

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

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

BUG=434363
TEST=git cl presubmit -uv
R=kalman@chromium.org

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

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

Show menu item to view original image if the request could go over
the data reduction proxy.

BUG=456247

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

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

It turns out that the permissions are working fine, but the documentation
is inadequate.

BUG=434430

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

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

Passed in argument was not meant to be deleted upon merge:
https://codereview.chromium.org/920523002

BUG=457385
TEST=trybots

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

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

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

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

Surfaces should be feature-complete, so try enabling them on Windows, Linux, and Mac OS. They're still disabled on Chrome OS because they rely on single-threaded browser compositing, and on Android because they haven't been implemented there yet.

BUG=334090
TBR=piman@chromium.org

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

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

Some tests regressed in r316774 because the way edit commands were
dispatched in BridgedContentView changed from going directly to the
TextInputClient to instead go through the InputMethod.

r316774 included a check added to views::Textfield to ensure any
scheduled edit command had been processed before scheduling a new one.
Some tests failed that check: Because the textfield did not have focus,
the dispatch of the event that would commit the edit command was not
being received.

This CL ensures the textfield requests focus in the test.

Fixes (mac-only) views_unittests:
BridgedNativeWidgetTest.TextInput_MoveLeftRight
BridgedNativeWidgetTest.TextInput_DeleteBackward
BridgedNativeWidgetTest.TextInput_DeleteForward

BUG=454353

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

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

This value corresponds to media::VideoFrame::YV12HD which was added in a previous CL.

BUG=None

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

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

BUG=455586

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

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

This patch changes tools/perf/run_benchmark to output the list command
on stdout instead of stderr. This makes grepping for benchmarks easier.

Note: The run command will still output the benchmark list on stderr if
an invalid benchmark is specified.

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

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

The TextInputClient is used to validate allowable operations in a
textfield (e.g. cut/copy/paste). On a readonly textfield, Copy should
work, but Cut/Paste should present to the user as being disabled.

Currently, a read-only textfield returns null from GetTextInputClient(),
making this validation impossible, except "inside" the textfield (e.g.
Textfield::ShowContextMenuForView(..)).

The logic originated in r80226, but doesn't seem to be for a specific
reason.

Note that TextInputClient::GetTextInputType() returns
TEXT_INPUT_TYPE_NONE for readonly textfields, and a test is updated to
check.

BUG=454353

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

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

R=tsepez@chromium.org
BUG=336640

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

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

Mips builds hit this warning when linking the mesa library with libc++.
Temporarily disable the warning until a permanent solution can be found.

BUG=456380

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

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

This changelist primarily changes HostProcess::OnPolicyError, so that it
doesn't anymore shut down the host, but instead restarts the host and prevents
the host from starting again until a valid policy has been read (the
"preventing" functionality has already been present in the form of
StartHostIfReady method for some time now).

This changelist also simplifies HostSignalingManager's functionality related to
sending host-offline-reason.  Previously HostProcess was destroyed while
HostSignalingManager was trying to send host-offline-reason.  Things are
simpler if HostProcess waits until HostSignalingManager sends
host-offline-reason or times out.  This synchronization is especially helpful
when restarting the host (to avoid having to deal with an old
HostSignalingManager from previous SendHostOfflineReason attempt, while we
construct a new HostSignalingManager when starting a host after a reset).

This changelist also introduces a separate host-offline-reason for policy
errors (and updates WebApp resources strings to reflect that).  Note that since
we are not shutting down the host, we probably cannot use one of HostExitCodes
as the host-offline-reason.

Testing done:
1. On Linux manually introduce policy errors at startup, verify host doesn't
   start and sends host-offline-reason, fix errors, verify host starts,
   reintroduce errors, verify host suspends and sends host-offline-reason, fix
   errors, verify host starts again.
2. Verify WebApp UI shows correct tooltip text for the new host-offline-reason.

BUG=455903

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

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

(e.g. scrolling, letterboxing the plugin, resizing the host) from
remoting.DesktopConnectedView into smaller single-purposed classes.

Summary of changes:
1. Moves bumpScrolling implementation into the remoting.BumpScroller.
2. Moves all viewport related implementation from remoting.DesktopConnectedView
   into remoting.DesktopViewport

BUG=457890

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/third_party/ffmpeg/+log/d4b1674..f5698b4

BUG=344767

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

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

Reboot and brain freeze screens can lead be followed by seeing the
browser compositor's background colour, which defaults to white.

BUG=internal b/19379351

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

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

Actually, this creates two libraries:
ppapi_simple
ppapi_simple_cpp

The second must be used if you want to use ppapi_cpp. If you use the C
interfaces exclusively, you can use ppapi_simple instead.

BUG=

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

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

Removing the isolate.py functionality and requiring tasks to be created from an archive hash rather than an isolate file.

Also adding back the controller_test.py and isolate files which were missed in the refactor cl due to a bad move. The only change to these files are to support isolate hashes rather than isolate files.

BUG=460340

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

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

To bring in:

6658243  Update the GN build with the cpu_arch -> current_cpu rename.

TBR=rtoy@chromium.org
BUG=344767

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

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

Revert of Whitespace change to test gnumbd mirror. (patchset #1 id:1 of https://codereview.chromium.org/928073005/)

Reason for revert:
This was only a test, cleaning up.

Original issue's description:
> Whitespace change to test gnumbd mirror.
>
> This will be manually committed to test for the replication lag from
> MTV.
>
> TBR=iannucci@chromium.org
> BUG=None
>
> Committed: https://chromium.googlesource.com/chromium/src/+/824fb9f7c83330b7470608d71bb6cff63b5f858a

TBR=iannucci@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=None

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

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

TBR=rockot@chromium.org
BUG=

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

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

Simply hide the Autofill/Wallet preferences. The correct fix will involve checking the state of the sync experiment, but for M42 it's enough to just disable this preference page.

Will require merge to m42.

BUG=460900
R=aurimas@chromium.org

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

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

This will be manually committed to test for the replication lag from
MTV.

TBR=iannucci@chromium.org
BUG=None

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

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

BUG=454009
TBR=newt@chromium.org

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

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

BUG=454908

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

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