Status: Assigned Owner: erik...@chromium.org CC: a...@chromium.org Labels: Type-Bug Pri-1 OS-All Area-Misc Size-Medium Regression
New issue 18240 by fin...@chromium.org: Regression: r21621 (add mole info...) breaks page action event setup http://code.google.com/p/chromium/issues/detail?id=18240 The changelist to "add mole info to toolstrip definition and enable moles" (checked in as r21621)... http://codereview.chromium.org/159202 ... seems to consistently break the extension specific event changelist (r20782), specifically the setup of page action events. After change r21621, RenderThread::OnPageActionsUpdated is called _after_ the js code calls the native function GetCurrentPageActions() and it therefore thinks no page actions exist (because the data hasn't been setup yet). I glanced at the changelist and nothing obvious stuck out, but I'm out of time (and on vacation) :) Erik, can I ask you to take a look? This may be due to a design flaw in how this was implemented to begin with, CC-ing Aaron for comments (in case it is). You should be able to test this with the sample RSS page action: src\chrome\test\data\extensions\samples\subscribe_page_action -- You received this message because you are listed in the owner or CC fields of this issue, or because you starred this issue. You may adjust your issue notification preferences at: http://code.google.com/hosting/settings --~--~---------~--~----~------------~-------~--~----~ Automated mail from issue updates at http://crbug.com/ Subscription options: http://groups.google.com/group/chromium-bugs -~----------~----~----~----~------~----~------~--~---
