GitLab

The following supressions are no longer needed:
bug_94195
bug_94345
bug_96069
bug_96568a
bug_96568b
bug_98568
bug_99304
bug_99308
bug_99321
bug_99435

BUG=94195,94345,96069,98568,99304,99308,99321,99435
TBR=groby@chromium.org
NOTRY=true

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

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

Cleaning up weak_ptr_factory destruction order in "src/chrome/browser/history"
module. WeakPtrFactory should remain the last member so it'll be destroyed and
invalidate its weak pointers before any other members are destroyed.

BUG=303818

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

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

TBR=gene@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/29c857d..a392c9d

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

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

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

Cleaning up weak_ptr_factory destruction order in "src/chrome/browser/chromeos"
module. WeakPtrFactory should remain the last member so it'll be destroyed and
invalidate its weak pointers before any other members are destroyed.

BUG=303818

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

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

BUG=416214
TEST=Manual, see bug

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

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

This patch adds a note to the FeatureObserver summary in histograms.xml
about the incorrect numbers due to crbug.com/341631.

BUG=341631

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

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

Given that MediaStream objects in the Blink side are allocated on Oilpan's heap, we need to trigger a GC to collect MediaStream objects in the Chromium side that depend on the lifetime of the MediaStream objects in the Blink side.

This CL adds Heap::collectAllGarbage to TearDown methods of MediaStream content_unittests. This guarantees that MediaStream objects used in the content_unittests are collected before the content_unittests finish. Otherwise, the LSan complains that the MediaStream objects leak in the content_unittests.

This is an issue that happens only in testing.

BUG=340522

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

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

Adding CanMinimize() (like CanMaximize()) function to WidgetDelegate class.
So that ColorChooserView or any other view inheriting WidgetDelegate, can
decide whether its window can show minimize button.

BUG=365768

R=sadrul@chromium.org

TEST=
1. Go to a page with <input type=color>. Here is one http://jsbin.com/yotasure/1
2. Click on the color input button to bring up the color picker window and observe
3. Color picker window should not have minimize button.

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

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

This API implementation is compiled on Chrome OS only, so the permission should
be available on Chrome OS only.

TEST=Tested manually that still works on Chrome OS.
BUG=248427

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

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

ImageButton::SetMinimumImageSize allows the client to explicitly set a
size larger than the image contained within the button. The image will
be aligned inside the button according to its alignment setting.

ImageButton::SetPreferredSize had a misleading name (since in fact it
only sets the size when there is no image) and is no longer needed. Its
original usage (r50040) is now gone. It wasn't having any effect in
either of its two remaining callers, since they always set an image
immediately afterwards. Therefore, deleted this method and removed both
calls to it. Note that we are keeping the default behaviour that an
ImageButton with no image has a size of 16x14. I wanted to remove this,
but the list of potential clients that could be relying on it is huge.

BUG=415909

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

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

The default app icon resource exists in the //extensions module resources
file and all implementations return the same icon. There's no need to delegate
this out.

BUG=none
TEST=app_shell, athena_main and chrome all run in debug without assertions
about missing resources

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/fdff3fb..8bc0f5b

TBR=

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

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

the notification order test.

* Before this patch, file_system_operation_impl_unittest can not test
  the notification order because it doesn't have the update observer.

BUG=410198
TEST=content_unittests

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

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

Test input streams that do not meet encoder size requirements exactly.
Because ARM platform need 64 bytes alignment for starting address of YUV
planes of frames, use MemoryMappedFile to create input buffer and copy
data line by line from visible size to coded size. So VEA test can
test more Hangout resolutions now.

Original flow: prepare input memory -> initialize VEAClient -> initialize Encoder
New flow: initialize VEAClient -> get coded size -> prepare input memory

BUG=378166
TEST=Manually run auto test with h264 and vp8 on peach pit.
     Pass ./base_unittests --gtest_filter=MemoryMappedFileTest.*

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

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

