GitLab

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/cee4ddf..b3eb687

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=mtklein@google.com

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

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

HTML Tags: head, input type tel, noscript
ARIA Role: listbox, separator
ARIA state/property: aria-required(true/false)

Following HTML tags are undergoing/done in some other CL:
HTML Tag - input type num, pre, header, blockquote.

HTML ruby tag is not exposed correctly. Will submit it with the fix in new CL.

BUG=170577

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

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

Instead of GetCommandLineString, uses
GetCommandLineStringWithPlaceholders, which is specially designed to
quote placeholder arguments like "%1". This is necessary when writing a
command line to the Windows registry, because otherwise Windows may put
a filename with a space in it, and recognise it as two separate
arguments.

BUG=130455

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

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

This CL refactors the current class hierarchy, currently looking sth like:

VideoCapture (abstract)
    ^    ^--- VideoCaptureAndroid
    +-------- VideoCaptureTango

into:

VideoCapture (abstract)
    ^    ^---VideoCaptureCamera (abstract)
    |           ^    ^--- VideoCaptureAndroid
    |           +-------- VideoCaptureTango
    |
    +--------VideoCaptureCamera2  (To be introduced in parallel CLs(*))

(*) Note that http://crrev.com/611283002 only introduces static
methods, so this "plugging" into this new hierarchy can be done
afterwards.

Some operators are also moved from end of line to beginning of the next
one following presubmit warnings.

Also some accessing qualifiers are revisited, and some static methods
consolidated at the top of the class declaration, where they seem to belong.

BUG=418052

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

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

BUG=None
R=msw@chromium.org

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

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

viewer hierarchy aware.

BUG=425017

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

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

It's only used by Loader code.

BUG=426442

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

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

This is the concrete implementation of the web::NavigationItem interface.

BUG=422033
TEST=None

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

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

BUG=422264,414550
TEST=RenderWidgetHostViewAuraTest.PopupClosesWhenParentLosesFocus

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

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

TBR=machenbach@chromium.org

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

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

Previously if the browser got an accessibility IPC it was unable to
interpret, it killed the renderer. This change makes it reset the
accessibility state instead so we're tolerant of corner cases that
are very difficult to completely eliminate in practice.

The accessibility reset is designed to be safe, it throws away all
accessibility state on the browser side and waits for an IPC from
the renderer acknowledging the renderer reset as well.

BUG=372478

