GitLab

- Model file is not included, as it will be delivered through component updater.
- UI continues to work as before when the model file is missing.
- Textures included for each button state.
- Conform to DD UX specs pending.
- Refactored rendering-classes inheritance to be able to generalize them to non quads.

BUG=644562

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

Since the WebLocalFrame* parameter being passed is already available
from WebViewPlugin::WebViewHelper, these should always match. This is
pre-work to remove the (now redundant) parameter since it's already
available from inside the method.

BUG=361765

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

The single tap sent during the VR screen tap tests wasn't always
getting registered since device response can be a bit slow during
the VR transition. For the Cardboard case, wait on Javascript to say
it received another click event. For the Daydream case, wait on VrShell
saying its parent has consumed the event before checking.
BUG=

Review-Url: https://codereview.chromium.org/2768583002
Cr-Original-Commit-Position: refs/heads/master@{#460452}
Committed: https://chromium.googlesource.com/chromium/src/+/3e2ce51e88f19f0cc5196284f580583ddd4dbd9a
Review-Url: https://codereview.chromium.org/2768583002
Cr-Commit-Position: refs/heads/master@{#460623}

- On WARNING condition, request a backgrounded child process to purge memory
- On CRITICAL condition, try to discard a tab first. If it failed, try to
  purge memory from all child processes. It it failed, try to purge memory
  from the browser process
- Use somewhat conservative heuristics for purging
  - Browser: suppress purging for 2 mins after a purge
  - Renderer: after purging, suppress the next purging until the renderer
    goes foreground -> background then remains backgrounded for 30 secs

BUG=696844

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/04f54bf6..dc1b150e

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/2784093002
Cr-Commit-Position: refs/heads/master@{#460621}

We've shrunk the OffscreenCanvas gpu pixel tests' width dimension in
https://codereview.chromium.org/2774423006/. That worked for all other
Android bots except Nexus 5. So we need to further shrink the width
dimension.

TBR=zmo@chromium.org
TBR_REASON=Test suppression on Nexus 5
NOTRY=true
BUG=706647
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/2781183003
Cr-Commit-Position: refs/heads/master@{#460620}

Revert of macOS: Fix a bug in logic to get all memory regions. (patchset #1 id:1 of https://codereview.chromium.org/2786733004/ )

Reason for revert:
This failed ProcessMetricsMemoryDumpProviderTest.NoDuplicateRegions on "Mac ASan 64 Tests (1)".

https://findit-for-me.appspot.com/waterfall/failure?url=https://build.chromium.org/p/chromium.memory/builders/Mac%20ASan%2064%20Tests%20%281%29/builds/28499

Original issue's description:
> macOS: Fix a bug in logic to get all memory regions.
>
> The logic was accidentally double-counting the size of a memory region, causing
> some memory regions to get skipped.
>
> BUG=706547
>
> Review-Url: https://codereview.chromium.org/2786733004
> Cr-Commit-Position: refs/heads/master@{#460585}
> Committed: https://chromium.googlesource.com/chromium/src/+/f955ebf64b34ec9cbdcb137c00bc9bba1867d071

TBR=primiano@chromium.org,erikchen@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=706547

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

https://skia.googlesource.com/skia.git/+log/291932e8e42e..8f7a9a9bfb51

$ git log 291932e8e..8f7a9a9bf --date=short --no-merges --format='%ad %ae %s'
2017-03-29 msarett 565 color xforms: make sure we clamp if necessary
2017-03-29 mtklein jumper, revert to generating .S files

Created with:
  roll-dep src/third_party/skia


Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


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=stephana@chromium.org

Change-Id: I24f4bc582a1958af5f5945ba574ded9bab5eea45
Reviewed-on: https://chromium-review.googlesource.com/462440Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#460618}

Change log:
https://github.com/google/closure-compiler/compare/b2f2d2f3309c7ee61816e068050651af27bccdfa...f9c738eb53fb5cd1d475871ef16c6d1cca19b815
chrome_extensions.js: a3f41820d1260b0d7ed13acac526b803ac6c1870 -> 04dad454b5797074b803c0bab27f1aa87e19d562

BUG=699302
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
R=dbeam@chromium.org

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

Changes:
SystemTray::CloseSystemBubble() !activating_ is caught, reported in crbug.com/702267. Widget::IsClosed() indicates a widget is in process of closing, so we use this to do the check in ActivateBubble().
This is speculative fix for the bug.

BUG=702267
TEST=none

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

R=pkasting@chromium.org
BUG=691895

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

This CL updates a comment in in test_runner.h to reflect the fact that:
 - imported/ has been renamed to external/.
 - csswg-test/ is being moved to wpt/css/.

Change-Id: I158c6da888b9af8a123047ffe8369e3c88217792
Reviewed-on: https://chromium-review.googlesource.com/462340
Commit-Queue: Quinten Yearsley <qyearsley@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#460614}

This change emits a UKM PageLoad metric for the HTTP RTT estimate, and
one for the transport RTT estimate at navigation start.

BUG=706590

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

Now that a valid WebWidgetClient must always be passed when creating a
WebFrameWidget we can remove some of the redundant m_client == nullptr checks.

m_client can now become nullptr if close() is called on the WebFrameWidget,
but this is generally caught by checking m_layerTreeView.

To be safe if(m_client) checks have been replaced with DCHECK(m_client).

BUG=696895

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

The current code appeared to work because a DialogDelegate actually
defaults to focusing the default button ("End Task") rather than nothing
(like a default WidgetDelegate). Thus when you press Shift+Escape the
row for the active tab is selected, and "End Task" is focused. This
"fixed" the blue line around the table.

But on ChromeOS it's possible to have no windows open and press
Shift+Escape, and then the Browser row is selected initially. You're not
allowed to end that task, so the button is disabled.
Widget::SetInitialFocus has code to advance focus if the initial focus
View can't actually receive focus, hence the table winds up receiving
focus.

BUG=702489

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

This change adds a DiscardableHandle class, which acts as the primary
synchronization mechanism for the GPU Discardable Memory system.

This is the first in a sequence of CLs which implement the GPU
Discardable Memory system. This class is currently only created/used by
unit test code.

BUG=706456
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/2732223004
Cr-Commit-Position: refs/heads/master@{#460610}

Add a (d)check in data reduction proxy (DRP) code which verifies that
if the CPAT header is present in the request headers, then ECT header
must be present too.

Other checks added: e.g., chrome-proxy header should only
be sent when the resolved proxy is a data saver proxy.

Finally, the CL also modifies few unittests to avoid tripping the
DCHECKs.

BUG=706650

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

IndexedDBCursor creation can currently cause OOM's in the browser.
Adding lifetime tracing so that this situation is visible.

R=cmumford
BUG=690517

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

It seems that corruption examples in crbug.com/699420 can cause extension
resource files to go missing or become unreadable.
Fix a bug where we never sent any notification to ContentVerifyJob,
in particular: we never called ContentVerifyJob::DoneReading().

Override URLRequestFileJob::OnOpenComplete in URLRequestExtensionJob
to catch these cases.

Also make ContentHashReader not skip legitimate deleted files: the
files that are listed in verified_contents.json. It will still skip
non-existent resources, e.g. xhrs requesting non-existent file (see
crbug.com/404802 for example).

BUG=703888
Test=Install an extension (not app) in chromeos. Chromeos because you want
content verification enforcement to kick in. Other way could be to use
--extension-content-verification=enforce_strict flag in other platform.
Either:
1) Delete background script file. Content verifcation would kick in when
you try to load/run the extension.
2) Same as above^^^, instead of deleting the file, remove the file
permission. In linux you can just do "chmod -r background.js". Expect
content verification failure.
In both cases extension should be reinstalled in the end.

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

These properties provide the width, height, and time of the last video
frame uploaded to a WebGL texture via texImage2D.

BUG=639174
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/2749653003
Cr-Commit-Position: refs/heads/master@{#460606}

This CL changes how the BackgroundFetchJobController interacts with the
requests that have to be made. It'll be given a sequence of requests when
starting a fetch -currently set to a single request-, will fetch them
and will then get the next request from the BackgroundFetchDataManager
once completed.

This significantly simplifies the state we have to maintain, since it
only knows about the in-progress requests, deferring knowledge of which
requests are pending to the DataManager. The tests have been updated to
match this.

In effect, Background Fetch requests made by the developer will now
actually (sequentially) fetch the files. We still have to feed them
back to the developer once all fetches have been completed.

BUG=692540

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

Adds ActiveProfilePrefService to allow mash, which currently
supports only one profile and runs its services as the root user,
to talk to that profile. Replaces PreferencesConnectionManager,
which achieves the equivalent goal in the old pref service.

BUG=654988

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

Bug: 705561, 693821
Change-Id: I682573698ac7c053c44962846744c510b88f4e18
Reviewed-on: https://chromium-review.googlesource.com/462042
Commit-Queue: Jeff Carpenter <jeffcarp@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#460603}

Same person, just the account name changed.

BUG=none
TEST=none

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

It exposes DOMModel already, there is no need for the target.
There is also no need to extend from Common.Object, have an extra field for nothing and provide event listening capabilities.

BUG=none

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

Here comes another small batch of coverage UX fixes.

BUG=697599

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

This also allows to use <label for='..'> and removes manual click events.

BUG=none

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

Ensure that the compositor draws these correctly.

BUG=704389
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/2769063006
Cr-Commit-Position: refs/heads/master@{#460598}

If the window type is EMBED_IN_OWNER, or TOP_LEVEL_IN_WM, then allocate
a cc::LocalSurfaceId on size change and propagate that new LocalSurfaceId
down to the WindowTreeHost of the embedded client. The embedded client
will use that LocalSurfaceId to submit CompositorFrames.

If surface synchronization is enabled (subsequent patch), then the embedder
can immediately embed the surface ID (if the FrameSinkId is available), and
submit its own CompositorFrame with that embedding. The parent's CompositorFrame
will block until the embedded client submits a CompositorFrame or a deadline is
hit.

BUG=672962

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

This patch starts to cleanup CSS sourcemaps as they get detached from
host stylesheets.

BUG=670180
R=dgozman

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

third_party/WebKit/Source/platform/wtf/PassRefPtr.h has a typo on line
47:
https://cs.chromium.org/chromium/src/third_party/WebKit/Source/platform/wtf/PassRefPtr.h?dr=CSs&l=47

"ReftPtr"

R=esprehn@chromium.org
BUG=706566

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

Currently, when collapsing a section, to another route that is part of
a section, we scroll to the new section.

This is not desirable for direct subpage->subpage navigations. This CL
limits that scroll behavior to only when navigating to a section route
(and not when navigating to a subpage route).

BUG=705927
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Local changes which were fixed upstream:

Issue 676623
Fixed upstream with
https://git.gnome.org/browse/libxslt/commit/?id=08ab2774b870de1c7b5a48693df75e8154addae5

Issue 670720
The norm:localTime extension function was removed with the following commit:
https://git.gnome.org/browse/libxslt/commit/?id=b9d63e5f53c29be83f72849e4ac93bb42f737b9b

BUG=705944

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

PolicyWatcher::Create automatically creates an underlying
AsyncPolicyLoader corresponding to the policy storage location of the
current platform. This commit adds an alternative constructor,
CreateFromPolicyLoaderForTesting, which allows a custom
AsyncPolicyLoader to be specified to allow testing other components with
different policy configurations.

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

This patch starts using fast malloc (aka partition alloc) for the following
paint-related data structures:
 PaintInfo
 PaintLayerRareData
 PaintArtifactCompositor
 GeometryMapper
 GeometryMapperClipCache
 GeometryMapperTransformCache
 PropertyTreeState

For more information about USING_FAST_MALLOC, see the Allocator.md docs.

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

QUIC HTTP stream, by centralizing the logic into a single method.

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

With new fromSurface parameter, this should work on all OSes.

BUG=638281

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

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

BUG=706486

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

Update min engagement_threshold_for_flash to 8, to match the external experiment for HTML5ByDefault.

BUG=706532

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