BUG=377410
R=vabr@chromium.org,dbeam@chromium.org
TEST=In chrome://settings/passwords, revealed passwords should be editable.

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

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

BUG=
TBR=stgao@chromium.org

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

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

NOTRY=true

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/70e09f7..fdff3fb

TBR=

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/00a138c..29c857d

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

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

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

These tests are flaky on OSX 10.6.
Also revert attempted fix: https://codereview.chromium.org/568673004 .

BUG=405386

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

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

Provide a method to get the chromium version for use as the
data reduction proxy version.

BUG=410127

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

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

BUG=408964
TEST=Manual

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

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

BUG=339240
TBR=joaodasilva@chromium.org

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

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

Broke check_licenses step http://build.chromium.org/p/chromium/builders/Linux/builds/53652/steps/check_licenses/logs/stdio

This reverts commit 7c33f442.

R=dzhioev@chromium.org
BUG=

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

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

BUG=348066
TEST=None
R=dpranke@chromium.org

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

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

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

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

Summary of changes available at:
https://chromium.googlesource.com/skia/+log/3c196de..00a138c

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

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

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

Disable NaClBrowserTestNewlib.IrtException on Windows to see if that gets rid of the hung browser test binaries.

BUG=416272
TBR=uekawa@chromium.org

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/e266d9c..70e09f7

TBR=

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/bcf65c8..e266d9c

TBR=

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

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

Sometimes, the status icon was shown in unexpected timing. This might be because the animation is re-fired when the dom is restyled. This patch fixes it.

BUG=none
TEST=Open window->play video->pause->close window. Repeat it 20 times and observe no issue.

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

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

Summary of changes available at:
https://chromium.googlesource.com/chromium/blink/+log/665ffc7..bcf65c8

TBR=

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

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

Previously, thumbnails of offline-available non-image files were not shown. This was because we tried to show self-generated thumbnails when the file cache was present. But we can't generate thumbnails of non-image so the thumbnails of video files were not shown.

With this patch, if thumbnail can't be generated, the thumbnail from the external provider shows instead.

BUG=415048
TEST=manual tested

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

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

List of changes:
* Added all core-elements and paper-elements provided by Polymer.
* Added web-animations-js (a dependency of core-animation element).
* Components moved to 'components' subdirectory.
* Added a presubmit check verifying that the list of dependencies from
  bower.json and the list of components installed to 'components' are the same.
  (see README.chromium for details).
* Updated README.chromium
* Updated pathes to polymer.js and platform.js in projects that use them.

This CL doesn't change polymer and platform components.

Landed manually, because of the bug in the commit queue (http://crbug.com/416255).

BUG=415696
R=mtomasz@chromium.org, raymes@chromium.org, rsadam@chromium.org
TBR=thestig@chromium.org

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

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

BUG=330264

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

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

Adding registration ID request and tests to GCM Account Mapper
* Adding code to get registration ID from GCM
* Making sure account tokens are only processed when reg ID is present
* Remembering only the last assigned set of account tokens
* Fixing tests that require registration ID
* Adding tests specific to registration ID setting

BUG=374969
R=zea@chromium.org

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

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

The implicit conversion has already been disabled for Linux and ChromeOS
builds. https://codereview.chromium.org/510323002 explains the rationale
for proceeding incrementally.

BUG=110610
TBR=thakis@chromium.org

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

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

Previously, if exact_matches was True but there were no exact matches,
it would return fuzzy matches instead. This is not what the caller asked
for.

GPU test names are changed to match the buildbot config, since they were
previously relying on fuzzy matching.

SHERIFFS: May cause errors like:

  No test named "some_test_name".

  Available tests are:
    ...

If so, this CL can be safely reverted.

BUG=413334,413442

Committed: https://crrev.com/566ed9f6e71cea313c88e8f259eca76581b72d67
Cr-Commit-Position: refs/heads/master@{#294757}

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

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