| Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // | 
|  | 2 | // Copyright (C) 2012 The Android Open Source Project | 
|  | 3 | // | 
|  | 4 | // Licensed under the Apache License, Version 2.0 (the "License"); | 
|  | 5 | // you may not use this file except in compliance with the License. | 
|  | 6 | // You may obtain a copy of the License at | 
|  | 7 | // | 
|  | 8 | //      http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 9 | // | 
|  | 10 | // Unless required by applicable law or agreed to in writing, software | 
|  | 11 | // distributed under the License is distributed on an "AS IS" BASIS, | 
|  | 12 | // WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | 
|  | 13 | // See the License for the specific language governing permissions and | 
|  | 14 | // limitations under the License. | 
|  | 15 | // | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 16 |  | 
| Alex Deymo | aab50e3 | 2014-11-10 19:55:35 -0800 | [diff] [blame] | 17 | #include "update_engine/subprocess.h" | 
|  | 18 |  | 
| Alex Deymo | 5d52780 | 2014-07-18 14:24:13 -0700 | [diff] [blame] | 19 | #include <fcntl.h> | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 20 | #include <netinet/in.h> | 
|  | 21 | #include <netinet/ip.h> | 
|  | 22 | #include <poll.h> | 
|  | 23 | #include <sys/socket.h> | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 24 | #include <sys/stat.h> | 
|  | 25 | #include <sys/types.h> | 
|  | 26 | #include <unistd.h> | 
| Gilad Arnold | 8e3f126 | 2013-01-08 14:59:54 -0800 | [diff] [blame] | 27 |  | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 28 | #include <set> | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 29 | #include <string> | 
|  | 30 | #include <vector> | 
| Gilad Arnold | 8e3f126 | 2013-01-08 14:59:54 -0800 | [diff] [blame] | 31 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 32 | #include <base/bind.h> | 
|  | 33 | #include <base/location.h> | 
| Alex Deymo | 0b3db6b | 2015-08-10 15:19:37 -0700 | [diff] [blame] | 34 | #include <base/message_loop/message_loop.h> | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 35 | #include <base/strings/string_util.h> | 
|  | 36 | #include <base/strings/stringprintf.h> | 
|  | 37 | #include <base/time/time.h> | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 38 | #include <chromeos/bind_lambda.h> | 
| Alex Deymo | 0b3db6b | 2015-08-10 15:19:37 -0700 | [diff] [blame] | 39 | #include <chromeos/message_loops/base_message_loop.h> | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 40 | #include <chromeos/message_loops/message_loop.h> | 
|  | 41 | #include <chromeos/message_loops/message_loop_utils.h> | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 42 | #include <chromeos/strings/string_utils.h> | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 43 | #include <gtest/gtest.h> | 
| Gilad Arnold | 8e3f126 | 2013-01-08 14:59:54 -0800 | [diff] [blame] | 44 |  | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 45 | #include "update_engine/test_utils.h" | 
|  | 46 | #include "update_engine/utils.h" | 
|  | 47 |  | 
| Gilad Arnold | 8e3f126 | 2013-01-08 14:59:54 -0800 | [diff] [blame] | 48 | using base::TimeDelta; | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 49 | using chromeos::MessageLoop; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 50 | using std::string; | 
|  | 51 | using std::vector; | 
|  | 52 |  | 
|  | 53 | namespace chromeos_update_engine { | 
|  | 54 |  | 
|  | 55 | class SubprocessTest : public ::testing::Test { | 
|  | 56 | protected: | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 57 | void SetUp() override { | 
|  | 58 | loop_.SetAsCurrent(); | 
| Alex Deymo | b7ca096 | 2014-10-01 17:58:07 -0700 | [diff] [blame] | 59 | async_signal_handler_.Init(); | 
|  | 60 | subprocess_.Init(&async_signal_handler_); | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 61 | } | 
|  | 62 |  | 
| Alex Deymo | 0b3db6b | 2015-08-10 15:19:37 -0700 | [diff] [blame] | 63 | base::MessageLoopForIO base_loop_; | 
|  | 64 | chromeos::BaseMessageLoop loop_{&base_loop_}; | 
| Alex Deymo | b7ca096 | 2014-10-01 17:58:07 -0700 | [diff] [blame] | 65 | chromeos::AsynchronousSignalHandler async_signal_handler_; | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 66 | Subprocess subprocess_; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 67 | }; | 
|  | 68 |  | 
|  | 69 | namespace { | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 70 |  | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 71 | int local_server_port = 0; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 72 |  | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 73 | void ExpectedResults(int expected_return_code, const string& expected_output, | 
|  | 74 | int return_code, const string& output) { | 
|  | 75 | EXPECT_EQ(expected_return_code, return_code); | 
|  | 76 | EXPECT_EQ(expected_output, output); | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 77 | MessageLoop::current()->BreakLoop(); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 78 | } | 
|  | 79 |  | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 80 | void ExpectedEnvVars(int return_code, const string& output) { | 
| Darin Petkov | 6f03a3b | 2010-11-10 14:27:14 -0800 | [diff] [blame] | 81 | EXPECT_EQ(0, return_code); | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 82 | const std::set<string> allowed_envs = {"LD_LIBRARY_PATH", "PATH"}; | 
|  | 83 | for (string key_value : chromeos::string_utils::Split(output, "\n")) { | 
|  | 84 | auto key_value_pair = chromeos::string_utils::SplitAtFirst( | 
|  | 85 | key_value, "=", true); | 
|  | 86 | EXPECT_NE(allowed_envs.end(), allowed_envs.find(key_value_pair.first)); | 
|  | 87 | } | 
| Alex Deymo | 29b8153 | 2015-07-09 11:51:49 -0700 | [diff] [blame] | 88 | MessageLoop::current()->BreakLoop(); | 
|  | 89 | } | 
|  | 90 |  | 
| Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 91 | }  // namespace | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 92 |  | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 93 | TEST_F(SubprocessTest, IsASingleton) { | 
|  | 94 | EXPECT_EQ(&subprocess_, &Subprocess::Get()); | 
|  | 95 | } | 
|  | 96 |  | 
|  | 97 | TEST_F(SubprocessTest, InactiveInstancesDontChangeTheSingleton) { | 
|  | 98 | std::unique_ptr<Subprocess> another_subprocess(new Subprocess()); | 
|  | 99 | EXPECT_EQ(&subprocess_, &Subprocess::Get()); | 
|  | 100 | another_subprocess.reset(); | 
|  | 101 | EXPECT_EQ(&subprocess_, &Subprocess::Get()); | 
|  | 102 | } | 
|  | 103 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 104 | TEST_F(SubprocessTest, SimpleTest) { | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 105 | EXPECT_TRUE(subprocess_.Exec({"/bin/false"}, | 
|  | 106 | base::Bind(&ExpectedResults, 1, ""))); | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 107 | loop_.Run(); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 108 | } | 
|  | 109 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 110 | TEST_F(SubprocessTest, EchoTest) { | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 111 | EXPECT_TRUE(subprocess_.Exec( | 
|  | 112 | {"/bin/sh", "-c", "echo this is stdout; echo this is stderr >&2"}, | 
|  | 113 | base::Bind(&ExpectedResults, 0, "this is stdout\nthis is stderr\n"))); | 
| Alex Deymo | 29b8153 | 2015-07-09 11:51:49 -0700 | [diff] [blame] | 114 | loop_.Run(); | 
|  | 115 | } | 
|  | 116 |  | 
|  | 117 | TEST_F(SubprocessTest, StderrNotIncludedInOutputTest) { | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 118 | EXPECT_TRUE(subprocess_.ExecFlags( | 
|  | 119 | {"/bin/sh", "-c", "echo on stdout; echo on stderr >&2"}, | 
|  | 120 | 0, | 
|  | 121 | base::Bind(&ExpectedResults, 0, "on stdout\n"))); | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 122 | loop_.Run(); | 
| Darin Petkov | 6f03a3b | 2010-11-10 14:27:14 -0800 | [diff] [blame] | 123 | } | 
|  | 124 |  | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 125 | TEST_F(SubprocessTest, EnvVarsAreFiltered) { | 
|  | 126 | EXPECT_TRUE(subprocess_.Exec({"/usr/bin/env"}, base::Bind(&ExpectedEnvVars))); | 
|  | 127 | loop_.Run(); | 
|  | 128 | } | 
|  | 129 |  | 
|  | 130 | TEST_F(SubprocessTest, SynchronousTrueSearchsOnPath) { | 
|  | 131 | int rc = -1; | 
|  | 132 | EXPECT_TRUE(Subprocess::SynchronousExecFlags( | 
|  | 133 | {"true"}, Subprocess::kSearchPath, &rc, nullptr)); | 
|  | 134 | EXPECT_EQ(0, rc); | 
|  | 135 | } | 
|  | 136 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 137 | TEST_F(SubprocessTest, SynchronousEchoTest) { | 
|  | 138 | vector<string> cmd = { | 
|  | 139 | "/bin/sh", | 
|  | 140 | "-c", | 
|  | 141 | "echo -n stdout-here; echo -n stderr-there > /dev/stderr"}; | 
| Darin Petkov | 85d02b7 | 2011-05-17 13:25:51 -0700 | [diff] [blame] | 142 | int rc = -1; | 
|  | 143 | string stdout; | 
|  | 144 | ASSERT_TRUE(Subprocess::SynchronousExec(cmd, &rc, &stdout)); | 
|  | 145 | EXPECT_EQ(0, rc); | 
|  | 146 | EXPECT_EQ("stdout-herestderr-there", stdout); | 
|  | 147 | } | 
|  | 148 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 149 | TEST_F(SubprocessTest, SynchronousEchoNoOutputTest) { | 
| Darin Petkov | 85d02b7 | 2011-05-17 13:25:51 -0700 | [diff] [blame] | 150 | int rc = -1; | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 151 | ASSERT_TRUE(Subprocess::SynchronousExec( | 
|  | 152 | {"/bin/sh", "-c", "echo test"}, | 
|  | 153 | &rc, nullptr)); | 
| Darin Petkov | 85d02b7 | 2011-05-17 13:25:51 -0700 | [diff] [blame] | 154 | EXPECT_EQ(0, rc); | 
|  | 155 | } | 
|  | 156 |  | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 157 | namespace { | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 158 | void CallbackBad(int return_code, const string& output) { | 
|  | 159 | ADD_FAILURE() << "should never be called."; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 160 | } | 
|  | 161 |  | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 162 | // TODO(garnold) this test method uses test_http_server as a representative for | 
|  | 163 | // interactive processes that can be spawned/terminated at will. This causes us | 
|  | 164 | // to go through hoops when spawning this process (e.g. obtaining the port | 
|  | 165 | // number it uses so we can control it with wget). It would have been much | 
|  | 166 | // preferred to use something else and thus simplify both test_http_server | 
|  | 167 | // (doesn't have to be able to communicate through a temp file) and the test | 
|  | 168 | // code below; for example, it sounds like a brain dead sleep loop with proper | 
|  | 169 | // signal handlers could be used instead. | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 170 | void StartAndCancelInRunLoop(bool* spawned) { | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 171 | // Create a temp file for test_http_server to communicate its port number. | 
|  | 172 | char temp_file_name[] = "/tmp/subprocess_unittest-test_http_server-XXXXXX"; | 
|  | 173 | int temp_fd = mkstemp(temp_file_name); | 
|  | 174 | CHECK_GE(temp_fd, 0); | 
|  | 175 | int temp_flags = fcntl(temp_fd, F_GETFL, 0) | O_NONBLOCK; | 
|  | 176 | CHECK_EQ(fcntl(temp_fd, F_SETFL, temp_flags), 0); | 
|  | 177 |  | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 178 | vector<string> cmd; | 
|  | 179 | cmd.push_back("./test_http_server"); | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 180 | cmd.push_back(temp_file_name); | 
| Alex Deymo | 461b259 | 2015-07-24 20:10:52 -0700 | [diff] [blame] | 181 | uint32_t tag = Subprocess::Get().Exec(cmd, base::Bind(&CallbackBad)); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 182 | EXPECT_NE(0, tag); | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 183 | *spawned = true; | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 184 | printf("test http server spawned\n"); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 185 | // Wait for server to be up and running | 
| Gilad Arnold | 8e3f126 | 2013-01-08 14:59:54 -0800 | [diff] [blame] | 186 | TimeDelta total_wait_time; | 
|  | 187 | const TimeDelta kSleepTime = TimeDelta::FromMilliseconds(100); | 
|  | 188 | const TimeDelta kMaxWaitTime = TimeDelta::FromSeconds(3); | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 189 | local_server_port = 0; | 
|  | 190 | static const char* kServerListeningMsgPrefix = "listening on port "; | 
|  | 191 | while (total_wait_time.InMicroseconds() < kMaxWaitTime.InMicroseconds()) { | 
|  | 192 | char line[80]; | 
|  | 193 | int line_len = read(temp_fd, line, sizeof(line) - 1); | 
|  | 194 | if (line_len > 0) { | 
|  | 195 | line[line_len] = '\0'; | 
|  | 196 | CHECK_EQ(strstr(line, kServerListeningMsgPrefix), line); | 
|  | 197 | const char* listening_port_str = | 
|  | 198 | line + strlen(kServerListeningMsgPrefix); | 
|  | 199 | char* end_ptr; | 
| Alex Vakulenko | d2779df | 2014-06-16 13:19:00 -0700 | [diff] [blame] | 200 | long raw_port = strtol(listening_port_str,  // NOLINT(runtime/int) | 
|  | 201 | &end_ptr, 10); | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 202 | CHECK(!*end_ptr || *end_ptr == '\n'); | 
|  | 203 | local_server_port = static_cast<in_port_t>(raw_port); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 204 | break; | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 205 | } else if (line_len < 0 && errno != EAGAIN) { | 
|  | 206 | LOG(INFO) << "error reading from " << temp_file_name << ": " | 
|  | 207 | << strerror(errno); | 
|  | 208 | break; | 
|  | 209 | } | 
| Alex Deymo | ea4444d | 2015-08-07 10:51:18 -0700 | [diff] [blame] | 210 | usleep(kSleepTime.InMicroseconds()); | 
| Darin Petkov | 27fa9c5 | 2010-07-15 15:11:55 -0700 | [diff] [blame] | 211 | total_wait_time += kSleepTime; | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 212 | } | 
| Gilad Arnold | b6c562a | 2013-07-01 02:19:26 -0700 | [diff] [blame] | 213 | close(temp_fd); | 
|  | 214 | remove(temp_file_name); | 
|  | 215 | CHECK_GT(local_server_port, 0); | 
|  | 216 | LOG(INFO) << "server listening on port " << local_server_port; | 
| Alex Deymo | 29b8153 | 2015-07-09 11:51:49 -0700 | [diff] [blame] | 217 | Subprocess::Get().KillExec(tag); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 218 | } | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 219 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 220 | void ExitWhenDone(bool* spawned) { | 
|  | 221 | if (*spawned && !Subprocess::Get().SubprocessInFlight()) { | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 222 | // tear down the sub process | 
|  | 223 | printf("tear down time\n"); | 
| Alex Deymo | 10875d9 | 2014-11-10 21:52:57 -0800 | [diff] [blame] | 224 | int status = test_utils::System( | 
| Alex Vakulenko | 75039d7 | 2014-03-25 12:36:28 -0700 | [diff] [blame] | 225 | base::StringPrintf("wget -O /dev/null http://127.0.0.1:%d/quitquitquit", | 
|  | 226 | local_server_port)); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 227 | EXPECT_NE(-1, status) << "system() failed"; | 
|  | 228 | EXPECT_TRUE(WIFEXITED(status)) | 
|  | 229 | << "command failed to run or died abnormally"; | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 230 | MessageLoop::current()->BreakLoop(); | 
|  | 231 | } else { | 
|  | 232 | // Re-run this callback again in 10 ms. | 
|  | 233 | MessageLoop::current()->PostDelayedTask( | 
|  | 234 | FROM_HERE, | 
|  | 235 | base::Bind(&ExitWhenDone, spawned), | 
|  | 236 | TimeDelta::FromMilliseconds(10)); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 237 | } | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 238 | } | 
|  | 239 |  | 
| Alex Deymo | 60ca1a7 | 2015-06-18 18:19:15 -0700 | [diff] [blame] | 240 | }  // namespace | 
|  | 241 |  | 
|  | 242 | TEST_F(SubprocessTest, CancelTest) { | 
|  | 243 | bool spawned = false; | 
|  | 244 | loop_.PostDelayedTask( | 
|  | 245 | FROM_HERE, | 
|  | 246 | base::Bind(&StartAndCancelInRunLoop, &spawned), | 
|  | 247 | TimeDelta::FromMilliseconds(100)); | 
|  | 248 | loop_.PostDelayedTask( | 
|  | 249 | FROM_HERE, | 
|  | 250 | base::Bind(&ExitWhenDone, &spawned), | 
|  | 251 | TimeDelta::FromMilliseconds(10)); | 
|  | 252 | loop_.Run(); | 
|  | 253 | // This test would leak a callback that runs when the child process exits | 
|  | 254 | // unless we wait for it to run. | 
|  | 255 | chromeos::MessageLoopRunUntil( | 
|  | 256 | &loop_, | 
|  | 257 | TimeDelta::FromSeconds(10), | 
|  | 258 | base::Bind([] { | 
|  | 259 | return Subprocess::Get().subprocess_records_.empty(); | 
|  | 260 | })); | 
| adlr@google.com | 3defe6a | 2009-12-04 20:57:17 +0000 | [diff] [blame] | 261 | } | 
|  | 262 |  | 
|  | 263 | }  // namespace chromeos_update_engine |