GitLab

BUG=

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

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

When shutting down HostProcess config_watcher_ is supposed to be deleted
on the nework thread. Previously ShutdownOnNetworkThread() was deleting
it, but that line was mistakenly removed in crrev.com/302162 . Adding
it back.

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

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

Update ImportController to push all updates to a new 'CommandWidget' class. Add more test coverage.

BUG=420680
TEST=browser_test: FileManagerJsTest.*

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

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

Add calls to hash generation and comparison code so that files being imported
into Google Drive are actually deduplicated.

BUG=420680
TEST=FileManagerJsTest.DuplicateFinderTest FileManagerJsTest.MediaImportHandlerTest

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

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

BUG=429254

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

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

The following lines are repeated in almost every test cc/scheduler/scheduler_unittest.cc file
  FakeSchedulerClient client;
  SchedulerSettings scheduler_settings;
  scheduler_settings.use_external_begin_frame_source = true;

  CREATE_SCHEDULER_AND_INIT_SURFACE(scheduler_settings);

created test fixture class and clean up to remove duplicate codes.

BUG=449028

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

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

Updated code so that the first row is focusable. This makes it possible to
tab into the first row making it more obvious that it is focused.

Applied Dan's feedback from previous CL (https://codereview.chromium.org/834323005).

BUG=449110

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

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

BUG=410900

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

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

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

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

moment there is only total time and time to load all non-component extensions.
Time to load component extensions and time to notify all listeners is not
instrumented.

R=mek@chromium.org, mpearson@chromium.org
BUG=453073

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

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

In Chrome Apps, some platform APIs can only be called from the background
page (e.g. reloading a chrome.app.AppWindow).  Likewise, some chrome API's
must be initiated by user interaction, which can only be called from the
foreground.

This CL base.IPC that provides helper functions to invoke methods on
different pages using chrome.runtime.sendMessage.

BUG=452317

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

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

BUG=417463
TEST=build with chromeos=1 use_ozone=1

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

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

This is the first step on support 'add to homescreen' banners for sites
that have manifests.

BUG=452825

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

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

We want to see how much GPU initialization contributes to startup slowness.

BUG=125248

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

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

"compositor: Fix texture flipping for SW mirroring with surfaceless"
and
"Exclude reflector_impl_unittest.cc from GN non-Aura, non-Mac builds."

Reason for revert:
The original patch breaks software mirroring mode on at least veyron_jerry,
and quite possibly all non-freon systems.

