- 31 Oct, 2014 1 commit
-
-
japhet authored
This involves a couple of browser-side plumbing changes, as well as teaching RenderFrameProxy how to swap itself out in favor of a RenderFrameImpl. BUG=422583 Review URL: https://codereview.chromium.org/600553003 Cr-Commit-Position: refs/heads/master@{#302193}
-
- 30 Oct, 2014 39 commits
-
-
morrita authored
TBR=yurys BUG= Review URL: https://codereview.chromium.org/691903003 Cr-Commit-Position: refs/heads/master@{#302192}
-
jbudorick authored
BUG=428729 Review URL: https://codereview.chromium.org/690973002 Cr-Commit-Position: refs/heads/master@{#302191}
-
peter authored
Many implementations of the NotificationDelegate include empty overrides because of behavior they don't actually need to be informed of. Provide empty default implementations so that they don't have to. (And remove the associated TODO.) BUG=428370 Review URL: https://codereview.chromium.org/684253002 Cr-Commit-Position: refs/heads/master@{#302190}
-
skia-deps-roller authored
Summary of changes available at: https://chromium.googlesource.com/skia/+log/e228ba3..9bde31e CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg TBR=hcm@google.com Review URL: https://codereview.chromium.org/688923002 Cr-Commit-Position: refs/heads/master@{#302189}
-
mdempsky authored
Review URL: https://codereview.chromium.org/691803004 Cr-Commit-Position: refs/heads/master@{#302188}
-
danduong authored
Revert "Revert of Reduce frequency of requesting bookmark clusters. (patchset #8 id:140001 of https://codereview.chromium.org/682933002/)" This reverts commit 8077169f. BUG=424254 Review URL: https://codereview.chromium.org/695493004 Cr-Commit-Position: refs/heads/master@{#302187}
-
kenobi authored
The new flag carries the has_media status of a volume through from the DiskMountManager, so the information is available to chrome.fileManagerPrivate.onMountCompleted javascript listeners. This is a re-land of issue 668013004, which fixes some initialization problems in VolumeInfo, which were causing flakey behaviour in tests. BUG=420680 TEST=browser_tests: FileManagerPrivateApiTest.* FileManagerJsTest.* Review URL: https://codereview.chromium.org/677413007 Cr-Commit-Position: refs/heads/master@{#302186}
-
sebmarchand authored
BUG=314403 Review URL: https://codereview.chromium.org/691193002 Cr-Commit-Position: refs/heads/master@{#302185}
-
sebmarchand authored
TBR=chrisha@chromium.org BUG= Review URL: https://codereview.chromium.org/680233004 Cr-Commit-Position: refs/heads/master@{#302184}
-
nfullagar authored
BUG=https://code.google.com/p/nativeclient/issues/detail?id=3987 TEST=NaCl SDK example and run on NaCl Perf Bots R=bradnelson@chromium.org Review URL: https://codereview.chromium.org/664543003 Cr-Commit-Position: refs/heads/master@{#302183}
-
sandersd authored
BUG=133828 Review URL: https://codereview.chromium.org/642453003 Cr-Commit-Position: refs/heads/master@{#302182}
-
jdduke authored
Using the Activity context ensures the content-targetted selection handles use a theme consistent with the rest of Chrome's UI. Review URL: https://codereview.chromium.org/696503002 Cr-Commit-Position: refs/heads/master@{#302181}
-
dcheng authored
The Google C++ style guide states: Explicitly annotate overrides of virtual functions or virtual destructors with an override or (less frequently) final specifier. Older (pre-C++11) code will use the virtual keyword as an inferior alternative annotation. For clarity, use exactly one of override, final, or virtual when declaring an override. To better conform to these guidelines, the following constructs have been rewritten: - if a base class has a virtual destructor, then: virtual ~Foo(); -> ~Foo() override; - virtual void Foo() override; -> void Foo() override; - virtual void Foo() override final; -> void Foo() final; This patch was automatically generated. The clang plugin can generate fixit hints, which are suggested edits when it is 100% sure it knows how to fix a problem. The hints from the clang plugin were applied to the source tree using the tool in https://codereview.chromium.org/598073004. BUG=417463 R=harrym@chromium.org Review URL: https://codereview.chromium.org/684643002 Cr-Commit-Position: refs/heads/master@{#302180}
-
rdevlin.cronin authored
Original was reverted due to a bad merge. Rename BrowserActionView to ToolbarActionView BrowserActionViews now represent significantly more than just Browser Actions. Rename it to ToolbarActionView. BUG=426448 TBR=sky@chromium.org Review URL: https://codereview.chromium.org/653933003 Cr-Commit-Position: refs/heads/master@{#302179}
-
dmazzoni authored
The change to accessibility_focus_ring_layer.cc prevents an endless loop if the input ring is all the same point. The change to accessibility_focus_ring_controller.cc addresses the underlying cause - the timestamp we get from CompositorAnimationObserver::OnAnimationStep is not guaranteed to be later than the most recent focus time (in practice, it's up to 0.002 s earlier on an ARM Chromebook). A negative delta time turned into NaN when we computed pow(fraction, 0.3) and that led to the input ring being all zeros, triggering the issue above. BUG=427649 Review URL: https://codereview.chromium.org/693623003 Cr-Commit-Position: refs/heads/master@{#302178}
-
ananta authored
We will revisit this at a later point in time. At the moment the decision is to have Win7 ASH mode as a developer only option for debug builds of Chrome. BUG=356475 Review URL: https://codereview.chromium.org/692863003 Cr-Commit-Position: refs/heads/master@{#302177}
-
lgarron authored
Also removes the MixedScript_LoadAnyway_Bubble count (unowned, no longer in histograms.xml). BUG=413057 Review URL: https://codereview.chromium.org/659573003 Cr-Commit-Position: refs/heads/master@{#302176}
-
kundaji authored
DataReductionProxyStatisticsPrefs is instantiated on UI thread and then its ownership is passed to IOThread. This causes weak_factory_ to be invalidated on a different thread from the one it is created on. This cl explicitly invalidates it on UI Thread. BUG=428931 Review URL: https://codereview.chromium.org/669783004 Cr-Commit-Position: refs/heads/master@{#302175}
-
reillyg authored
These tests do not need the full power of browser_tests and API tests like these will eventually be moved out of app_shell_browsertests into a new extensions_browsertests target that takes advantage of the lighter app_shell framework. BUG= Review URL: https://codereview.chromium.org/695613002 Cr-Commit-Position: refs/heads/master@{#302174}
-
kkimlabs authored
Missing enhanced bookmark owners coverage from https://codereview.chromium.org/682103003 NOTRY=true BUG=None Review URL: https://codereview.chromium.org/691853003 Cr-Commit-Position: refs/heads/master@{#302173}
-
jamiewalch authored
BUG= Review URL: https://codereview.chromium.org/687873003 Cr-Commit-Position: refs/heads/master@{#302172}
-
bnc authored
Add Spdy4Experiment group which can be kept at the same size as Spdy4Control, while Spdy4Enabled can be arbitrary. This allows enabling SPDY/4 on more than 50% while having a group the same size as Spdy4Control for convenient comparison. BUG=428690 Review URL: https://codereview.chromium.org/689923002 Cr-Commit-Position: refs/heads/master@{#302171}
-
estade authored
CVC isn't recognized. TBR=isherman@chromium.org BUG=428926 Review URL: https://codereview.chromium.org/691073002 Cr-Commit-Position: refs/heads/master@{#302170}
-
navabi authored
I think this is a fix to this CL: https://codereview.chromium.org/656383002/ One of the internal bots has the following error: PullProfile at tools/telemetry/telemetry/core/platform/android_platform_backend.py:455 self._ProfileDir(package), output_profile_path) AttributeError: 'AndroidPlatformBackend' object has no attribute '_ProfileDir' There does not seem to be a _ProfileDir, but there is a _GetProfileDir. BUG=417542 Review URL: https://codereview.chromium.org/664583009 Cr-Commit-Position: refs/heads/master@{#302169}
-
mgiuca authored
The GetAnyChromePath and GetChromePathForInstallationLevel functions previously returned the official Google Chrome executable path, even on Chromium. Now on a Chromium build, they will return the installed path to Chromium, if any. BUG=428600 Review URL: https://codereview.chromium.org/686383002 Cr-Commit-Position: refs/heads/master@{#302168}
-
mbajpai authored
setsockopt() can now be called with these options: SO_RCVBUF SO_SNDBUF BUG=none Review URL: https://codereview.chromium.org/304373008 Cr-Commit-Position: refs/heads/master@{#302167}
-
pavely authored
Currently metadata record only contains size and it is not used anywhere. In the future size will be used for storage management by datatypes. Next step is to add crc to metadata record. BUG=424304 R=maniscalco@chromium.org Review URL: https://codereview.chromium.org/690723004 Cr-Commit-Position: refs/heads/master@{#302166}
-
danduong authored
Revert "Revert of Add Search Service in Enhanced Bookmark Bridge (patchset #6 id:100001 of https://codereview.chromium.org/637323005/)" This reverts commit 1491f336. BUG=415774 TBR=yfriedman Review URL: https://codereview.chromium.org/693613003 Cr-Commit-Position: refs/heads/master@{#302165}
-
estade authored
TBR=isherman@chromium.org BUG=428930 Review URL: https://codereview.chromium.org/688373002 Cr-Commit-Position: refs/heads/master@{#302164}
-
skia-deps-roller authored
Summary of changes available at: https://chromium.googlesource.com/skia/+log/058f01e..e228ba3 CQ_EXTRA_TRYBOTS=tryserver.blink:linux_blink_rel,linux_blink_dbg TBR=hcm@google.com Review URL: https://codereview.chromium.org/693493005 Cr-Commit-Position: refs/heads/master@{#302163}
-
kelvinp authored
This CL links the it2me host to the Chrome binary on ChromeOS behind a flag. The following changes are made to the it2me host so that it can be run in the browser process. 1. Initializes SSL server sockets and specific CPU media features on ChromeOS startup. 2. Fixes a crash in it2me shutdown by making It2meHost owns the ChromotingHostContext. 3. Replace the blocking shutdown wait on PolicyWatcher with a callback. Implements policy_watcher on ChromeOS using policy services. 4. Re-use existing threads, url request context getters and policy service on ChromeOS. 5. Fixed a incorrect DCHECK regarding the color format of the frames captured on ChromeOS. BUG=334087 Committed: https://crrev.com/54dde6f02d121ff745e66b57205583087ff720ec Cr-Commit-Position: refs/heads/master@{#302034} Review URL: https://codereview.chromium.org/639233002 Cr-Commit-Position: refs/heads/master@{#302162}
-
davidben authored
BUG=338884 Review URL: https://codereview.chromium.org/687833002 Cr-Commit-Position: refs/heads/master@{#302161}
-
ccameron authored
Use NSWindowDidChangeOcclusionStateNotification (new in 10.9 Mavericks) to detect when the application is hidden, the window is minimized or obscured by another window. Send the WasHidden to the WebContentsImpl, because this will take into account tab capture. When RenderWidgetHostViewMac::WasHidden is called, the previous behavior was to destroy the BrowserCompositorViewMac. This has the undesirable effect of creating a white flash when a hidden window is re-exposed. To prevent this, only destroy the BrowserCompositorViewMac when the the RenderWidgetHostViewMac is removed from all NSWindows. Manage the state of BrowserCompositorViewMac explicitly as being Active (previously whenever it existed), Suspended (the new state where it is kept around, but the RWH and DFH are hidden), and Destroyed (previously wherever it did not exist). Move some 10.7 APIs to base because they belong there, not in content. BUG=310374 Review URL: https://codereview.chromium.org/684343002 Cr-Commit-Position: refs/heads/master@{#302160}
-
quiche authored
- add a protobuf for WiFi credentials - add a ModelType for syncing WiFi credentials - add UI for controlling whether or not WiFi credentials are synced - add new data type to testserver While there: - update comment in sync_setup_overlay.js - fix ordering of arguments to EXPECT_EQ in CheckBool (sync_setup_handler_unittest) - move GetSelectableTypeNameMap from sync_setup_handler.cc to model_type.cc BUG=chromium:422045 TEST=ProfileSync, SyncSetup, SyncModel, ModelType, NigoriUtil, ProtoEnumConversions, ProtoValueConversions, SyncEncryptionHandlerImpl Review URL: https://codereview.chromium.org/674633002 Cr-Commit-Position: refs/heads/master@{#302159}
-
chirantan authored
Consumers of this class (powerd) might want to know if it actually can wake the system from suspend so that they can try alternatives if it doesn't have this ability. BUG=chrome-os-partner:31970 TBR=derat Review URL: https://codereview.chromium.org/685503006 Cr-Commit-Position: refs/heads/master@{#302158}
-
dpranke authored
There are now files in the primary locations for flac and trace-viewer. R=brettw@chromium.org BUG= Review URL: https://codereview.chromium.org/690613004 Cr-Commit-Position: refs/heads/master@{#302157}
-
kundaji authored
Disable data reduction proxy by deleting user level proxy prefs only if the proxy is known to be a data_reduction_proxy. Remove duplicate implementation in off_the_record_profile_impl.cc. BUG=426220 Review URL: https://codereview.chromium.org/669163003 Cr-Commit-Position: refs/heads/master@{#302156}
-
kundaji authored
BUG=411543 Review URL: https://codereview.chromium.org/691123002 Cr-Commit-Position: refs/heads/master@{#302155}
-
kjoswiak authored
We should not assume that the underlying decoders keep their state after a Seek. So for example, the SPS/PPS should be added again. BUG=140371 Review URL: https://codereview.chromium.org/683573002 Cr-Commit-Position: refs/heads/master@{#302154}
-