Committed: https://crrev.com/0c5e8d16661df7e08c708997f24c0a3070e58080
Cr-Commit-Position: refs/heads/master@{#298297}

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

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

The file blink_skia_config.gyp is changing to a gypi.
This allows easier sharing between the gyp and gn build systems.

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

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

This CL adds a list of active field groups to NetLog dump. The string
representation is human readable, in the form of trial_name:group_name.

BUG=426474

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

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

This makes it clear that this target depends on //content and lays the
groundwork for a future sessions_ios target.

The files in components/sessions/core/ are moved to a variable so that
they can be shared across targets.

TBR=finnur@chromium.org
TBR=sky@chromium.org
BUG=371476
TEST=None

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

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

R=thestig@chromium.org

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

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

gfx::Font already contains a FontRenderParams struct. Make
RenderTextHarfBuzz use it when possible instead of always
querying for a new struct.

BUG=none

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

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

This extracts the common SystemWebview gyp params to a shared .gypi file.

BUG=None
TEST=compiles

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

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

Revert of Revert of Inline sign in extracts gaia id from HTTP header and seeds account tracker (patchset #1 id:1 of https://codereview.chromium.org/671183002/)

Reason for revert:
White success page and no way to proceed only reproduces for use_athena=1 builds.

This CL fixes that: https://codereview.chromium.org/642693003/

Re-landing.

Original issue's description:
> Revert of Inline sign in extracts gaia id from HTTP header and seeds account tracker (patchset #46 id:1740001 of https://codereview.chromium.org/473153002/)
>
> Reason for revert:
> This breaks Chrome OS GAIA login.
>
> After successful GAIA authentication uses sees blank white page and nothing happens.
>
> Please build chromeos=1 build and check that sign in as new user works fine before relanding this CL. Unfortunately autotests rely on fake gaia server and doesn't check this end-to-end sign in flow.
>
> Original issue's description:
> > Inline login handler extracts gaia id from HTTP header and seeds account tracker
> > service during sign in.
> >
> > BUG=341408
> >
> > Committed: https://crrev.com/a8e7c94b1b79a0948d05a1fcfff53391d22ce37a
> > Cr-Commit-Position: refs/heads/master@{#300750}
>
> TBR=guohui@chromium.org,bartfab@chromium.org,xiyuan@chromium.org,achuith@chromium.org,davemoore@chromium.org,rogerta@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=341408
>
> Committed: https://crrev.com/bbaf207927765f4b7c7cae2a4af8db6c4c4a67ce
> Cr-Commit-Position: refs/heads/master@{#300873}

TBR=guohui@chromium.org,bartfab@chromium.org,xiyuan@chromium.org,achuith@chromium.org,davemoore@chromium.org,rogerta@chromium.org
NOTREECHECKS=true
NOTRY=true
BUG=341408

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

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

The version was so old that recording some modern web pages failed.

BUG=425548

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

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

The size of the cache on disk is used if the backend is persistent, else the size of the entries in memory is returned.

Downstream of: https://codereview.chromium.org/674873002/

BUG=420159

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

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

Disable AppListControllerAppInfoDialogBrowserTest.DoShowAppInfoFlow as it's still flaky. More information in bug.

TBR=sashab@chromium.org
BUG=378251
NOTRY=True

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

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

Notable changes (other than just moving code aruond)

* kJavaScriptMessageExpectedDelay constant has been moved from chrome_constants to javascript_dialog_manager.cc because that's the only place used.
* NOTIFICATION_APP_MODAL_DIALOG_SHOWN notification has been removed and replaced by test only waiter class. (as this notification was used only in tests)
* Moved and creates string resource for app_modal_dialog.

Next step: move dialog implementations to compoents/app_modal_dialogs (javascript_dialog_manager/  javascript_app_modal_dialogs_views/, javascript_app_modal_dialogs_cocoa ....)

BUG=422092
TBR=sky@chromium.org

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

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

BUG=N/A
TEST=N/A

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/976386b..cee4ddf

CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg
TBR=mtklein@google.com

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

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

Chrome OS retrieves device state from DMServer during OOBE. If this
state indicates that the device should be disabled, it will show a
corresponding screen and prevent the user from proceeding further.

The feature is on by default because we are aiming for M40 but a flag
is provided that can easily be flipped to turn it off if we do not
make the deadline.

BUG=425574
TEST=Unit and browser tests

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

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

Enabling the unit tests again on debug.
If they still timeout, please revert.

Several changes to reduce the runtime:
- Ensure that there are no duplicate Shill profile notifications, which triggered multiple policy applications and certificate resolves.
- Ensure that each test runs at most one certificate resolve task.
- Use other certificates that are faster to load and import.

Overall this reduce on my workstation runtime in debug from ~6 seconds to <4 seconds.

Also remove the unnecessary TPMTokenLoader initialization.

BUG=418369

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

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

Changed the flag '--enable-extension-info-dialog' to
'--disable-extension-info-dialog', which enables the App Info dialog as
the default dialog for displaying permissions on the extensions page.
Does not affect Mac platforms (where the old-style dialog is still
used).

Contains patches from issues 668163002 and 664473005, since the CQ
couldn't land them individually.

TBR=kalman,asvitkine,benwells,mpearson
BUG=395495

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

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

Replace pump.py generated gin/function_template.h with variadic template version.

This results in a concise code which does not require the use
of pump preprocessor.
Similar effort for base/callback.h is happening in https://codereview.chromium.org/610423003/

BUG=

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

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

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

Compared to the previous CL, this picks up in breakpad:
Fix breakpad on mips and x86_64 for the NDK r10c update.

Original issue's description:
> Revert of Roll android_tools and breakpad to the latest. (patchset #5 id:80001 of https://codereview.chromium.org/671633003/)
>
> Reason for revert:
> Downstream x64 and mips bots didn't like it.
>
> Original issue's description:
> > Roll android_tools and breakpad to the latest.
> >
> > This picks up in android_tools:
> > Updated SDK build tools to 21.0.1 and removed old versions.
> > Upgrade NDK to r10c.
> >
> > And in breakpad:
> > Update breakpad to support Android NDK r10c
> > Fixing the Mac Inspector build by adding the launch_report dependency to the breakpadUtilities dylib project after it got pulled out from Inspector.mm.
> > Refactor .so name detection logic in minidump/linux_dumper.
> > The process uptime computation when the Windows out-of-process Crash Generation Server is used is done by subtracting the crashing process creation time from the current time when the crash is generated.
> > Linux multiprocess crash reporter: use pipe instead of socketpair in the client.
> > Remove workaround for Linux credential-passing bug.
> >
> > TBR=brettw, danakj
> > NOTRY=true
> > BUG=424680, 425077, 358831
> >
> > Committed: https://crrev.com/79bd5f12620f81dd241984d925b87a888e1dc37b
> > Cr-Commit-Position: refs/heads/master@{#300671}
>
> TBR=brettw@chromium.org,primiano@chromium.org,rmcilroy@chromium.org,aurimas@chromium.org,danakj@chromium.org,fdegans@chromium.org
> NOTREECHECKS=true
> NOTRY=true
> BUG=424680, 425077, 358831
>
> Committed: https://crrev.com/e6319d7f7d5c2fc96c9cb1f5a81ae16bde19f068
> Cr-Commit-Position: refs/heads/master@{#300688}

TBR=brettw@chromium.org,primiano@chromium.org,rmcilroy@chromium.org,aurimas@chromium.org,danakj@chromium.org,ppi@chromium.org
BUG=424680, 425077, 358831

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

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

With this call the quota manager can delete origins modified in the last X hours.

Changes:
* Plumb origin and QuotaManagerProxy through to ServiceWorkerCache.

* Call NotifyStorageModified on SWCache::Put() and SWCache::Delete() with the delta being the number of content bytes written (not including caching structure overhead).

* Update the tests to use a MockQuotaManagerProxy instead of passing a nullptr.

Upstream of: https://codereview.chromium.org/672943002

BUG=420159

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

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

The Task Queue Manager previously would give clients a
SingleThreadTaskRunner which held a weak pointer back to the manager in
order to post tasks. This is wrong because the task runner and hence
the weak pointer may be accessed from arbitrary threads, which is
not safe.

This patch corrects the problem by combining the task runner and the
incoming task queue into a single task queue object which owns both
the incoming task as well as the pointer back to the task queue
manager. This allows us to protect the incoming task queue and the
pointer using the same lock.

BUG=391005

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

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

Layout tests currently utilize a mocked NotificationPresenter implemented
in the test runner, but this won't be sufficient for Web Worker-based
notifications given that these won't be tied to a WebFrame.

Instead, pull up the layout test functionality to the browser process in
content_shell (only available when running layout tests!), similar to the
ability to request Notification permissions, so that we can allow displaying
and interacting with "displayed" notifications in layout tests as well.

The existing code path for Document-based Notifications is kept intact, but
will be removed once both Blink and Chromium switch over.

BUG=392187

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

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

This was a compiler error on GCC 4.9.1:
unable to deduce ‘const auto*’ from ‘((((content::PpFrameWriter::FrameWriterDelegate*)this)->content::PpFrameWriter::FrameWriterDelegate::endian_ == (content::PpFrameWriter::FrameWriterDelegate::PixelEndian)1u) ? libyuv::BGRAToI420 :  libyuv::ARGBToI420)’

Introduced in https://codereview.chromium.org/663183002

BUG=426020

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

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

BUG=406995
TEST=none

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

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

BUG=422235
R=sergiyb@chromium.org

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

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

Use size_t throughout, and avoid mixing size_t and uint32_t.

No behavioural effect.  Unsigned leb128 is currently used
only on 32-bit arm, and here uint32_t and size_t are equivalent.

BUG=385553

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

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

BUG=426020

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

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

BUG=

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

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

The removed comment speaks about StubContentSettingsClient, which was not created in https://codereview.chromium.org/646973002 after all, and thus does not exist in the ToT.

This is a comment-only change, thus skipping tests.

TBR=blundell@chromium.org
NOTRY=true
BUG=393248

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

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