Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(396)

Issue 286993003: Fixed case when DevtoolsAdbBridge may start few polling loops. (Closed)

Created:
6 years, 7 months ago by Dmitry Zvorygin
Modified:
6 years, 6 months ago
CC:
chromium-reviews, vsevik, yurys, paulirish+reviews_chromium.org, devtools-reviews_chromium.org, aandrey+blink_chromium.org, pfeldman
Visibility:
Public.

Description

Fixed case when DevtoolsAdbBridge may start few polling loops. BUG=

Patch Set 1 #

Patch Set 2 : Fixed addressed comments. #

Patch Set 3 : Removed explicit timeouts. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+335 lines, -33 lines) Patch
A chrome/browser/devtools/device/DEPS View 1 chunk +5 lines, -0 lines 0 comments Download
M chrome/browser/devtools/device/devtools_android_bridge.h View 1 2 5 chunks +19 lines, -0 lines 0 comments Download
M chrome/browser/devtools/device/devtools_android_bridge.cc View 1 2 10 chunks +68 lines, -33 lines 0 comments Download
A chrome/browser/devtools/device/devtools_android_bridge_browser_test.cc View 1 2 1 chunk +242 lines, -0 lines 0 comments Download
M chrome/chrome_tests.gypi View 1 2 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
Dmitry Zvorygin
Please take a look.
6 years, 7 months ago (2014-05-15 10:09:50 UTC) #1
Dmitry Zvorygin
Please take a look.
6 years, 7 months ago (2014-05-19 12:33:54 UTC) #2
Dmitry Zvorygin
Ping?
6 years, 7 months ago (2014-05-21 11:19:26 UTC) #3
Dmitry Zvorygin
6 years, 7 months ago (2014-05-26 14:59:44 UTC) #4
Please take a look.

Powered by Google App Engine
This is Rietveld 408576698