| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
| |
that will eventually replace the NetworkLibrary method
of configuring networks.
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/11260047
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@168338 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Rename DiskMountManager::Observer methods to On____Event format.
* Break up DiskMountManagerEventType enum.
* Add separate observer method for formatting events.
* Pass formatting event success using bool instead of adding '!' in front of the
path (but only in events from DiskMountManager; will remove it from
CrosDisksClient events in another patch).
* Add unit tests for formatting handling in DiskMountManager.
(TODO: add more tests)
TBR:vandebo@chromium.org
(for mechanical changes in chrome/browser/system_monitor)
BUG=157587, 126815
TEST=manual: formatting works (and Formatting pending notification goes away)
chromeos_unittests::DiskMountManagerTest.*
Review URL: https://chromiumcodereview.appspot.com/11365142
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@167799 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
The intention for this class is to provide up-to-date state information about the network for the majority of the network related UI. It should also serve as a foundation for classes monitoring the state of the active network.
This code is not designed to provide detailed information about specific network service or device properties, not should it provide the ability to set (configure) those properties.
BUG=154072
Review URL: https://chromiumcodereview.appspot.com/11192024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166375 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This class is compiled and tested but not used in production binary at this moment.
This patch set depends http://codereview.chromium.org/11271040/.
BUG=158265
TEST=ran chromeos_unittest, unit_tests
Review URL: https://chromiumcodereview.appspot.com/11273114
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166176 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This class provide IBus UI related functionality.
This class is compiled and tested but not used in production binary.
This patch set depends http://codereview.chromium.org/11343038/.
BUG=158264
TEST=ran chromeos_unittests, unit_tests
Review URL: https://chromiumcodereview.appspot.com/11342040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@166119 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch set introduce only empty implementation for IBusPanelService.
The actual implementation will be done with http://codereview.chromium.org/11342040/.
BUG=158264
TEST=ran chromeos_unittests, unit_tests
Review URL: https://chromiumcodereview.appspot.com/11343038
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165914 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
The actual implementation will be done with http://codereview.chromium.org/11273114/.
BUG=158265
TEST=ran chromeos_unittests, unit_tests
Review URL: https://chromiumcodereview.appspot.com/11271040
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165912 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
device/bluetooth/. device/bluetooth/ is a new directory to host the bluetooth related files.
I also changed the namespace from chromeos to bluetooth under new directory.
BUG=135470
Review URL: https://chromiumcodereview.appspot.com/11075006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@162811 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
| |
BUG=151685
Review URL: https://codereview.chromium.org/11028089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@161775 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
(i.e. BluetoothAdapter => BluetoothAdapterChromeOs) and creating interfaces: BluetoothAdapter and BluetoothDevice.
This CL does the Step 1 & 2 of:
1) Renames the classes in chrome/browser/chromeos/bluetooth/ chromeos-specific
(i.e. BluetoothAdapter => BluetoothAdapterChromeOs)
2) Create interfaces of the classes in chrome/browser/chromeos/bluetooth/.
These interfaces will be used in the platform-independent logics.
3) Move everything out of chrome/browser/chromeos/bluetooth/ into
devices/bluetooth/ since the code is no longer specific to linux/chromeos.
4) Add Windows implementations. (i.e. Create BluetoothAdapterWindows)
BUG=135470
Review URL: https://chromiumcodereview.appspot.com/10899037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@159615 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
PowerStateOverride isn't dependent on anything within
chrome/, and I want to use it from content/.
BUG=114128
TEST=none
TBR=sky@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10963044
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@158212 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:29557
Review URL: https://chromiumcodereview.appspot.com/10913048
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156468 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=147658
TEST=chromeos_unittests passed all of asan/heapchecker/valgrind tests
Review URL: https://chromiumcodereview.appspot.com/10914211
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156133 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a re-attempt at http://codereview.chromium.org/10916123/, which also ended up being reverted.
This was reverted due to a failing unit test on Win_Aura. Through an unfortunate set of mistakes this builder was not in the default try set or the CQ set, but could close the tree leading to the revert. Through testing I have been able to reproduce the failure on linux_chromeos, so it might be a flakey failure, since it passed sometimes on that bot. This CL is basically the same as the CL list above, except that there is now a window->Focus() call in the unit test, since the reason the test was failing was when it was being made full screen it ceased to visible. This fixes the issue within this test. I will added a comment about this on the bug tracking the underlying issue.
This is a re-attempt at http://codereview.chromium.org/10905026/, which ended up being reverted.
The crash that caused the revert was actually due to powerd not being able to parse the protobuf correctly. https://gerrit.chromium.org/gerrit/32243/ corrects this issue, so once that CL lands this CL is good to go. I confirmed the combination of CLs works correctly.
This is the original CL description:
This adds a boolean to the video update message sent to powerd that indicates whether or not the video playing window is fullscreen'd. This is information used in powerd for controlling when to enable/disable keyboard backlights. Specifically if video is playing and fullscreen we assume that the user is doing something like watching a movie so will disable backlight.
An additional change that I have made is to convert the message being sent from an int64 to a protobuffer that contains a int64 and a boolean. Currently, since the receiver of this message does not live in the same repo it means that every time one wants to change the message format there is a small window where bad builds can be generated or one has to land two changes to the other repo, one to add handling of the new message format and then a second one to remove the old format. Converting to a protobuffer means that for common cases this issues are removed.
This CL depends on CLs for power_manager (https://gerrit.chromium.org/gerrit/32092/), one for system_api (https://gerrit.chromium.org/gerrit/31916), and a DEPS roll(http://codereview.chromium.org/10915032/). There will be future CLs for power_manager that will use this information and more thourghly tests this code.
BUG=chrome-os-partner:9203
TEST=Run updated unittests.
Confirm there is no messages in the cros logs about this method being mishandled.
Make sure that powerd is not crashing due to protocol buffer processing failing.
Review URL: https://chromiumcodereview.appspot.com/10913134
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@156008 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TBR=oshima@chromium.org
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10912184
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155807 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
sleep/suspend.
In either case, the display info will be updated properly
when they're turned on, so no need to change them when disconnected.
BUG=145627, 146899
TEST=added new tests. also tested manually. See bug for repro step (145627)
Review URL: https://chromiumcodereview.appspot.com/10917159
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155798 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Does not rename things from the flimflam:: namespace in service_constants.h: that will be saved for a later pass.
I think this counts as refactoring, so I'm TBR'ing the OWNERS checks.
TBR=zelidrag@chromium.org,glotov@chromium.org
BUG=chromium:146616
TEST=built and ran
Review URL: https://chromiumcodereview.appspot.com/10915106
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155477 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
BUG=139266
TEST=manual
Review URL: https://chromiumcodereview.appspot.com/10915055
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155313 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
http://build.chromium.org/p/chromium/builders/Win%20Aura/builds/21280/steps/ash_unittests/logs/stdio#failure1
[ RUN ] VideoDetectorTest.FullscreenWindow
wm\video_detector_unittest.cc(250): error: Value of:
observer_->num_invocations()
Actual: 0
Expected: 1
wm\video_detector_unittest.cc(251): error: Value of:
observer_->num_fullscreens()
Actual: 0
Expected: 1
[ FAILED ] VideoDetectorTest.FullscreenWindow (138 ms)
This is a re-attempt at http://codereview.chromium.org/10905026/, which ended up
being reverted.
The crash that caused the revert was actually due to powerd not being able to
parse the protobuf correctly. https://gerrit.chromium.org/gerrit/32243/ corrects
this issue, so once that CL lands this CL is good to go. I confirmed the
combination of CLs works correctly.
This is the original CL description:
This adds a boolean to the video update message sent to powerd that indicates
whether or not the video playing window is fullscreen'd. This is information
used in powerd for controlling when to enable/disable keyboard
backlights. Specifically if video is playing and fullscreen we assume that the
user is doing something like watching a movie so will disable backlight.
An additional change that I have made is to convert the message being sent from
an int64 to a protobuffer that contains a int64 and a boolean. Currently, since
the receiver of this message does not live in the same repo it means that
everytime one wants to change the message format there is a small window where
bad builds can be generated or one has to land two changes to the other repo,
one to add handling of the new message format and then a second one to remove
the old format. Converting to a protobuffer means that for common cases this
issues are removed.
This CL depends on CLs for power_manager
(https://gerrit.chromium.org/gerrit/32092/), one for system_api
(https://gerrit.chromium.org/gerrit/31916), and a DEPS
roll(http://codereview.chromium.org/10915032/). There will be future CLs for
power_manager that will use this information and more thourghly tests this code.
BUG=chrome-os-partner:9203
TEST=Run updated unittests.
Confirm there is no messages in the cros logs about this method being
mishandled.
Make sure that powerd is not crashing due to protocol buffer processing
failing.
Review URL: https://chromiumcodereview.appspot.com/10916123
TBR=rharrison@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10928037
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155232 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This is a re-attempt at http://codereview.chromium.org/10905026/, which ended up
being reverted.
The crash that caused the revert was actually due to powerd not being able to
parse the protobuf correctly. https://gerrit.chromium.org/gerrit/32243/ corrects
this issue, so once that CL lands this CL is good to go. I confirmed the
combination of CLs works correctly.
This is the original CL description:
This adds a boolean to the video update message sent to powerd that indicates
whether or not the video playing window is fullscreen'd. This is information
used in powerd for controlling when to enable/disable keyboard
backlights. Specifically if video is playing and fullscreen we assume that the
user is doing something like watching a movie so will disable backlight.
An additional change that I have made is to convert the message being sent from
an int64 to a protobuffer that contains a int64 and a boolean. Currently, since
the receiver of this message does not live in the same repo it means that
everytime one wants to change the message format there is a small window where
bad builds can be generated or one has to land two changes to the other repo,
one to add handling of the new message format and then a second one to remove
the old format. Converting to a protobuffer means that for common cases this
issues are removed.
This CL depends on CLs for power_manager
(https://gerrit.chromium.org/gerrit/32092/), one for system_api
(https://gerrit.chromium.org/gerrit/31916), and a DEPS
roll(http://codereview.chromium.org/10915032/). There will be future CLs for
power_manager that will use this information and more thourghly tests this code.
BUG=chrome-os-partner:9203
TEST=Run updated unittests.
Confirm there is no messages in the cros logs about this method being
mishandled.
Make sure that powerd is not crashing due to protocol buffer processing
failing.
Review URL: https://chromiumcodereview.appspot.com/10916123
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@155224 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Breaks cros_x86, cros_amd64 buildbots:
http://build.chromium.org/p/chromium.chromiumos/builders/ChromiumOS%20%28x86%29/builds/8115
This adds a boolean to the video update message sent to powerd that indicates whether or not the video playing window is fullscreen'd. This is information used in powerd for controlling when to enable/disable keyboard backlights. Specifically if video is playing and fullscreen we assume that the user is doing something like watching a movie so will disable backlight.
An additional change that I have made is to convert the message being sent from an int64 to a protobuffer that contains a int64 and a boolean. Currently, since the receiver of this message does not live in the same repo it means that every time one wants to change the message format there is a small window where bad builds can be generated or one has to land two changes to the other repo, one to add handling of the new message format and then a second one to remove the old format. Converting to a protobuffer means that for common cases this issues are removed.
This CL depends on CLs for power_manager (https://gerrit.chromium.org/gerrit/32092/), one for system_api (https://gerrit.chromium.org/gerrit/31916), and a DEPS roll(http://codereview.chromium.org/10915032/). There will be future CLs for power_manager that will use this information and test this code.
BUG=chrome-os-partner:9203
TEST=Run updated unittests.
Confirm there is no messages in the cros logs about this method being mishandled.
Review URL: https://chromiumcodereview.appspot.com/10905026
TBR=rharrison@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10915089
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154952 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This adds a boolean to the video update message sent to powerd that indicates whether or not the video playing window is fullscreen'd. This is information used in powerd for controlling when to enable/disable keyboard backlights. Specifically if video is playing and fullscreen we assume that the user is doing something like watching a movie so will disable backlight.
An additional change that I have made is to convert the message being sent from an int64 to a protobuffer that contains a int64 and a boolean. Currently, since the receiver of this message does not live in the same repo it means that every time one wants to change the message format there is a small window where bad builds can be generated or one has to land two changes to the other repo, one to add handling of the new message format and then a second one to remove the old format. Converting to a protobuffer means that for common cases this issues are removed.
This CL depends on CLs for power_manager (https://gerrit.chromium.org/gerrit/32092/), one for system_api (https://gerrit.chromium.org/gerrit/31916), and a DEPS roll(http://codereview.chromium.org/10915032/). There will be future CLs for power_manager that will use this information and test this code.
BUG=chrome-os-partner:9203
TEST=Run updated unittests.
Confirm there is no messages in the cros logs about this method being mishandled.
Review URL: https://chromiumcodereview.appspot.com/10905026
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154905 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
BUG=None
TEST=build
TBR=ben@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10878061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153410 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
GetSizeStatsOnFileThread is moved from disk_mount_manager.cc to file_browser_private_api.cc
BUG=None
TEST=build
TBR=estade@chromium.org, kaznacheev@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10874067
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@153402 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
| |
BUG=chromium-os:29557
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10825170
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@150382 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=126947
TEST=chromeos_unittests and try bots
Review URL: https://chromiumcodereview.appspot.com/10806028
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@148068 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
With this CL, these classes are compiled and tested but not in use production binary at this moment.
BUG=126947
TEST=chromeos_unittests,ui_unittests
Review URL: https://chromiumcodereview.appspot.com/10693144
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@147176 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Let me divide CL from actual implementation and interface definition for CL simplicity.
This CL contains only interface definition and stub implementation of it. And also contains method handler accesser implementation.
BUG=126947
TEST=chromeos_unittests,ui_unittests
Review URL: https://chromiumcodereview.appspot.com/10757016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@146955 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusComponent is one of representation an obeject used in communication with ibus-daemon.
IBusComponent is used when the engine object register itself to ibus-daemon.
With this CL, ibus_component.cc will be comipled and tested but not in used production binary at this moment.
BUG=126947
TEST=chromeos_unittests, unit_tests, dbus_unittests
Review URL: https://chromiumcodereview.appspot.com/10692120
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145660 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Rename the remaining usage of Monitor to Display
BUG=none
TEST=none
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=144499
Review URL: https://chromiumcodereview.appspot.com/10675011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144585 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Temporarily reverting rename change to investigate 133784
TBR=oshima@chromium.org
BUG=123160
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10689014
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144573 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
| |
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/10675011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@144499 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusProperty and IBusPropertyList are one of representations an obeject used in communication with ibus-daemon.
According to this CL, ibus_property.cc will be comipled and tested but not in
used production binary at thi moment.
BUG=chromium-os:26334
TEST=chromeos_unittests, unit_tests, dbus_unittests
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=143355
Review URL: https://chromiumcodereview.appspot.com/10383253
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143545 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusProperty and IBusPropertyList are one of representations an obeject used in communication with ibus-daemon.
According to this CL, ibus_property.cc will be comipled and tested but not in
used production binary at thi moment.
BUG=chromium-os:26334
TEST=chromeos_unittests, unit_tests, dbus_unittests
Review URL: https://chromiumcodereview.appspot.com/10383253
TBR=nona@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10619003
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143358 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusProperty and IBusPropertyList are one of representations an obeject used in communication with ibus-daemon.
According to this CL, ibus_property.cc will be comipled and tested but not in
used production binary at thi moment.
BUG=chromium-os:26334
TEST=chromeos_unittests, unit_tests, dbus_unittests
Review URL: https://chromiumcodereview.appspot.com/10383253
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@143355 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
The reason for introducing this class is that gmock is banned in some directory like ui/*.
Currently each mock object has just a empty implementation.
BUG=126947
TEST=chromeos_unittests, unit_tests, aura_shell_unittests, aura_unittests
Review URL: https://chromiumcodereview.appspot.com/10536167
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142352 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
| |
TEST=none (though tests are included with the extension APIs)
BUG=119473
Review URL: https://chromiumcodereview.appspot.com/10546010
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@142337 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Add dbus support the org.freedesktop.ModemManager1.SMS and
org.freedesktop.ModemManager1.Modem.Messaging interfaces.
This CL is authored by Jason Glasgow <jglasgow@chromium.org> and
reviewed on https://chromiumcodereview.appspot.com/10533006
BUG=chromium-os:28421
TEST=chromeos_unittests
Review URL: https://chromiumcodereview.appspot.com/10545133
Patch from Jason Glasgow <jglasgow@chromium.org>.
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@141634 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
New MonitorConfigurator class interprets events (ctrl-F4, display add/remove, and brightness-related CRTC state changes).
Ctrl-F4 events come in through the AcceleratorController.
Display add/remove events are interpretted by MonitorChangeObserverX11.
CRTC enable/disable events related to power management come over DBus and are interpretted by the new OutputObserver class.
Note the the chromeos/monitor/DEPS change can be removed once the now-unused code is retired from powerd (since that is the only reason for a DBus call-out).
Valgrind suppressions.txt was updated because this change re-orders our initial XRR calls such that a different top-level call is blamed for its leak.
BUG=chromium:126493
TEST=Manually tested with an additional display and mode switching on Lumpy.
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=140031
Review URL: https://chromiumcodereview.appspot.com/10384103
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140203 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
New MonitorConfigurator class interprets events (ctrl-F4, display add/remove, and brightness-related CRTC state changes).
Ctrl-F4 events come in through the AcceleratorController.
Display add/remove events are interpretted by MonitorChangeObserverX11.
CRTC enable/disable events related to power management come over DBus and are interpretted by the new OutputObserver class.
BUG=chromium:126493
TEST=Manually tested with an additional display and mode switching on Lumpy.
Review URL: https://chromiumcodereview.appspot.com/10384103
TBR=disher@chromium.org
Review URL: https://chromiumcodereview.appspot.com/10459076
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140035 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
New MonitorConfigurator class interprets events (ctrl-F4, display add/remove, and brightness-related CRTC state changes).
Ctrl-F4 events come in through the AcceleratorController.
Display add/remove events are interpretted by MonitorChangeObserverX11.
CRTC enable/disable events related to power management come over DBus and are interpretted by the new OutputObserver class.
BUG=chromium:126493
TEST=Manually tested with an additional display and mode switching on Lumpy.
Review URL: https://chromiumcodereview.appspot.com/10384103
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@140031 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since XmlReader was moved from chrome/ to third_party/libxml/, now chromeos/ can depend on it.
BUG=119583
TEST=chromeos_unittests
Review URL: https://chromiumcodereview.appspot.com/10440013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@139520 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusLookupTable is one of representations an obeject used in communication with
ibus-daemon.
According to this CL, ibus_lookup_table will be comipled and tested but not in
used production binary at thi moment.
BUG=chromium-os:26334
TEST=chromeos_unittests, unit_tests, dbus_unittests
Review URL: https://chromiumcodereview.appspot.com/10392039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138507 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusInputContextClient performs performs dbus communication with following spec.
Target Server: ibus-daemon
Service: org.freedesktop.IBus
ObjectPath: /org/freedesktop/IBus/InputContext_XXXX
(The object path is not determined until call CreateInputContext method, please
read commend for detail)
At this moment, all signal handler and method call implementation is not used in
production. According to this CL, IBusInputContextClient is
compiled and tested by chromeos_unittests.
BUG=chromium-os:26334
TEST=unit_tests,chromeos_unittests,dbus_unittests
Review URL: https://chromiumcodereview.appspot.com/10310090
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@138484 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
With the current setup, if you have a header file my_class.h
class BASE_EXPORT MyClass {
public:
void MyInlineMethod() { /* do stuff, inline */ }
};
then every cc file that includes my_class.h will have a public symbol
for MyInlineMethod (because inline methods need to be emitted to
every translation unit, and the linker sorts them out). With the
components build, the linker can't decide to drop these inline methods,
so every .so that uses this header file will have the same public symbol.
With this proposed change, the symbol will only be visible in the target
the header file belongs to, and it will be hidden in all other components.
That's cleaner, and it also prevents accident hidden dependencies (say
target A depends on B, and B depends on C. A accidentally uses an inline
function from a class in C. With this change, that would result in a linker
error, and an explicit dependency from A on C would have to be added).
Also add a missing CHROMEOS_IMPLEMENTATION define which went
unnoticed until now.
BUG=90078
TEST=Things still build.
TBR=ben, tony, viettrungluu, thestig, agl, willchan
Review URL: https://chromiumcodereview.appspot.com/10386108
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137130 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Includes stub implementations for flimflam Device and Manager to support SMS.
BUG=124724
TEST=Run NetworkSmsHandlerTest in chromeos_unittests
Review URL: https://chromiumcodereview.appspot.com/10310095
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@137049 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The IBusText is corresponding to text object used in dbus communication with
ibus-daemon.
According to this CL, IBusText will be compiled and tested by
chromeos_unittests, but not used in production binary at this moment.
This CL is dependent to http://codereview.chromium.org/10332022/
TEST=chromeos_unittests, unit_tests, dbus_unittests
BUG=chromium-os:26334
Review URL: https://chromiumcodereview.appspot.com/10381025
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136746 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The ibus-daemon sends their objects as variant type.
The IBusObjectReader/IBusObjectWriter provides I/O with them.
This CL is compiled and is tested by chromeos_unittests, but not in used in
production binary at this moment.
TEST=chromeos_unittests, unit_tests, dbus_unittests
BUG=chromium-os:26334
Review URL: https://chromiumcodereview.appspot.com/10332022
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136729 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
IBusClient performs dbus communication with following spec.
Target Server: ibus-daemon
Service: org.freedesktop.IBus
ObjectPath: /org/freedesktop/IBus
At this moment, CreateInputContext is implementated with unittest.
According to this CL, IBusClient is compiled and tested by chromeos_unittests
but not actually in used production binary.
TEST=unit_tests,chromeos_unittests,dbus_unittests
BUG=chromium-os:26334
Review URL: https://chromiumcodereview.appspot.com/10342011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136652 0039d316-1c4b-4281-b951-d872f2087c98
|
|
|
|
|
|
|
|
|
|
|
| |
Change-Id: I4730996e6a860833ce7e66792d62dbd1f6db00ea
BUG=none
TEST=Add --dbus-stub to /sbin/session_manager_startup.sh; dbus stub implementation should be used for non-critical components (e.g. power manager)
Review URL: https://chromiumcodereview.appspot.com/10392029
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@136629 0039d316-1c4b-4281-b951-d872f2087c98
|