>Original issue's descriptions:
>   Exclude reflector_impl_unittest.cc from GN non-Aura, non-Mac builds.
>    This fixes the GN build on Android after https://codereview.chromium.org/846063002.
>
>   (The GYP build didn't break because content/browser/compositor/ is completely excluded there; see https://code.google.com/p/chromium/codesearch#chromium/src/content/content_tests.gypi&sq=package:chromium&&type=cs&l=1099)
>
>   TBR=bauerb@chromium.org
>
>   Review URL: https://codereview.chromium.org/879543002
>
>    Cr-Commit-Position: refs/heads/master@{#313074}

> compositor: Fix texture flipping for SW mirroring with surfaceless
>
> Ozone-surfaceless renders the scene flipped, so the texture contents are
> inverted from what they should be. This fixes the mirroring logic to
> un-flip the texture when needed, and also to send the right sub-buffer
> update rect.
>
> BUG=434115
>
> Committed: https://crrev.com/c6a481fe79e0c4851a576495105edf2600bfe5be
> Cr-Commit-Position: refs/heads/master@{#312979}

TBR=danakj@chromium.org,achaulk@chromium.org,bauerb@chromium.org
BUG=434115,451804

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

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

Also renames existing CreateSession to GenerateRequest to better match
the purpose of the promise.

BUG=351501
TEST=existing EME tests pass

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

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

1. Use auto to improve readability.
2. Rename methods to remove "Extension" prefix.
3. Use const where possible.
4. Use using content::* to improve readability.

BUG=none

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

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

NOTRY=True

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

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

Java layer needs an exact way to determine whether a URL uses an external
protocol or not.
is_external_protocol will be true if a URL can't be handled by Chrome's
internal protocol handlers.

BUG=426751

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

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

When chrome.usb.closeDevice is called the device is explicitly closed in
the extension function implementation and then closed again when the
UsbDeviceResource object is destroyed. Only the latter is necessary as
that covers other ways in which the resource can be released.

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

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

This change merges the initial call to SetCommState/tcsetattr with
the call to ConfigurePort so that parameters such as baud rate are
configured immediately when the port is opened. Default values for
these parameters are now always applied.

This works around an issue with some serial adapter drivers on Windows
that report an invalid baud rate through GetCommState. Having the
intended baud rate available for the first call to SetCommState allows
an application to provide a valid value so that Chrome does not have
to guess.

BUG=448407

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

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

This is in conformance with the CA/Browser Forum Baseline Requirements for
certificate issuance.

This CL is adapted from a diff provided by sigbjorn@opera.com. Thanks!

BUG=119211
TBR=phajdan.jr@chromium.org

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

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

I've noticed that the localization of the strings are done differently, so
it's better to use the same string for both.

BUG=

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

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

Add a GetDisplayColorProfile variant on Mac to return the display
color profile associated with a native window (NSWindow).

Add unit tests for on-screen, partially-on-screen, off-screen and
empty and null windows.

Note an sRGB color profile is a common case, especially on win32,
so GetDisplayColorProfile returns true and an empty color profile
in that case. Add a header file comment about that.

TEST=gfx_unittests --gtest_filter="ColorProfileTest*Window"
BUG=368694

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

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

Picked up changes:
56f7579 Rewrite SimilarSiblingContentExpansion
2fda06e Fix printing of logged non-ascii characters
8f4713b Only strip one stack frame from JS exceptions
efa4dce Remove unnecessary indent from code generation.
6134342 Fix TerminatingBlocksFinderTest
1f237e8 Make TerminatingBlocksFinder use a RegExp
80a34cf Change LogUtil.LOG_BUILDER to a String
9b2cdb2 Move most tests from test/ to jstest/
169cb83 Add JavaScript.parseFloat and parseInt
22cebe8 Change how TextBlock nodes are tracked
9be60c8 Add correct imports for generated JsTestEntryImpl
6b17bca Convert uses of emulated Pattern/Matcher to RegExp
c6bbc1e Add StringUtil.countWords and simplify TextBlockBuilder
7e2865e Simplify TextBlockBuilder
c73a770 Split the handling of TextNodes out of BHTMLCH
8fad736 Make test debugging more better
3e1e81e Handle equally scored candidate lead images.
a7ec3c8 Make JsTestEntry report correct success result
d2213f3 Remove unused TextBlock statistics
755df35 Add some more fancy gwt flags
6380f5c Add ElementAction that simplifies TagAction+labelStack
870b65e Move ignoring elements into FilteringDomVisitor.
48aaaa2 Use JUnit-style null assertions in OpenGraphProtocolParserTest
4ba963b querySelectorAll usage in parseMetaTags is covered in jstests
6bb3e26 Add DomDistillerJsTestCase
3f5efb3 Flesh out jstest environment
ef708c4 Use bash in install-build-deps.sh
9ef7a04 Add protobuf and xvfb packages to build deps.
33b9a04 Implement rest of the Assert interface
d75ad6b Correct function name
1d72a60 Scope repeated value for JSON converter.
e482386 Fix install-build-deps.sh for missed package
da40e84 Make jstests not fail the build
0d6b80e Add automated driver for jstests
92ae3ba Add better stack traces for jstests
a97c624 Modular heuristics for images
3bf867e Add framework for running tests in Chrome
cea631f Add .clang-format file to enable git cl format
fc23a51 Make image source absolute.
486fd14 Use java 7 and closure compiler
0dbffc4 Fix TextDocumentConstruction expectations
05c76c0 Remove gwt-2.5.1 and gwt_exporter
ed4df25 Roll gwt to 2.7.0 (from 2.5.1)
6dc14d3 Optimize SchemaOrgParser.parse for speed
d7c6805 Optimize OpenGraphProtocolParser.parseMetaTags for speed
64df32e Add fine-grained speed profiling inside MarkupParser
cd97d4f Remove a bunch of unused stuff
bf5347c Refactor handling of labels and tagactions
03cbda8 Rewrite DomWalker.walk
ee46df0 Create a simple reader mode extension

BUG=431067,435264,435371,440102,440977

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

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

BUG=453037

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

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

So that changes to this file do not need approval from
chromeos/OWNERS

BUG=none

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

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

So that changes in build files don't need approval from
google_apis/OWNERS.

BUG=none

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

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

R=spang

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

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

BUG=0

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

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

Revert of Tweak output of (memcheck|drmemory)_analyze.py. (patchset #1 id:1 of https://codereview.chromium.org/884813002/)

Reason for revert:
broke tree. don't use NOTRY=true again http://build.chromium.org/p/chromium.linux/builders/Linux%20Tests%20%28dbg%29%281%29/builds/38252

Original issue's description:
> Tweak output of (memcheck|drmemory)_analyze.py.
>
> Wrap error reports in "### (BEGIN|END) MEMORY TOOL REPORT". This will allow us
> to extract them with GTestLogParser and use them as snippets.
>
> BUG=453020
> R=thestig@chromium.org
> NOTRY=true
>
> Committed: https://crrev.com/9794e18ce8653e49c1b9f6da0033276a79f88579
> Cr-Commit-Position: refs/heads/master@{#313572}

TBR=thestig@chromium.org,earthdok@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=453020

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

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

With the new profiles UI and the high res avatar downloading code, we are sending out
waaaay too many OnProfileAvatarChanged notifications. This is particularly terrible on
Windows, where basically on every Chrome startup we refresh all the icons.

Most importantly, we need to stop downloading the avatar files every time profiles get
loaded in the ProfileInfoCache. We did this originally because the avatar files were pretty
volatile, but it's safe to assume they've stopped changing.

Secondly, the only things that care about the high res avatar files are the UserManager
and the profile switcher, so they should get a special notification that this particular
file is ready, rather that annoy all the other listeners who only care about the tiny avatars.

BUG=449569

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

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

This is the first patch to separate JNI registration with initialization, currently, we have JNI_OnLoad defined multiple places which has a lot of
duplicated code and JNI registration and initialization are mixed;

This patch
- Added JNIOnLoadDelegate class for each component  specific initialization.
- Added base::android::OnJNIOnLoad() to call a list of delegates
- Migrate testing, content_shell, chrome_shell.

BUG=447393
TBR=thakis

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

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

Fix a crash which occurs due to a CHECK firing in the HWNDMessageHandler::OnKeyEvent code path on Windows.

This crash occurs when the LegacyRenderWidgetHostHWND instance sends WM_CHAR/WM_SYSCHAR messages via the
WindowEventTarget::HandleKeyboardMessage function.

The HWNDMessageHandler class handles these messages via the OnImeMessages handler in the normal case.

The CHECK fires in the KeyEvent ctor where we validate that the KeyEvent instance has a valid event type.
In this case the WM_CHAR/WM_SYSCHAR messages are processed via the OnKeyEvent handler which leads to the problem.

Fix is to add a check for WM_CHAR/WM_SYSCHAR messages in the HWNDMessageHandler::HandleKeyboardMessage function
and call the OnImeMessages handler there. The alternative would be to add a new method to the WindowEventTarget
interface and change the LegacyRenderWidgetHostHWND class to handle these messages separately and call the new
method. I think that change would be a touch big for this case.

I also added WM_SYSDEADCHAR to the list of messages processed by the EventTypeFromNative function. We set the
type as ET_KEY_RELEASED on the same lines as WM_DEADCHAR.

BUG=451797
TEST=Covered by views_unittest WidgetTest.CharMessagesAsKeyboardMessagesDoesNotCrash

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/073720e..52f401e

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

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

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

BUG=369661
TEST=tests stay green

Committed: https://crrev.com/66f3ec8e5ee6c3b528b5a6054cdca85ac0b29699
Reverted: https://crrev.com/96545af0f94ec1ac19a57c86806c95834de5d33c

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

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

With throttled, async touch dispatch, an uncancelable touchmove is sent
as soon as the touch sequence exceeds an arbitrarily defined
application slop region, separate from the platform slop region. The
intent in doing so was to allow web apps to clear any tap/press-related
state after a scroll has begun. However, the sending of this touchmove
at the start of a scroll sequence can be problematic; the early part of
a scroll sequence is often sensitive to minor bottlenecks, and the
dispatch of an additional touchmove can conceivably push the app into
high latency mode by delaying frame production.

Stop sending this slop-exceeding touchmove, instead relying on the
existing timer-based async touchmove dispatch.

Note that while this may regress touch behavior on certain sites (e.g.,
causing faulty tap interpretation), it's likely that behavior on such
sites was already racy and fragile, subject to the arbitrary value of
the fixed "application" slop region.

BUG=449275

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

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

This pulls in the following Native Client changes:

58c9abe: (dschuff@chromium.org) Update revision for PNaCl
fd92de8: (teravest@chromium.org) Revert "Output depfile from build_nexe."
fb70836: (dschuff@chromium.org) PNaCl: Update libc++abi revision in pnacl/COMPONENT_REVISIONS
939a13a: (teravest@chromium.org) Provide an irt_core_lib GN target.
dbf2af4: (sbc@chromium.org) Remove include of build_config.h from nacl_exception.h
e205317: (jfb@chromium.org) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS
7c8a96b: (jfb@chromium.org) Update revision for PNaCl
3d00a54: (jfb@chromium.org) PNaCl: Update LLVM revision in pnacl/COMPONENT_REVISIONS
1de8ba8: (mseaborn@chromium.org) Update codereview.settings following the switch from SVN to Git
7a6cf46: (jfb@chromium.org) Update revision for PNaCl
bc28c0b: (jvoung@chromium.org) Fix the native_client URL in the PNaCl REV file.
f8aa9dc: (bradnelson@google.com) Adding le32-nacl build of pnacl tools.
cfda521: (teravest@chromium.org) Explicitly include build_config.h.
61b7ba9: (teravest@chromium.org) Output depfile from build_nexe.
2d65c23: (jfb@chromium.org) PNaCl: Update LLVM and Subzero revision in pnacl/COMPONENT_REVISIONS
5f5cd88: (sbc@chromium.org) Fix PNaCl redirectors for i686-nacl-xx -> x86_64-nacl-xx
1baa204: (jvoung@chromium.org) Remove some more dead code and variables from pnacl/build.sh.
f116471: (dschuff@chromium.org) Update revision for PNaCl
01f3d97: (teravest@chromium.org) Always define product_dir in build_nexe.
8a5a846: (mseaborn@chromium.org) Remove the IRT's now-unneeded call to NaClSrpcModuleInit()
c14b58a: (mseaborn@chromium.org) Remove NaClPluginLowLevelInitializationComplete() (caller of "init_done" SRPC)
c5d8394: (bradnelson@google.com) Change driver bash scripts to work in devenv.
490e554: (dschuff@chromium.org) PNaCl: Update Clang revision in pnacl/COMPONENT_REVISIONS
2a7391a: (sbc@chromium.org) Fix several issues with update_pnacl_tool_revisions.py
fbc3b5c: (dschuff@chromium.org) Support filenames with spaces in the PNaCl driver scripts.
4f726e8: (sbc@chromium.org) PNaCl: Update Clang revision in pnacl/COMPONENT_REVISIONS
28a1341: (dschuff@chromium.org) Move pnacl driver test runs from buildbot_pnacl.sh to buildbot_pnacl_toolchain.py
9f9d773: (sbc@chromium.org) Update GCC revision in tools/REVISIONS
29ae54a: (jfb@chromium.org) Update revision for PNaCl

BUG=none
TEST=browser_tests and nacl_integration
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32,linux_arm_compile,linux_nacl_sdk

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

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

This will make it have the same behaviour as the User Manager, vs. navigating to
chrome://settings/createProfile.

BUG=452125
TEST=Choose Add Person from the OS menu bar. This should create a profile and
switch to it immediately.

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

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

net: fix typo and add ustr.gov to HSTS preloaded list.
Cr-Commit-Position: refs/heads/master@{#313585}