GitLab

To ensure layer is set needsRepaint().

BUG=536999
TEST=scrollbars/scrollbar-added-during-drag.html etc with sync painting enabled

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/2280bfa0..01158a14

Please follow these instructions for assigning/CC'ing issues:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

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

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

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

BUG=none
TEST=none
R=ben@chromium.org

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

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

This removes the DocumentTabObserver as a separate class inside
DocumentTab. It also add the favicon image as a param to
onFaviconUpdated inside TabObserver and uses that to update the
icon used for task descriptions to largest size in both Document
and WebAppActivities.

BUG=546182

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

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

For a normal SQLite build using uClibc, autoconf would generate a
config.h which SQLite would use.  Chromium's build system doesn't make
it easy to use different versions of a file depending on platform.
Use SQLite's config.h support to build a hybrid which allows additional
preprocessor detection.

BUG=489784

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

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

Revert of DevTools: Add dark theme hidden experiment (patchset #2 id:20001 of https://codereview.chromium.org/1458223005/ )

Reason for revert:
Since the theme does not work and can't be used, this should really be behind 10 shifts.

Original issue's description:
> DevTools: Add dark theme hidden experiment
>
> BUG=559524
>
> Committed: https://crrev.com/a7d0f43f83b514a46ec3850aab3160412cdd8a53
> Cr-Commit-Position: refs/heads/master@{#361015}

TBR=paulirish@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=559524

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

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

BUG=510485

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

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

This fixes initializer for benchmark_enabled to actually check for both benchmark and
latencyInfo to match the categories actually used in trace events and renames it to
latency_info_enabled.

BUG=520659

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

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

R=tkent
BUG=

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

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

win platform to be 1 to temporarily fix the race condition on
win bot.

BUG=559154
NOTRY=true

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

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

It used to be that WebContentsImpl only knew about the RWHs created
  through WebContentsImpl::CreateNewWidget() and stored them in
  WCI::created_widgets_. However, widgets created through
  FrameTree::CreateRenderViewHost() used to be left out.
This made RenderWidgetHostImpl::WasResized() to incorrectly send
  ViewMsg_Resize while the WebContents was being torn down. WasResized()
  has a check for !delegate_ but |delegate_| never became nullptr during
  WebContentsImpl shutdown. Adding all widgets to WCI::created_widgets_
  will ensure that we notify RWHs about the delegate's (WC) destruction
  via DetachDelegate and do not send ViewMsg_Resize while a the delegate
  (WC) is shutting down.

BUG=523644, 523451
Test=See https://code.google.com/p/chromium/issues/detail?id=523644#c16
  for repro steps. Ensure the renderer does not crash.

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

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

There are instances where callbacks and events can be triggered
and then the host tab for Reader Mode is destroyed. This change
adds checks for these cases.

BUG=556930, 556456, 556354

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

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

BUG=558839

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

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

TBR=brettw@chromium.org
NOPRESUBMIT=true

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

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

TBR=haraken
BUG=559300

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

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

BUG=527856

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

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

Revert of views/mus: Fix showing menu in chrome in mus. (patchset #2 id:20001 of https://codereview.chromium.org/1460003004/ )

Reason for revert:
Suspected in build failure. see crbug.com/560461

Original issue's description:
> views/mus: Fix showing menu in chrome in mus.
>
> For menus in chrome-in-mus, we attempt to create a Widget hosted in a native
> window. For desktop builds, this means we attempt to host in a HWND or an
> X11 window created by DesktopNativeWidgetAura. But for mus, we want to use
> NativeWidgetMus. To do this, use the native-widget factory first if it's
> available.
>
> BUG=none
>
> Committed: https://crrev.com/2914df72a605451828a05e108c35fe9a1d72c971
> Cr-Commit-Position: refs/heads/master@{#361151}

TBR=sky@chromium.org,jam@chromium.org,sadrul@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=none

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

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

We'll need this for cursors and shadows.

BUG=548426
TEST=none
R=erg@chromium.org

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

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

BUG=557739

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

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

1. Scrolls the page when navigating to a node, and updates the active indicator
location when a scroll event is received.

2. Update the active indicator location on every Output, rather than only
for speech or only for braille. (It was sometimes getting set and then
overwritten with nothing.)

3. Handle subranges within a node with a new boundsForRange API.

4. Automation API: implement boundsForRange, take frame offsets and
scroll offsets into account when returning a node's bounding box,
and use heuristics to avoid returning an empty bounding box.

BUG=537248
TBR=aboxhall

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

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

TEST=conformance2/renderbuffers/framebuffer-test.html, gpu_unittests

BUG=429053

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

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

Previously, the interface for URLRequestJob::ReadRawData was as follows:

  bool ReadRawData(IOBuffer*, int, int*)

Subclasses were expected to signal completion of the ReadRawData call by
calling NotifyDone, SetStatus, or maybe one of the other Notify* functions
on URLRequestJob, most of which do internal housekeeping and also drive the
URLRequest's state machine. This made it difficult to reason about the
URLRequestJob's state machine and needlessly complicated most of URLRequestJob.

The new interface is as follows:

  int ReadRawData(IOBuffer*, int)

Subclasses are required to either:

a) Return ERR_IO_PENDING, and call ReadRawDataComplete when the read completes in any way, or
b) Return a count of bytes read >= 0, indicating synchronous success, or
c) Return another error code < 0, indicating synchronous failure.

This substantially narrows the interface between URLRequestJob and its
subclasses and moves the logic for the URLRequest state machine largely into
URLRequestJob.

Also, the signature of URLRequestJob::ReadFilteredData and some other internal
URLRequestJob helpers changes to propagate detailed error codes instead of
coercing all errors to FAILED.

TBR=michaeln@chromium.org,mnaganov@chromium.org,skyostil@chromium.org,eugenebut@chromium.org,davidben@chromium.org,falken@chromium.org,mtomasz@chromium.org, sky@chromium.org,jianli@chromium.org,zork@chromium.org,rdsmith@chromium.org
BUG=553300

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

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

BUG=

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

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

BUG=

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

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

Previous change was not correct and lacked sufficient testing. Now testing with two forms to test for correct vector order.

BUG=555025
TEST=AutofillManagerTest

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

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

BUG=560331

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

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

BUG=536895,543405

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

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

The following items are included in this CL.

1- Javascipt column for child processes that don't use V8 memory should always show "N/A".
2- Idle wakeups should show up only once per process.
3- NaCl debug ports column shows "Disabled" when the enable-nacl-debug flag is disabled.
4- NaCl modules should show the name of the app/extension instead of the URL.
5- Network usage can now be reported per process.

R=nick@chromium.org
BUG=551580

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

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

For menus in chrome-in-mus, we attempt to create a Widget hosted in a native
window. For desktop builds, this means we attempt to host in a HWND or an
X11 window created by DesktopNativeWidgetAura. But for mus, we want to use
NativeWidgetMus. To do this, use the native-widget factory first if it's
available.

BUG=none

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

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

I'm going to need more than one object to detect focus changes.

BUG=548424
TEST=covered by tests
R=ben@chromium.org

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

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

Fix the missing "data_store.pb.h" error when building extensions gn target on a clean output directory.

data_reduction_proxy_api.cc includes data_store.pb.h directly and indirectly (via data_reduction_proxy_compression_stats.h). This CL adds //components/data_reduction_proxy/proto:data_reduction_proxy_proto to the dependencies of extensions to solve this problem.

BUG=560321

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

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

BUG=547246

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

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

Improperly-formatted WAV data causes this class to attempt to divide by
zero in the constructor. Check for this condition, and invalidate
internal state if the WAV data does not parse properly. It should be able
to accept any data without crashing. Add unittests to verify.

Bug: b/21759728

BUG=
TEST=wav_audio_handler_unittest.cc

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

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

1) Set the enable status of icon and label before it's being added as a
child of HoverHighlightView

2) Refuse SetHoverHightlight to true when disabled, and SetHoverHightlight
to false when OnEnabledChanged triggered and enabled() changed to false.

BUG=560013, 560027

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

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/a40b4f5c..2280bfa0

Please follow these instructions for assigning/CC'ing issues:
https://code.google.com/p/v8-wiki/wiki/TriagingIssues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

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

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

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

BUG=208378

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

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

Putative fix for crbug.com/545479. If WebUIManager is created but
navigation occurs before the WebUI URL starts loading, provisional
navigation for a non-WebUI URL could be observed by WebUIManager, and
WebUI should not be loaded in that case.

BUG=545479

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

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

This used to work because the accessibility audit library pulls in parts
of the closure libray. Since it doesn't leak its internals in a newer
version, this dependency problem was exposed. The underlying issue is that
GEN_INCLUDES in the js test are included before closureModuleDeps, so
declarations from the latter are not available when evaluating the former.

Needed for https://codereview.chromium.org/1447693002/

BUG=555753
R=aboxhall@chromium.org

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

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

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

BUG=295792, 483282, 559359, 559362, 559342
NOTRY=true

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

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