GitLab

Replacing - with _.
Chromium project prefers using _ for separating words in path names.

BUG=685886
TBR=kochi@chromium.org

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

PaymentOptions, suitable for use in components/core. This will allow
forthcoming profile filtering and deduplication options (and anything
else which depends on these values) to be written cross-platform, and
reduces the bleeding of Mojo into views logic that doesn't really need
to know about it.

Usage details:
- Desktop: This CL changes PaymentRequestSpec to implement this
  interface. It removes the options() accessor, which is no longer
  necessary.
- iOS: the iOS version of payment_request.h already has a PaymentOptions
  class which corresponds to, and can be trivially modified to
  implement, this interface.
- Android: A small utility class implementing this interface is
  forthcoming. The needed data is present as accessible members of
  PaymentRequestImpl.

See also crrev.com/2775553004 for further context. That CL will be
updated to pass PaymentOptionsProvider instead of PaymentRequestSpec.

BUG=706117

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

Currently ContentVerifyJob never sees DoneReading() for (legitimate
or not) 0-byte files. This means 0-byte files (legitimate or corrupted)
never goes through content verification process.
Override URLRequestJob::DoneReading() in URLRequestExtensionJob to
detect the end of 0-byte files so that ContentVerifyJob knows about
these 0-byte files.

Add a test to ensure that URLRequestExtensionJob will see 0-byte files.
ContentVerifyJobUnittest.DeletedAndMissingFiles already makes sure
that 0-byte read will fail content verification.

BUG=703888,703892
Test=With content verification turned on (e.g. with
--extension-content-verification=enforce_strict flag), install an
extension from webstore and modify a JS content inside that extension
to 0 byte file. Content verification should fail for that extension.
If the extension is part of system policy, the extension will be
reinsalled.

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

A few hundred crash reports in rendering path during execution of
AudioClock::ComputeBufferedMediaDurationMicros. Locks and various
contracts should protect us, so I assume one of the contracts is
being broken and hope to find out where.

AudioClock's buffered_ field is only modified during Render()
and during destruction. Perhaps its being destroyed while still
in-use.

BUG=674856

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

Add support for OfflineItems showing up in the DownloadNotificationService.
This consists of a few changes:
- Building a glue class between the OfflineContentAggregator and the
SystemDownloadNotifier.
- Adding some properties to DownloadInfo to support the properties
OfflineItem needs ('isTransient' and 'isOpenable').
- Update the SharedPrefs to store the isTransient property.

BUG=691805

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

On Android O we need the REQUEST_INSTALL_PACKAGE to be able to install
APKs.  Adding the permission to our manifest and adding it to be set by
default on developer builds.  Has no effect on pre-M builds.

BUG=706570

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

The respondWith() method is used by the payment app to provide a
PaymentAppResponse when the payment successfully completes.

This CL includes a content-side implementation and browser test. Especially, add
a callback to InvokePaymentApp() in PaymentAppProvider. The callback is used to
pass a response data from payment app using respondWith().

Related Spec Link:
  https://w3c.github.io/webpayments-payment-apps-api/#idl-def-paymentrequestevent

See the other CLs in this series:
  [1/3] https://codereview.chromium.org/2715663002/ (RespondWithObserver)
  [2/3] https://codereview.chromium.org/2705293010/ (blink side)
  [3/3] This patch

BUG=661608
TEST=payment_app_browsertest.cc

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

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

BUG=701654

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

We need to do this before the calls to GetSvnRevision, because they run
svn.

BUG=706552
R=thakis@chromium.org,krasin@chromium.org

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

Now that the align-content and justify-content CSS properties are
adapted to the new baseline-position CSS values syntax we can adapt the
shorthand that controls such properties to the new syntax as well.

In addition, this patch makes public some of the utility functions
defined in the CSSPropertiesAlignmentUtils class, so that they can be
shared between the different parsing and computed style logic of the
different alignment related CSS properties.

BUG=703584

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

The tests are:
WebNavigationApiTest.ForwardBack
LoginPromptBrowserTest.SupplyRedundantAuths
LoginPromptBrowserTest.CancelRedundantAuths

I'll investigate these to see if they're real failures or just test issues.

BUG=705559
TBR=nasko@chromium.org

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

I'll investigate these to see if rr460581 made it more flaky.

BUG=651844,705559
TBR=nasko@chromium.org

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

BUG=702640

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

This CL adds an indicator for dragging and dropping bookmarks. This
indicator will highlight folders that can be dragged onto and add a line
above/below items for dragging between items. This affects both the list
and the sidebar, and bookmark elements in both are now draggable.

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

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

- 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}