GitLab

Original CL was reverted due to breaking webkit_tests in
chromium.webkit/builders/WebKit%20Mac10.9/builds/40056

This is due to a problem that already bit me :-)
build infra script to zip up mojom files wasn't updated,
ongoing in https://chromium-review.googlesource.com/c/414565/

Original CL description ------------------------------
ImageCapture: move image_capture.mojom to media/capture/mojo/

This CL moves image_capture.mojom from media/mojo/interfaces/ to
media/capture/mojo/ where it belongs logically speaking, and
updates BUILD.gns and include paths.

image_capture.mojom was in media/mojo/interfaces because
when it landed, media/capture/mojo was not existing.

BUG=518807
TEST=No code touched, if compile passes, it works
(but there's also content_browsertests).

TBR=xhwang@chromium.org for the removal in media/mojo/interfaces/BUILD.gn

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

The Linux-specific prlimit() system call combines and extends the
functionality of setrlimit() and getrlimit().  It can be used to both
set and get the resource limits of an arbitrary process.

Since version 2.13, the glibc getrlimit() and setrlimit() wrapper
functions no longer invoke the corresponding system calls, but
instead employ prlimit(), for the reasons described in BUGS.

If new_limit is not NULL, then the rlimit structure to which it points is
used to set new values.  If it is NULL, then prlimit() acts as getrlimit().

So, allow prlimit() with new_limit=NULL, and pid is the current process
(or 0), so the glib implementation of getrlimit can succeed.

BUG=chromium:662450
TEST=boot on ChromeOS w/ sandbox enabled
 No messages like:
   getrlimit(RLIMIT_NOFILE) failed

R=rickyz,jcliang,vapier

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

* Remove out-of-date comment in OnStreamEnd(); inline |buffer|.
* Remove non-HTTP/2 code path in OnStreamPadding().

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

Combine ash::TYPE_PLATFORM_APP and TYPE_WINDOWED_APP.
(V1 and V2 apps can be distinguished by IsPlatformApp functions)

Remove ChromeLauncherController::IsWindowedAppInLauncher.
(check for item presence and pinned state by id instead)

Remove unused LauncherItemController::TYPE_WINDOWED_APP.
The only behavior change *may* be the combination of drag groups.
(but afaict, we can already mix V1 and V2 app order on the shelf...)

Update users and tests.

BUG=557406
TEST=Automated; no Chrome OS shelf behavior changes for apps.
R=skuhne@chromium.org,jamescook@chromium.org

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

BUG=617539

Committed: https://crrev.com/bfbc065caf6187d00c0e549f07ad264bb1614873
Review-Url: https://codereview.chromium.org/2536523002
Cr-Original-Commit-Position: refs/heads/master@{#434612}
Cr-Commit-Position: refs/heads/master@{#434869}

Change http://crrev.com/21d17ca06e742cbfdcaaa54e0f4964248e3fbfff
incorrectly assumed the input channel layout never changes. This
patch fixes that issue by allowing the channel mask to change as
layouts change.

BUG=666813
TEST=updated unittests
TEST=http://storage.googleapis.com/chcunningham-chrome-shared/534301/aac_test.html

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

This code is no longer used. So remove this.

BUG=608017
TBR=piman@ because presubmit checks after the patch.

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

Revert of [DevTools] wrap async call frames in pausedDetails (patchset #2 id:20001 of https://codereview.chromium.org/2498263003/ )

Reason for revert:
We need to do it everywhere or nowhere.

Original issue's description:
> [DevTools] wrap async call frames in pausedDetails
>
> Currently CallStackSiderbarPane uses Protocol type directly and do a lot of other evil things.
> Wrapping async call frames is first step on the way to bright future.
>
> BUG=none
> R=lushnikov@chromium.org
>
> Committed: https://crrev.com/92e19e512ea0dcb0ed32bcf0508e6797135f98ea
> Cr-Commit-Position: refs/heads/master@{#432708}

TBR=lushnikov@chromium.org,dgozman@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=none

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

Remove useless class and struct forward declarations, and add
missing ones in gpu header files.

BUG=662195
CQ_INCLUDE_TRYBOTS=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/2532763002
Cr-Commit-Position: refs/heads/master@{#434865}

This is necessary to prevent the crash (crbug.com/669223) because
the handler object is destructed when the MD settings dialog is
closed.

BUG=669223
TEST=try

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

- This CL adds AshDevToolsDOMAgent as an observer to all widgets and views.
The WidgetRemovalObserver is specifically to remove the RootView subtree,
while the ViewObserver is to update any other descendant views.
AddViewTree was created so that whenever any |view| is added, it can
be pushed to the frontend.
- Add tests for when views are inserted, removed, rearranged,
removed & inserted.
- Add test to check if a widget owned by a native widget is represented
properly in the DOM.

BUG=648701

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

add has_element_in_any_list check in ElementAnimations::UpdateActivation during
PushProperities process. This solves a corner case if dom for animation be
removed before animation starts. See bug for details.

BUG=668086
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/dade4fe3..a72be232

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

TBR=hablich@chromium.org,machenbach@chromium.org,littledan@chromium.org,vogelheim@chromium.org

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

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

Allow any integer types but always promote the result to unsigned.
This is more consistent with how these operations are normally used.

NOTRY=true

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

Previously the host binary wasn't rebuilt when building
remoting_dev_me2me_host. This CL adds direct dependency between these
targets to avoid this problem.

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

This patch gets rid of |DCHECK()|, which is a wrong assumption, in
|reconcileTextDecorationProperties()| since we have a test case against the
assumption.

BUG=663613
TEST=LayoutTests/editing/execCommand/insert-list-and-strikethrough.html

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

Many subpages show some form of details for a given extension. When the
backing data changes, those changes need to be reflected in the open
subpage. Right now, since the data is just set to the plain JS object,
it's not notified when a change occurs.

Add a subpageItem_ property to extensions page manager so that we can
leverage polymer databinding and link this to the subpages. This way, if
the backing data changes, the open subpage is updated.

Add a test for the same.

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

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

This puts the crash database in the user data directory, rather than in
the global shared one. In order to accomplish this, the variable
expansion from policy needs to be moved to be suitable for use in
chrome_elf. This code handles variable expansions in a registry key that
can be set to override the user data dir (in either HKLM or HKCU).

We do not need to completely remove the usage of DLLs other than
kernel32 in the code despite the code being used in chrome_elf, as in
all cases, it will not execute until after the DllMain() of chrome_elf
has completed, and we load the used functions via GetProcAddress(), not
by import lib. Currently, chrome_elf is signalled from WinMain() of
chrome to start crash reporting. In the near future it will be started
earlier from a background thread spawned in DllMain(). However, that
thread cannot execute until DllMain has completed, so it's then safe to
cause the other DLL loads that the variable expansion requires.

(Regarding the future thread-based spawning, ref:
https://blogs.msdn.microsoft.com/oldnewthing/20070904-00/?p=25283/ which
explains why the creation of the thread with which we don't synchronize
is OK, because DllMain calls are serialized. But we can discuss that
more in https://codereview.chromium.org/2475863004/ too. )

BUG=565446
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win10_chromium_x64_rel_ng

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

The detail view for extensions includes a button to open the extension's
home page (which defaults to the webstore page if an extension doens't
specify one). Hook this up, and add a test.

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

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

BUG=667208

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

Moving milestone to M58.

BUG=605661
TEST=messages logged with EME layout tests

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

Looks like a possible C&P error.  Compile-tested only.

BUG=none
TEST=none
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_nacl_sdk;master.tryserver.chromium.mac:mac_nacl_sdk;master.tryserver.chromium.win:win_nacl_sdk

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

//base's MRUCache allowed specifying key and value types, like
an underlying std::map<>, but the default base::MRUCache explicitly
specialized the comparator as std::less<>. Rather than require the
caller to overload std::less<> or operator<, allow the comparator
to be specified as a templated argument.

BUG=665735

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

Adding getCoalescedEvents API to the idl file
of PointerEventInit and getter method to
PointerEvent idl.
Also add the plumbing and creation in EventHandler
layer and PointerEventManager.

BUG=665937

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

This patch makes |Range::extractContents()| to postpone DOM mutation event
dispatching just before returning value rather than updating C++ variables
which holds |Node*| for every DOM mutation calls, e.g. Node::appendChild(),
Node::removeChild(), etc., to avoid running JavaScript code during in
|extractContents()| for simplicity.

For executing script in |RangeTest|, this patch makes |RangeTest| class to
derived from |EditingTestBase|.

This patch removes
"fast/dom/Range/range-created-in-mutation-event-crash.xhtml" since it causes
script error by calling |Range#getBoundingClientRect()| with |null| value and
coverage of this test is as same as newly added gTest.

This patch is similar to http://crrev.com/199383004 which makes
|Range::deleteContents()| to postpone DOM mutation event.

BUG=642537
TEST=run_webkit_unittets --gtest_filter=extractContentsWithDOMMutationEvent

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

This patch teaches UISourceCodeFrame to display messages from both
UISourceCodes of PersistenceBinding. This is reasonable, since
there might be different "builders" which add messages to either
of the source codes.

This patch also slightly modifies the handling of decorations:
in case of persistence binding, we prefer network uiSourceCode decorations
over the fileSystem ones: it's hard to merge decorations.

This is also an imperative step towards displaying fileSystem file
by default in the editor.

BUG=649837
R=dgozman

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

As they evolved to be quite different and the amount of
common code is minimal, it just make sense to define
which member function is used to render borders and which
is used for backgrounds.

BUG=662504

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

UpdaterState::GetState returns nullptr, if the current platform is
not Windows, and then the test calls at() method on the returned map.

This fixes LTO Linux bot:
https://build.chromium.org/p/chromium.fyi/builders/LTO%20Linux/builds/1560

BUG=615187

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

This patch makes CompositeEditCommand::moveParagraphs abort in the above
mentioned case, because there is no way to move the contents to a destination
inside itself.

There might be a deeper cause of passing such invalid parameter combination
to the function, which is not covered by this patch.

BUG=585048
TEST=webkit_unit_tests --gtest_filter=TypingCommandTest.insertLineBreakWithIllFormedHTML

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

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964

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

This CL pulls out the content on writing LayoutTests from
layout_tests.md, and does a significant overhaul of it.

BUG=665494

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

Previously, the icon fill color was gray. This change restores it to
white, as the specs indicate.

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

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

This test harness is slowing us down, so we are migrating away from it.

BUG=none

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

Earlier this year the default browser infobar icon was changed to a
stylized blue Chrome icon. This cl, per rpop@, reverts the icon to its
original full-color version.

R=sky@chromium.org
BUG=654340

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

BUG=668321

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

BUG=667644

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

https://pdfium.googlesource.com/pdfium.git/+log/3c0f366daafe..405ac0f09e16

$ git log 3c0f366da..405ac0f09 --date=short --no-merges --format='%ad %ae %s'
2016-11-28 tsepez Use CFX_MaybeOwned<> in fpdf_edit_create.cpp

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

TBR=dsinclair@chromium.org

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

Setting the testing build time ensures that the test passes even if the
system clock is incorrect when running the test.

BUG=668539

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

Only Chrome OS and Windows can choose the language.

BUG=619771
R=stevenjb@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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